Skip to content

Commit

Permalink
Merge pull request ninja-build#2414 from heitbaum/warning
Browse files Browse the repository at this point in the history
fix: add explict braces to avoid ambiguous else
  • Loading branch information
jhasse authored Apr 14, 2024
2 parents da11ea2 + 8ea5fa8 commit 4a66995
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/build_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1363,8 +1363,9 @@ void TestPhonyUseCase(BuildTest* t, int i) {
// Build number 1
EXPECT_TRUE(builder_.AddTarget("test" + ci, &err));
ASSERT_EQ("", err);
if (!builder_.AlreadyUpToDate())
if (!builder_.AlreadyUpToDate()) {
EXPECT_TRUE(builder_.Build(&err));
}
ASSERT_EQ("", err);

// Touch the input file
Expand Down

0 comments on commit 4a66995

Please sign in to comment.