Best-case for inlining reportAssertionStarting

horenmar · horenmar · commit 96574cfa3521 · 2026-04-07T10:14:09.000+02:00
diff --git a/src/catch2/internal/catch_assertion_handler.cpp b/src/catch2/internal/catch_assertion_handler.cpp
@@ -27,7 +27,7 @@ namespace Catch {
     :   m_assertionInfo{ macroName, lineInfo, capturedExpression, resultDisposition },
         m_resultCapture( static_cast<RunContext&>(getResultCapture()) )
     {
-        m_resultCapture.notifyAssertionStarted( m_assertionInfo );
+        //m_resultCapture.notifyAssertionStarted( m_assertionInfo );
     }
 
     void AssertionHandler::handleExpr( ITransientExpression const& expr ) {

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ namespace Catch {`
`27`	`27`	`: m_assertionInfo{ macroName, lineInfo, capturedExpression, resultDisposition },`
`28`	`28`	`m_resultCapture( static_cast<RunContext&>(getResultCapture()) )`
`29`	`29`	`{`
`30`		`- m_resultCapture.notifyAssertionStarted( m_assertionInfo );`
	`30`	`+ //m_resultCapture.notifyAssertionStarted( m_assertionInfo );`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`void AssertionHandler::handleExpr( ITransientExpression const& expr ) {`