[FFmpeg-cvslog] tiff.c: Use switch / case instead of if / else where appropriate.

Carl Eugen Hoyos git at videolan.org
Sat Dec 31 02:03:52 CET 2011


ffmpeg | branch: master | Carl Eugen Hoyos <cehoyos at ag.or.at> | Sat Dec 31 01:53:22 2011 +0100| [b46fb615c441801f1d6949fac002d9f2ea6a8e15] | committer: Carl Eugen Hoyos

tiff.c: Use switch / case instead of if / else where appropriate.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=b46fb615c441801f1d6949fac002d9f2ea6a8e15
---

 libavcodec/tiff.c |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c
index 1060a44..ba838df 100644
--- a/libavcodec/tiff.c
+++ b/libavcodec/tiff.c
@@ -108,19 +108,22 @@ static void av_always_inline horizontal_fill(unsigned int bpp, uint8_t* dst,
 {
     int i;
 
-    if (bpp == 2) {
+    switch (bpp) {
+    case 2:
         for (i = 0; i < width; i++) {
             dst[(i+offset)*4+0] = (usePtr ? src[i] : c) >> 6;
             dst[(i+offset)*4+1] = (usePtr ? src[i] : c) >> 4 & 0x3;
             dst[(i+offset)*4+2] = (usePtr ? src[i] : c) >> 2 & 0x3;
             dst[(i+offset)*4+3] = (usePtr ? src[i] : c) & 0x3;
         }
-    } else if (bpp == 4) {
+        break;
+    case 4:
         for (i = 0; i < width; i++) {
             dst[(i+offset)*2+0] = (usePtr ? src[i] : c) >> 4;
             dst[(i+offset)*2+1] = (usePtr ? src[i] : c) & 0xF;
         }
-    } else {
+        break;
+    default:
         if (usePtr) {
             memcpy(dst + offset, src, width);
         } else {



More information about the ffmpeg-cvslog mailing list