3rd-party/catch2
1
0
Fork 0
mirror of https://github.com/catchorg/Catch2.git synced 2024-12-23 11:43:29 +01:00
Code Issues Packages Projects Releases Wiki Activity

Address more of PVS studio warnings

Browse Source
This commit is contained in:
Martin Hořeňovský 2017-07-25 15:45:50 +02:00
parent 46bf7605f4
commit b6f62af7d1
4 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
include/internal/catch_list.cpp
View File

@ -44,7 +44,7 @@ namespace Catch {
if( config.verbosity() >= Verbosity::High ) { if( config.verbosity() >= Verbosity::High ) {
Catch::cout() << Column( Catch::Detail::stringify( testCaseInfo.lineInfo ) ).indent(4) << std::endl; Catch::cout() << Column( Catch::Detail::stringify( testCaseInfo.lineInfo ) ).indent(4) << std::endl;
std::string description = testCaseInfo.description; std::string description = testCaseInfo.description;
if( description == "" ) if( description.empty() )
description = "(NO DESCRIPTION)"; description = "(NO DESCRIPTION)";
Catch::cout() << Column( description ).indent(4) << std::endl; Catch::cout() << Column( description ).indent(4) << std::endl;
} }
@ -129,10 +129,10 @@ namespace Catch {
Catch::cout() << "Available reporters:\n"; Catch::cout() << "Available reporters:\n";
IReporterRegistry::FactoryMap const& factories = getRegistryHub().getReporterRegistry().getFactories(); IReporterRegistry::FactoryMap const& factories = getRegistryHub().getReporterRegistry().getFactories();
std::size_t maxNameLen = 0; std::size_t maxNameLen = 0;
for( auto const& factoryKvp : getRegistryHub().getReporterRegistry().getFactories() ) for( auto const& factoryKvp : factories )
maxNameLen = (std::max)( maxNameLen, factoryKvp.first.size() ); maxNameLen = (std::max)( maxNameLen, factoryKvp.first.size() );
for( auto const& factoryKvp : getRegistryHub().getReporterRegistry().getFactories() ) { for( auto const& factoryKvp : factories ) {
Catch::cout() Catch::cout()
<< Column( factoryKvp.first + ":" ) << Column( factoryKvp.first + ":" )
.indent(2) .indent(2)

2
include/internal/catch_result_builder.cpp
View File

@ -91,7 +91,7 @@ namespace Catch {
std::string actualMessage = Catch::translateActiveException(); std::string actualMessage = Catch::translateActiveException();
if( !matcher.match( actualMessage ) ) { if( !matcher.match( actualMessage ) ) {
data.resultType = ResultWas::ExpressionFailed; data.resultType = ResultWas::ExpressionFailed;
data.reconstructedExpression = actualMessage; data.reconstructedExpression = std::move(actualMessage);
} }
AssertionResult result( m_assertionInfo, data ); AssertionResult result( m_assertionInfo, data );
handleResult( result ); handleResult( result );

4
include/internal/catch_stringdata.h
View File

@ -30,12 +30,12 @@ namespace Catch {
void addRef() const noexcept { void addRef() const noexcept {
if( m_refs > 0 ) if( m_refs > 0 )
m_refs++; ++m_refs;
} }
void release() const noexcept { void release() const noexcept {
unsigned int refs = m_refs; unsigned int refs = m_refs;
if( refs > 1 ) if( refs > 1 )
m_refs--; --m_refs;
else if( refs == 1 ) else if( refs == 1 )
delete[] reinterpret_cast<char const*>( this ); delete[] reinterpret_cast<char const*>( this );
} }

2
include/reporters/catch_reporter_bases.hpp
View File

@ -181,7 +181,7 @@ namespace Catch {
node = *it; node = *it;
} }
m_sectionStack.push_back( node ); m_sectionStack.push_back( node );
m_deepestSection = node; m_deepestSection = std::move(node);
} }
void assertionStarting(AssertionInfo const&) override {} void assertionStarting(AssertionInfo const&) override {}