commit faa02f0daf6bc11bf480ada1987088416132f93c
author Wan-Teh Chang <wtc@google.com> Thu Apr 22 13:08:09 2021 -0700
committer Wan-Teh Chang <wtc@google.com> Thu Apr 22 22:28:26 2021 +0000
tree 1df92073b1faabea0017795291bb5d86ab57d9b9
parent 9164dee4f63060cb54858fcbdd8b99cabc3f69c2

Remove disable_diagonal_intra from cfg_options_t

Remove the disable_diagonal_intra field from the cfg_options_t struct.
The cfg_options_t struct is a public type and is embedded in the
aom_codec_enc_cfg_t struct, so adding a field to the cfg_options_t
struct breaks binary compatibility with libaom v3.0.0.

BUG=aomedia:3033

Change-Id: Id5c4b47e7a2395220922cdd4c42aa1694b42ebcb

aom/aom_encoder.h

diff --git a/aom/aom_encoder.h b/aom/aom_encoder.h
index beac764..b2926e1 100644
--- a/aom/aom_encoder.h
+++ b/aom/aom_encoder.h
@@ -293,10 +293,6 @@
    *
    */
   unsigned int disable_smooth_intra;
-  /*!\brief disable D45 to D203 intra modes
-   *
-   */
-  unsigned int disable_diagonal_intra;
   /*!\brief disable filter intra
    *
    */

apps/aomenc.c

diff --git a/apps/aomenc.c b/apps/aomenc.c
index 27439f40..03c34b4 100644
--- a/apps/aomenc.c
+++ b/apps/aomenc.c
@@ -1311,7 +1311,6 @@
     SHOW_PARAMS(disable_intrabc);
     SHOW_PARAMS(disable_cfl);
     SHOW_PARAMS(disable_smooth_intra);
-    SHOW_PARAMS(disable_diagonal_intra);
     SHOW_PARAMS(disable_filter_intra);
     SHOW_PARAMS(disable_dual_filter);
     SHOW_PARAMS(disable_intra_angle_delta);

av1/av1_cx_iface.c

diff --git a/av1/av1_cx_iface.c b/av1/av1_cx_iface.c
index 4c49485..7362d3a 100644
--- a/av1/av1_cx_iface.c
+++ b/av1/av1_cx_iface.c
@@ -835,7 +835,6 @@
   extra_cfg->enable_smooth_intra = (cfg->disable_smooth_intra == 0);
   extra_cfg->enable_paeth_intra = (cfg->disable_paeth_intra == 0);
   extra_cfg->enable_cfl_intra = (cfg->disable_cfl == 0);
-  extra_cfg->enable_diagonal_intra = (cfg->disable_diagonal_intra == 0);
   extra_cfg->enable_obmc = (cfg->disable_obmc == 0);
   extra_cfg->enable_palette = (cfg->disable_palette == 0);
   extra_cfg->enable_intrabc = (cfg->disable_intrabc == 0);
@@ -3696,7 +3695,7 @@
       0,                       // use_fixed_qp_offsets
       { -1, -1, -1, -1, -1 },  // fixed_qp_offsets
       { 0, 128, 128, 4, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
-        0, 0,   0,   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 },  // cfg
+        0, 0,   0,   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 },  // cfg
   },
 #endif  // !CONFIG_REALTIME_ONLY
   {
@@ -3767,7 +3766,7 @@
       0,                       // use_fixed_qp_offsets
       { -1, -1, -1, -1, -1 },  // fixed_qp_offsets
       { 0, 128, 128, 4, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
-        0, 0,   0,   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 },  // cfg
+        0, 0,   0,   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 },  // cfg
   },
 #if !CONFIG_REALTIME_ONLY
   {
@@ -3838,7 +3837,7 @@
       0,                       // use_fixed_qp_offsets
       { -1, -1, -1, -1, -1 },  // fixed_qp_offsets
       { 0, 128, 128, 4, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
-        0, 0,   0,   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 },  // cfg
+        0, 0,   0,   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 },  // cfg
   },
 #endif  // !CONFIG_REALTIME_ONLY
 };

common/args.c

diff --git a/common/args.c b/common/args.c
index 64d6e03..ed62294 100644
--- a/common/args.c
+++ b/common/args.c
@@ -92,7 +92,6 @@
     GET_PARAMS(disable_intrabc);
     GET_PARAMS(disable_cfl);
     GET_PARAMS(disable_smooth_intra);
-    GET_PARAMS(disable_diagonal_intra);
     GET_PARAMS(disable_filter_intra);
     GET_PARAMS(disable_dual_filter);
     GET_PARAMS(disable_intra_angle_delta);