Eliminate some single-line if statements.

pjohnmeyer · pjohnmeyer · commit 3f5095ba13dd · 2016-02-06T09:30:15.000-06:00
diff --git a/UnitTest++/ThrowingTestReporter.cpp b/UnitTest++/ThrowingTestReporter.cpp
@@ -14,26 +14,38 @@ namespace UnitTest {
    //virtual
    void ThrowingTestReporter::ReportTestStart(TestDetails const& test)
    {
-      if(m_decoratedReporter) m_decoratedReporter->ReportTestStart(test);
+      if(m_decoratedReporter)
+      {
+         m_decoratedReporter->ReportTestStart(test);
+      }
    }
 
    //virtual
    void ThrowingTestReporter::ReportFailure(TestDetails const& test, char const* failure)
    {
-      if(m_decoratedReporter) m_decoratedReporter->ReportFailure(test, failure);
+      if(m_decoratedReporter)
+      {
+         m_decoratedReporter->ReportFailure(test, failure);
+      }
       throw RequiredCheckException();
    }
 
    //virtual
    void ThrowingTestReporter::ReportTestFinish(TestDetails const& test, float secondsElapsed)
    {
-      if(m_decoratedReporter) m_decoratedReporter->ReportTestFinish(test, secondsElapsed);
+      if(m_decoratedReporter)
+      {
+         m_decoratedReporter->ReportTestFinish(test, secondsElapsed);
+      }
    }
 
    //virtual
    void ThrowingTestReporter::ReportSummary(int totalTestCount, int failedTestCount, int failureCount, float secondsElapsed)
    {
-      if(m_decoratedReporter) m_decoratedReporter->ReportSummary(totalTestCount, failedTestCount, failureCount, secondsElapsed);
+      if(m_decoratedReporter)
+      {
+         m_decoratedReporter->ReportSummary(totalTestCount, failedTestCount, failureCount, secondsElapsed);
+      }
    }
 
 }

Original file line number	Diff line number	Diff line change
`@@ -14,26 +14,38 @@ namespace UnitTest {`
`14`	`14`	`//virtual`
`15`	`15`	`void ThrowingTestReporter::ReportTestStart(TestDetails const& test)`
`16`	`16`	`{`
`17`		`- if(m_decoratedReporter) m_decoratedReporter->ReportTestStart(test);`
	`17`	`+ if(m_decoratedReporter)`
	`18`	`+ {`
	`19`	`+ m_decoratedReporter->ReportTestStart(test);`
	`20`	`+ }`
`18`	`21`	`}`
`19`	`22`
`20`	`23`	`//virtual`
`21`	`24`	`void ThrowingTestReporter::ReportFailure(TestDetails const& test, char const* failure)`
`22`	`25`	`{`
`23`		`- if(m_decoratedReporter) m_decoratedReporter->ReportFailure(test, failure);`
	`26`	`+ if(m_decoratedReporter)`
	`27`	`+ {`
	`28`	`+ m_decoratedReporter->ReportFailure(test, failure);`
	`29`	`+ }`
`24`	`30`	`throw RequiredCheckException();`
`25`	`31`	`}`
`26`	`32`
`27`	`33`	`//virtual`
`28`	`34`	`void ThrowingTestReporter::ReportTestFinish(TestDetails const& test, float secondsElapsed)`
`29`	`35`	`{`
`30`		`- if(m_decoratedReporter) m_decoratedReporter->ReportTestFinish(test, secondsElapsed);`
	`36`	`+ if(m_decoratedReporter)`
	`37`	`+ {`
	`38`	`+ m_decoratedReporter->ReportTestFinish(test, secondsElapsed);`
	`39`	`+ }`
`31`	`40`	`}`
`32`	`41`
`33`	`42`	`//virtual`
`34`	`43`	`void ThrowingTestReporter::ReportSummary(int totalTestCount, int failedTestCount, int failureCount, float secondsElapsed)`
`35`	`44`	`{`
`36`		`- if(m_decoratedReporter) m_decoratedReporter->ReportSummary(totalTestCount, failedTestCount, failureCount, secondsElapsed);`
	`45`	`+ if(m_decoratedReporter)`
	`46`	`+ {`
	`47`	`+ m_decoratedReporter->ReportSummary(totalTestCount, failedTestCount, failureCount, secondsElapsed);`
	`48`	`+ }`
`37`	`49`	`}`
`38`	`50`
`39`	`51`	`}`