shithub: libvpx

Download patch

ref: fb220d257b3179b6c7e9c04c82003a1b6c1d7c29
parent: 05c6eca4db7f2abec32c9ce0fc325d9a0b933fb7
author: John Koleszar <jkoleszar@google.com>
date: Thu Jun 10 04:56:31 EDT 2010 

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


--- a/vp8/decoder/decodframe.c
+++ b/vp8/decoder/decodframe.c
@@ -317,20 +317,19 @@
 
     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. */