commit f8b0e9bd5408812be5c8799c944387f893e9d84d
author Yaowu Xu <yaowu.google.com> Tue Mar 31 13:14:38 2020 -0700
committer Yaowu Xu <yaowu@google.com> Wed Apr 01 03:30:27 2020 +0000
tree d941d2c39509414d9f5c095b5a8a22b08fed7d19
parent a52f507a7219ef0dad2224c1150e8bf9d35f8e91

Use const consistently for function parameters

This fixes MSVC build warnings.

Change-Id: I8bde66cfa7ad4d1b915b805678b80c401d2f9f96

av1/common/quant_common.c

diff --git a/av1/common/quant_common.c b/av1/common/quant_common.c
index e976752..e96d71a 100644
--- a/av1/common/quant_common.c
+++ b/av1/common/quant_common.c
@@ -248,8 +248,8 @@
 // horizontal and vertical directions.
 static INLINE bool is_2d_transform(TX_TYPE tx_type) { return (tx_type < IDTX); }
 
-const qm_val_t *av1_get_iqmatrix(const CommonQuantParams *const quant_params,
-                                 const MACROBLOCKD *const xd, int plane,
+const qm_val_t *av1_get_iqmatrix(const CommonQuantParams *quant_params,
+                                 const MACROBLOCKD *xd, int plane,
                                  TX_SIZE tx_size, TX_TYPE tx_type) {
   const struct macroblockd_plane *const pd = &xd->plane[plane];
   const MB_MODE_INFO *const mbmi = xd->mi[0];
@@ -261,8 +261,8 @@
              : quant_params->giqmatrix[NUM_QM_LEVELS - 1][0][qm_tx_size];
 }
 
-const qm_val_t *av1_get_qmatrix(const CommonQuantParams *const quant_params,
-                                const MACROBLOCKD *const xd, int plane,
+const qm_val_t *av1_get_qmatrix(const CommonQuantParams *quant_params,
+                                const MACROBLOCKD *xd, int plane,
                                 TX_SIZE tx_size, TX_TYPE tx_type) {
   const struct macroblockd_plane *const pd = &xd->plane[plane];
   const MB_MODE_INFO *const mbmi = xd->mi[0];

av1/encoder/encodemb.c

diff --git a/av1/encoder/encodemb.c b/av1/encoder/encodemb.c
index e3915bf..906f9de 100644
--- a/av1/encoder/encodemb.c
+++ b/av1/encoder/encodemb.c
@@ -339,7 +339,7 @@
   qparam->iqmatrix = NULL;
 }
 void av1_setup_qmatrix(const CommonQuantParams *quant_params,
-                       const MACROBLOCKD *const xd, int plane, TX_SIZE tx_size,
+                       const MACROBLOCKD *xd, int plane, TX_SIZE tx_size,
                        TX_TYPE tx_type, QUANT_PARAM *qparam) {
   qparam->qmatrix = av1_get_qmatrix(quant_params, xd, plane, tx_size, tx_type);
   qparam->iqmatrix =

av1/encoder/mcomp.c

diff --git a/av1/encoder/mcomp.c b/av1/encoder/mcomp.c
index d3ba114..d2ca1cf 100644
--- a/av1/encoder/mcomp.c
+++ b/av1/encoder/mcomp.c
@@ -2799,7 +2799,7 @@
 
 int av1_find_best_sub_pixel_tree(MACROBLOCKD *xd, const AV1_COMMON *const cm,
                                  const SUBPEL_MOTION_SEARCH_PARAMS *ms_params,
-                                 const MV start_mv, MV *bestmv, int *distortion,
+                                 MV start_mv, MV *bestmv, int *distortion,
                                  unsigned int *sse1,
                                  int_mv *last_mv_search_list) {
   const int allow_hp = ms_params->allow_hp;
@@ -2870,7 +2870,7 @@
 // Returns the maximum MV.
 int av1_return_max_sub_pixel_mv(MACROBLOCKD *xd, const AV1_COMMON *const cm,
                                 const SUBPEL_MOTION_SEARCH_PARAMS *ms_params,
-                                const MV start_mv, MV *bestmv, int *distortion,
+                                MV start_mv, MV *bestmv, int *distortion,
                                 unsigned int *sse1,
                                 int_mv *last_mv_search_list) {
   (void)xd;
@@ -2897,7 +2897,7 @@
 // Returns the minimum MV.
 int av1_return_min_sub_pixel_mv(MACROBLOCKD *xd, const AV1_COMMON *const cm,
                                 const SUBPEL_MOTION_SEARCH_PARAMS *ms_params,
-                                const MV start_mv, MV *bestmv, int *distortion,
+                                MV start_mv, MV *bestmv, int *distortion,
                                 unsigned int *sse1,
                                 int_mv *last_mv_search_list) {
   (void)xd;
@@ -3299,7 +3299,7 @@
 
 int av1_find_best_obmc_sub_pixel_tree_up(
     MACROBLOCKD *xd, const AV1_COMMON *const cm,
-    const SUBPEL_MOTION_SEARCH_PARAMS *ms_params, const MV start_mv, MV *bestmv,
+    const SUBPEL_MOTION_SEARCH_PARAMS *ms_params, MV start_mv, MV *bestmv,
     int *distortion, unsigned int *sse1, int_mv *last_mv_search_list) {
   (void)last_mv_search_list;
   const int allow_hp = ms_params->allow_hp;

av1/encoder/mcomp.h

diff --git a/av1/encoder/mcomp.h b/av1/encoder/mcomp.h
index 5af577a..4887bca 100644
--- a/av1/encoder/mcomp.h
+++ b/av1/encoder/mcomp.h
@@ -282,8 +282,8 @@
 
 typedef int(fractional_mv_step_fp)(MACROBLOCKD *xd, const AV1_COMMON *const cm,
                                    const SUBPEL_MOTION_SEARCH_PARAMS *ms_params,
-                                   const MV start_mv, MV *bestmv,
-                                   int *distortion, unsigned int *sse1,
+                                   MV start_mv, MV *bestmv, int *distortion,
+                                   unsigned int *sse1,
                                    int_mv *last_mv_search_list);
 
 extern fractional_mv_step_fp av1_find_best_sub_pixel_tree;