From 7db1d6556d2aefcf4e5e787477b7cd22104e15ac Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Sun, 21 Jul 2013 08:16:37 +0000 Subject: code cleanup: add break statements in switch ()'s, (even at the last case). --- source/blender/imbuf/intern/cineon/logImageCore.c | 1 + source/blender/imbuf/intern/colormanagement.c | 1 + source/blender/imbuf/intern/jpeg.c | 7 +++++-- source/blender/imbuf/intern/png.c | 1 + 4 files changed, 8 insertions(+), 2 deletions(-) (limited to 'source/blender/imbuf') diff --git a/source/blender/imbuf/intern/cineon/logImageCore.c b/source/blender/imbuf/intern/cineon/logImageCore.c index 822f495d5bc..1500f4282e5 100644 --- a/source/blender/imbuf/intern/cineon/logImageCore.c +++ b/source/blender/imbuf/intern/cineon/logImageCore.c @@ -1470,6 +1470,7 @@ static int convertRGBAToLogElement(float *src, float *dst, LogImageFile *logImag case descriptor_YA: /* this descriptor is for internal use only */ default: rvalue = 1; + break; } if (srcIsLinearRGB != 0) { diff --git a/source/blender/imbuf/intern/colormanagement.c b/source/blender/imbuf/intern/colormanagement.c index baaee8cb553..238e68b141c 100644 --- a/source/blender/imbuf/intern/colormanagement.c +++ b/source/blender/imbuf/intern/colormanagement.c @@ -1094,6 +1094,7 @@ const char *IMB_colormanagement_role_colorspace_name_get(int role) default: printf("Unknown role was passed to %s\n", __func__); BLI_assert(0); + break; } return NULL; diff --git a/source/blender/imbuf/intern/jpeg.c b/source/blender/imbuf/intern/jpeg.c index ed4f6d40cee..38caf224c42 100644 --- a/source/blender/imbuf/intern/jpeg.c +++ b/source/blender/imbuf/intern/jpeg.c @@ -366,6 +366,7 @@ static ImBuf *ibJpegImageFromCinfo(struct jpeg_decompress_struct *cinfo, int fla rect[0] = r; rect += 4; } + break; } } @@ -549,7 +550,8 @@ next_stamp_info: break; /* default was missing... intentional ? */ default: - ; /* do nothing */ + /* do nothing */ + break; } jpeg_write_scanlines(cinfo, row_pointer, 1); @@ -594,7 +596,8 @@ static int init_jpeg(FILE *outfile, struct jpeg_compress_struct *cinfo, struct I break; /* default was missing... intentional ? */ default: - ; /* do nothing */ + /* do nothing */ + break; } jpeg_set_defaults(cinfo); diff --git a/source/blender/imbuf/intern/png.c b/source/blender/imbuf/intern/png.c index e07eef1868e..b620dbd9b10 100644 --- a/source/blender/imbuf/intern/png.c +++ b/source/blender/imbuf/intern/png.c @@ -567,6 +567,7 @@ ImBuf *imb_loadpng(unsigned char *mem, size_t size, int flags, char colorspace[I default: printf("PNG format not supported\n"); longjmp(png_jmpbuf(png_ptr), 1); + break; } ibuf = IMB_allocImBuf(width, height, 8 * bytesperpixel, 0); -- cgit v1.2.3