From 14b2a35c8bfd6d77f6357f929ddd82845436725f Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Sat, 19 Sep 2020 16:01:32 +1000 Subject: Cleanup: use parenthesis for if statements in macros --- source/blender/render/intern/include/texture.h | 18 ++++++++++++------ source/blender/render/intern/source/render_result.c | 6 ++++-- 2 files changed, 16 insertions(+), 8 deletions(-) (limited to 'source/blender/render') diff --git a/source/blender/render/intern/include/texture.h b/source/blender/render/intern/include/texture.h index 22023baab95..44ad508fe33 100644 --- a/source/blender/render/intern/include/texture.h +++ b/source/blender/render/intern/include/texture.h @@ -44,12 +44,15 @@ extern "C" { texres->tg = tex->gfac * ((texres->tg - 0.5f) * tex->contrast + tex->bright - 0.5f); \ texres->tb = tex->bfac * ((texres->tb - 0.5f) * tex->contrast + tex->bright - 0.5f); \ if (!(tex->flag & TEX_NO_CLAMP)) { \ - if (texres->tr < 0.0f) \ + if (texres->tr < 0.0f) { \ texres->tr = 0.0f; \ - if (texres->tg < 0.0f) \ + } \ + if (texres->tg < 0.0f) { \ texres->tg = 0.0f; \ - if (texres->tb < 0.0f) \ + } \ + if (texres->tb < 0.0f) { \ texres->tb = 0.0f; \ + } \ } \ if (tex->saturation != 1.0f) { \ float _hsv[3]; \ @@ -57,12 +60,15 @@ extern "C" { _hsv[1] *= tex->saturation; \ hsv_to_rgb(_hsv[0], _hsv[1], _hsv[2], &texres->tr, &texres->tg, &texres->tb); \ if ((tex->saturation > 1.0f) && !(tex->flag & TEX_NO_CLAMP)) { \ - if (texres->tr < 0.0f) \ + if (texres->tr < 0.0f) { \ texres->tr = 0.0f; \ - if (texres->tg < 0.0f) \ + } \ + if (texres->tg < 0.0f) { \ texres->tg = 0.0f; \ - if (texres->tb < 0.0f) \ + } \ + if (texres->tb < 0.0f) { \ texres->tb = 0.0f; \ + } \ } \ } \ ((void)0) diff --git a/source/blender/render/intern/source/render_result.c b/source/blender/render/intern/source/render_result.c index ef88584974a..d2f21ab035c 100644 --- a/source/blender/render/intern/source/render_result.c +++ b/source/blender/render/intern/source/render_result.c @@ -579,8 +579,10 @@ static int passtype_from_name(const char *name) int len = BLI_str_partition(name, delim, &sep, &suf); #define CHECK_PASS(NAME) \ - if (STREQLEN(name, RE_PASSNAME_##NAME, len)) \ - return SCE_PASS_##NAME + if (STREQLEN(name, RE_PASSNAME_##NAME, len)) { \ + return SCE_PASS_##NAME; \ + } \ + ((void)0) CHECK_PASS(COMBINED); CHECK_PASS(Z); -- cgit v1.2.3