Cr8-xyz/blender
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Cleanup: extra semicolon, comma warnings

Browse Source
This commit is contained in:
Campbell Barton
2019-12-10 12:00:49 +11:00
parent 3a4fa40313
commit dfb6af3011
2 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
tests/gtests/blenlib/BLI_listbase_test.cc
View File

@@ -16,8 +16,10 @@ extern "C" {
static bool listbase_is_valid(const ListBase *listbase) static bool listbase_is_valid(const ListBase *listbase)
{ {
#define TESTFAIL(test) \ #define TESTFAIL(test) \
if (!(test)) \ if (!(test)) { \
goto fail; goto fail; \
} \
((void)0)
if (listbase->first) { if (listbase->first) {
const Link *prev, *link; const Link *prev, *link;
@@ -31,14 +33,14 @@ static bool listbase_is_valid(const ListBase *listbase)
link = (Link *)listbase->first; link = (Link *)listbase->first;
do { do {
TESTFAIL(link->prev == prev); TESTFAIL(link->prev == prev);
} while ((prev = link), (link = link->next)); } while ((void)(prev = link), (link = link->next));
TESTFAIL(prev == listbase->last); TESTFAIL(prev == listbase->last);
prev = NULL; prev = NULL;
link = (Link *)listbase->last; link = (Link *)listbase->last;
do { do {
TESTFAIL(link->next == prev); TESTFAIL(link->next == prev);
} while ((prev = link), (link = link->prev)); } while ((void)(prev = link), (link = link->prev));
TESTFAIL(prev == listbase->first); TESTFAIL(prev == listbase->first);
} }
else { else {

3
tests/gtests/blenlib/BLI_path_util_test.cc
View File

@@ -474,7 +474,8 @@ TEST(path_util, SplitDirfile)
BLI_path_frame_strip(path, ext); \ BLI_path_frame_strip(path, ext); \
EXPECT_STREQ(path, expect_path); \ EXPECT_STREQ(path, expect_path); \
EXPECT_STREQ(ext, expect_ext); \ EXPECT_STREQ(ext, expect_ext); \
} } \
((void)0)
/* BLI_path_frame_strip */ /* BLI_path_frame_strip */
TEST(path_util, PathFrameStrip) TEST(path_util, PathFrameStrip)