mirror of https://git.ffmpeg.org/ffmpeg.git
avcodec/error_resilience: make error an local variable where possible
This makes the code easier to understand as the scope of the variable is smaller. Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
This commit is contained in:
parent
459996325d
commit
c103c48525
|
@ -1057,7 +1057,7 @@ void ff_er_frame_end(ERContext *s)
|
||||||
if (!s->partitioned_frame) {
|
if (!s->partitioned_frame) {
|
||||||
for (i = 0; i < s->mb_num; i++) {
|
for (i = 0; i < s->mb_num; i++) {
|
||||||
const int mb_xy = s->mb_index2xy[i];
|
const int mb_xy = s->mb_index2xy[i];
|
||||||
error = s->error_status_table[mb_xy];
|
int error = s->error_status_table[mb_xy];
|
||||||
if (error & ER_MB_ERROR)
|
if (error & ER_MB_ERROR)
|
||||||
error |= ER_MB_ERROR;
|
error |= ER_MB_ERROR;
|
||||||
s->error_status_table[mb_xy] = error;
|
s->error_status_table[mb_xy] = error;
|
||||||
|
@ -1068,7 +1068,7 @@ void ff_er_frame_end(ERContext *s)
|
||||||
dc_error = ac_error = mv_error = 0;
|
dc_error = ac_error = mv_error = 0;
|
||||||
for (i = 0; i < s->mb_num; i++) {
|
for (i = 0; i < s->mb_num; i++) {
|
||||||
const int mb_xy = s->mb_index2xy[i];
|
const int mb_xy = s->mb_index2xy[i];
|
||||||
error = s->error_status_table[mb_xy];
|
int error = s->error_status_table[mb_xy];
|
||||||
if (error & ER_DC_ERROR)
|
if (error & ER_DC_ERROR)
|
||||||
dc_error++;
|
dc_error++;
|
||||||
if (error & ER_AC_ERROR)
|
if (error & ER_AC_ERROR)
|
||||||
|
@ -1084,7 +1084,7 @@ void ff_er_frame_end(ERContext *s)
|
||||||
/* set unknown mb-type to most likely */
|
/* set unknown mb-type to most likely */
|
||||||
for (i = 0; i < s->mb_num; i++) {
|
for (i = 0; i < s->mb_num; i++) {
|
||||||
const int mb_xy = s->mb_index2xy[i];
|
const int mb_xy = s->mb_index2xy[i];
|
||||||
error = s->error_status_table[mb_xy];
|
int error = s->error_status_table[mb_xy];
|
||||||
if (!((error & ER_DC_ERROR) && (error & ER_MV_ERROR)))
|
if (!((error & ER_DC_ERROR) && (error & ER_MV_ERROR)))
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
|
@ -1112,7 +1112,7 @@ void ff_er_frame_end(ERContext *s)
|
||||||
const int mv_dir = dir ? MV_DIR_BACKWARD : MV_DIR_FORWARD;
|
const int mv_dir = dir ? MV_DIR_BACKWARD : MV_DIR_FORWARD;
|
||||||
int mv_type;
|
int mv_type;
|
||||||
|
|
||||||
error = s->error_status_table[mb_xy];
|
int error = s->error_status_table[mb_xy];
|
||||||
|
|
||||||
if (IS_INTRA(mb_type))
|
if (IS_INTRA(mb_type))
|
||||||
continue; // intra
|
continue; // intra
|
||||||
|
@ -1149,7 +1149,7 @@ void ff_er_frame_end(ERContext *s)
|
||||||
const int mb_type = s->cur_pic.mb_type[mb_xy];
|
const int mb_type = s->cur_pic.mb_type[mb_xy];
|
||||||
int mv_dir = MV_DIR_FORWARD | MV_DIR_BACKWARD;
|
int mv_dir = MV_DIR_FORWARD | MV_DIR_BACKWARD;
|
||||||
|
|
||||||
error = s->error_status_table[mb_xy];
|
int error = s->error_status_table[mb_xy];
|
||||||
|
|
||||||
if (IS_INTRA(mb_type))
|
if (IS_INTRA(mb_type))
|
||||||
continue;
|
continue;
|
||||||
|
@ -1253,7 +1253,7 @@ void ff_er_frame_end(ERContext *s)
|
||||||
const int mb_xy = mb_x + mb_y * s->mb_stride;
|
const int mb_xy = mb_x + mb_y * s->mb_stride;
|
||||||
const int mb_type = s->cur_pic.mb_type[mb_xy];
|
const int mb_type = s->cur_pic.mb_type[mb_xy];
|
||||||
|
|
||||||
error = s->error_status_table[mb_xy];
|
int error = s->error_status_table[mb_xy];
|
||||||
|
|
||||||
if (IS_INTER(mb_type))
|
if (IS_INTER(mb_type))
|
||||||
continue;
|
continue;
|
||||||
|
|
Loading…
Reference in New Issue