diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2010-12-23 20:58:53 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2010-12-23 20:58:53 +0000 |
| commit | bbf9b1d7bcbbabb589e35e1fcf9b06259854538b (patch) | |
| tree | 8e092055c74c91b4b11ef03f9b6a0fc125e91c18 /extensions/test | |
| parent | d26209184ba9a685f4ee8e1313b5e54a92df89dd (diff) | |
cppcheck: prefer prefix variant
Diffstat (limited to 'extensions/test')
| -rw-r--r-- | extensions/test/sax/testwriter.cxx | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/extensions/test/sax/testwriter.cxx b/extensions/test/sax/testwriter.cxx index 692dcb1647ad..820f962de8b0 100644 --- a/extensions/test/sax/testwriter.cxx +++ b/extensions/test/sax/testwriter.cxx @@ -333,8 +333,10 @@ UString AttributeListImpl::getTypeByName( const UString& sName ) THROWS( (UsrSys { vector<struct TagAttribute>::iterator ii = m_pImpl->vecAttribute.begin(); - for( ; ii != m_pImpl->vecAttribute.end() ; ii ++ ) { - if( (*ii).sName == sName ) { + for (; ii != m_pImpl->vecAttribute.end(); ++ii) + { + if( (*ii).sName == sName ) + { return (*ii).sType; } } @@ -345,8 +347,10 @@ UString AttributeListImpl::getValueByName(const UString& sName) THROWS( (UsrSyst { vector<struct TagAttribute>::iterator ii = m_pImpl->vecAttribute.begin(); - for( ; ii != m_pImpl->vecAttribute.end() ; ii ++ ) { - if( (*ii).sName == sName ) { + for (; ii != m_pImpl->vecAttribute.end(); ++ii) + { + if( (*ii).sName == sName ) + { return (*ii).sValue; } } |