diff options
author | John Koleszar <jkoleszar@google.com> | 2010-06-10 08:56:31 -0400 |
---|---|---|
committer | John Koleszar <jkoleszar@google.com> | 2010-06-10 20:15:21 -0400 |
commit | fb220d257b3179b6c7e9c04c82003a1b6c1d7c29 (patch) | |
tree | a69552a39d8d6ef9b40b79ac077f885dfacddd86 /vp8/decoder/decodframe.c | |
parent | 05c6eca4db7f2abec32c9ce0fc325d9a0b933fb7 (diff) | |
download | libvpx-fb220d257b3179b6c7e9c04c82003a1b6c1d7c29.tar libvpx-fb220d257b3179b6c7e9c04c82003a1b6c1d7c29.tar.gz libvpx-fb220d257b3179b6c7e9c04c82003a1b6c1d7c29.tar.bz2 libvpx-fb220d257b3179b6c7e9c04c82003a1b6c1d7c29.zip |
replace while(0) construct with if/else
No good reason to be tricky here. I don't know why 'break' occurred to me
as the natrual replacement for the 'return', but an if/else block is
definitely clearer.
Change-Id: I08a336307afeb0dc7efa494b37398f239f66c2cf
Diffstat (limited to 'vp8/decoder/decodframe.c')
-rw-r--r-- | vp8/decoder/decodframe.c | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/vp8/decoder/decodframe.c b/vp8/decoder/decodframe.c index 04fb03fa4..72c312fc1 100644 --- a/vp8/decoder/decodframe.c +++ b/vp8/decoder/decodframe.c @@ -317,20 +317,19 @@ void vp8_decode_macroblock(VP8D_COMP *pbi, MACROBLOCKD *xd) xd->mode_info_context->mbmi.dc_diff = 1; - do { - if (xd->mbmi.mode != B_PRED && xd->mbmi.mode != SPLITMV && eobtotal == 0) - { - xd->mode_info_context->mbmi.dc_diff = 0; - skip_recon_mb(pbi, xd); - break; - } - + if (xd->mbmi.mode != B_PRED && xd->mbmi.mode != SPLITMV && eobtotal == 0) + { + xd->mode_info_context->mbmi.dc_diff = 0; + skip_recon_mb(pbi, xd); + } + else + { if (xd->segmentation_enabled) mb_init_dequantizer(pbi, xd); de_quantand_idct(pbi, xd); reconstruct_mb(pbi, xd); - } while(0); + } /* Restore the original MV so as not to affect the entropy context. */ |