Remove superfluous semicolon in the INFO macro

Closes #1456
catchorg · Oct 29, 2019 · 557b336 · 557b336
1 parent 1417617
commit 557b336
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 9 deletions.
diff --git a/docs/release-notes.md b/docs/release-notes.md
@@ -33,8 +33,10 @@
 ## 3.0.0 (in progress)
 
 ### Breaking changes
-* `ANON_TEST_CASE` has been removed, use `TEST_CASE` with no arguments instead.
+* `ANON_TEST_CASE` has been removed, use `TEST_CASE` with no arguments instead (#1220)
 
+### Fixes
+* The `INFO` macro no longer contains superfluous semicolon (#1456)
 
 ## 2.10.2
 

diff --git a/include/internal/catch_capture.hpp b/include/internal/catch_capture.hpp
@@ -129,7 +129,7 @@
 
 ///////////////////////////////////////////////////////////////////////////////
 #define INTERNAL_CATCH_INFO( macroName, log ) \
-    Catch::ScopedMessage INTERNAL_CATCH_UNIQUE_NAME( scopedMessage )( Catch::MessageBuilder( macroName##_catch_sr, CATCH_INTERNAL_LINEINFO, Catch::ResultWas::Info ) << log );
+    Catch::ScopedMessage INTERNAL_CATCH_UNIQUE_NAME( scopedMessage )( Catch::MessageBuilder( macroName##_catch_sr, CATCH_INTERNAL_LINEINFO, Catch::ResultWas::Info ) << log )
 
 ///////////////////////////////////////////////////////////////////////////////
 #define INTERNAL_CATCH_UNSCOPED_INFO( macroName, log ) \

diff --git a/projects/SelfTest/IntrospectiveTests/GeneratorsImpl.tests.cpp b/projects/SelfTest/IntrospectiveTests/GeneratorsImpl.tests.cpp
@@ -181,9 +181,9 @@ TEST_CASE("Generators internals", "[generators][internals]") {
                     const auto step = .1;
 
                     auto gen = range(rangeStart, rangeEnd, step);
-                    auto expected = rangeStart; 
+                    auto expected = rangeStart;
                     while( (rangeEnd - expected) > step ) {
-                        INFO( "Current expected value is " << expected )
+                        INFO( "Current expected value is " << expected );
                         REQUIRE(gen.get() == Approx(expected));
                         REQUIRE(gen.next());
 
@@ -198,9 +198,9 @@ TEST_CASE("Generators internals", "[generators][internals]") {
                     const auto step = .3;
 
                     auto gen = range(rangeStart, rangeEnd, step);
-                    auto expected = rangeStart; 
+                    auto expected = rangeStart;
                     while( (rangeEnd - expected) > step ) {
-                       INFO( "Current expected value is " << expected )
+                       INFO( "Current expected value is " << expected );
                        REQUIRE(gen.get() == Approx(expected));
                        REQUIRE(gen.next());
 
@@ -214,16 +214,16 @@ TEST_CASE("Generators internals", "[generators][internals]") {
                     const auto step = .3;
 
                     auto gen = range(rangeStart, rangeEnd, step);
-                    auto expected = rangeStart; 
+                    auto expected = rangeStart;
                     while( (rangeEnd - expected) > step ) {
-                       INFO( "Current expected value is " << expected )
+                       INFO( "Current expected value is " << expected );
                        REQUIRE(gen.get() == Approx(expected));
                        REQUIRE(gen.next());
 
                        expected += step;
                     }
                     REQUIRE_FALSE(gen.next());
-                }                
+                }
             }
         }
         SECTION("Negative manual step") {