summaryrefslogtreecommitdiffstats
path: root/encoder
diff options
context:
space:
mode:
authorMartin Storsjo <martin@martin.st>2015-06-27 00:19:48 +0300
committerMarco Nelissen <marcone@google.com>2015-07-31 18:47:37 +0000
commit4b09a8c011279fd5bb29d238ebef17810e446783 (patch)
tree8c4b6d6f60f0d35e689e1e26d9879039077d1f4a /encoder
parent698de0ed5cb2aa10ed62becb210ec1491642abcf (diff)
downloadandroid_external_libavc-4b09a8c011279fd5bb29d238ebef17810e446783.tar.gz
android_external_libavc-4b09a8c011279fd5bb29d238ebef17810e446783.tar.bz2
android_external_libavc-4b09a8c011279fd5bb29d238ebef17810e446783.zip
Don't declare variables after statements
This fixes building with compilers that are strict about the pre-C99 rule about having all variable declarations before statements. (The previous version of this commit was originally made before the commit adding support for main profile, which added a bunch more similar issues.) Bug: 22860270 Change-Id: I0a927deb2ef4125f79429d108439e249fa531607
Diffstat (limited to 'encoder')
-rw-r--r--encoder/ih264e_cabac_encode.c4
-rw-r--r--encoder/ih264e_encode.c20
-rw-r--r--encoder/ih264e_me.c18
3 files changed, 22 insertions, 20 deletions
diff --git a/encoder/ih264e_cabac_encode.c b/encoder/ih264e_cabac_encode.c
index ebcd418..bf3e88e 100644
--- a/encoder/ih264e_cabac_encode.c
+++ b/encoder/ih264e_cabac_encode.c
@@ -1206,11 +1206,13 @@ static void ih264e_cabac_encode_residue(entropy_ctxt_t *ps_ent_ctxt,
}
else
{
+ UWORD8 u1_csbf;
+
PARSE_COEFF_DATA_BLOCK_4x4(pv_mb_coeff_data, ps_mb_coeff_data,
u1_nnz, u2_sig_coeff_map,
pi2_res_block);
- UWORD8 u1_csbf = !!(u1_nnz);
+ u1_csbf = !!(u1_nnz);
{
UWORD8 u1_a, u1_b;
UWORD32 u4_ctx_inc;
diff --git a/encoder/ih264e_encode.c b/encoder/ih264e_encode.c
index c027321..d4d1090 100644
--- a/encoder/ih264e_encode.c
+++ b/encoder/ih264e_encode.c
@@ -393,16 +393,6 @@ WORD32 ih264e_encode(iv_obj_t *ps_codec_obj, void *pv_api_ip, void *pv_api_op)
/* Only encode if the current frame is not pre-encode skip */
if (!i4_rc_pre_enc_skip && s_inp_buf.s_raw_buf.apv_bufs[0])
{
- /* array giving pic cnt that is being processed in curr context set */
- ps_codec->ai4_pic_cnt[ctxt_sel] = ps_codec->i4_pic_cnt;
-
- /* initialize all relevant process ctxts */
- error_status |= ih264e_pic_init(ps_codec, &s_inp_buf);
- SET_ERROR_ON_RETURN(error_status,
- IVE_FATALERROR,
- ps_video_encode_op->s_ive_op.u4_error_code,
- IV_FAIL);
-
/* proc ctxt base idx */
WORD32 proc_ctxt_select = ctxt_sel * MAX_PROCESS_THREADS;
@@ -414,6 +404,16 @@ WORD32 ih264e_encode(iv_obj_t *ps_codec_obj, void *pv_api_ip, void *pv_api_op)
/* number of addl. threads to be created */
WORD32 num_thread_cnt = ps_codec->s_cfg.u4_num_cores - 1;
+ /* array giving pic cnt that is being processed in curr context set */
+ ps_codec->ai4_pic_cnt[ctxt_sel] = ps_codec->i4_pic_cnt;
+
+ /* initialize all relevant process ctxts */
+ error_status |= ih264e_pic_init(ps_codec, &s_inp_buf);
+ SET_ERROR_ON_RETURN(error_status,
+ IVE_FATALERROR,
+ ps_video_encode_op->s_ive_op.u4_error_code,
+ IV_FAIL);
+
for (i = 0; i < num_thread_cnt; i++)
{
ret = ithread_create(ps_codec->apv_proc_thread_handle[i],
diff --git a/encoder/ih264e_me.c b/encoder/ih264e_me.c
index 12a6f55..a97df3d 100644
--- a/encoder/ih264e_me.c
+++ b/encoder/ih264e_me.c
@@ -241,6 +241,9 @@ static void ih264e_get_search_candidates(process_ctxt_t *ps_proc,
WORD32 i4_srch_range_e = ps_me_ctxt->i4_srch_range_e;
WORD32 i4_srch_range_w = ps_me_ctxt->i4_srch_range_w;
+ /* num of candidate search candidates */
+ UWORD32 u4_num_candidates = 0;
+
ps_left_mv = &ps_proc->s_left_mb_pu_ME.s_me_info[i4_reflist].s_mv;
ps_top_mv = &(ps_proc->ps_top_row_pu_ME + i4_mb_x)->s_me_info[i4_reflist].s_mv;
ps_top_left_mv = &ps_proc->s_top_left_mb_pu_ME.s_me_info[i4_reflist].s_mv;
@@ -251,9 +254,6 @@ static void ih264e_get_search_candidates(process_ctxt_t *ps_proc,
i4_top_left_mode = ps_proc->s_top_left_mb_pu_ME.b2_pred_mode != i4_cmpl_predmode;
i4_top_right_mode = (ps_proc->ps_top_row_pu_ME + i4_mb_x + 1)->b2_pred_mode != i4_cmpl_predmode;
- /* num of candidate search candidates */
- UWORD32 u4_num_candidates =0 ;
-
/* Taking the Zero motion vector as one of the candidates */
ps_me_ctxt->as_mv_init_search[i4_reflist][u4_num_candidates].i2_mvx = 0;
ps_me_ctxt->as_mv_init_search[i4_reflist][u4_num_candidates].i2_mvy = 0;
@@ -893,12 +893,12 @@ void ih264e_compute_me_single_reflist(process_ctxt_t *ps_proc)
/* quantization parameters */
quant_params_t *ps_qp_params = ps_proc->ps_qp_params[0];
- /* Sad therholds */
- ps_me_ctxt->pu2_sad_thrsh = ps_qp_params->pu2_sad_thrsh;
-
/* Mb part ctxts for SKIP */
mb_part_ctxt s_skip_mbpart;
+ /* Sad therholds */
+ ps_me_ctxt->pu2_sad_thrsh = ps_qp_params->pu2_sad_thrsh;
+
{
WORD32 rows_above, rows_below, columns_left, columns_right;
@@ -1800,12 +1800,12 @@ void ih264e_compute_me_multi_reflist(process_ctxt_t *ps_proc)
/* quantization parameters */
quant_params_t *ps_qp_params = ps_proc->ps_qp_params[0];
- /* Sad therholds */
- ps_me_ctxt->pu2_sad_thrsh = ps_qp_params->pu2_sad_thrsh;
-
/* Mb part ctxts for SKIP */
mb_part_ctxt as_skip_mbpart[2];
+ /* Sad therholds */
+ ps_me_ctxt->pu2_sad_thrsh = ps_qp_params->pu2_sad_thrsh;
+
{
WORD32 rows_above, rows_below, columns_left, columns_right;