mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Some cppcheck cleaning

Browse Source
This commit is contained in:
Julien Nabet
2011-01-12 12:54:06 +00:00
committed by Caolán McNamara
parent d7cf133547
commit c34bae9706
6 changed files with 5 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cppuhelper/qa/ifcontainer/cppu_ifcontainer.cxx
View File

@@ -129,7 +129,7 @@ namespace cppu_ifcontainer
for (i = 0; i < nTests; i++) for (i = 0; i < nTests; i++)
{ {
Reference<XEventListener> xRef = new ContainerListener(&aStats); Reference<XEventListener> xRef = new ContainerListener(&aStats);
int nNewLen = pContainer->addInterface(xRef); pContainer->addInterface(xRef);
aListeners.push_back(xRef); aListeners.push_back(xRef);
} }
Sequence< Reference< XInterface > > aElements; Sequence< Reference< XInterface > > aElements;

1
cppuhelper/qa/unourl/cppu_unourl.cxx
View File

@@ -126,7 +126,6 @@ namespace cppu_unourl
{ "abc,def=%22", "abc,def=%22" }, { "abc,def=%22", "abc,def=%22" },
{ "abc,def=\"", "abc,def=\"" }, { "abc,def=\"", "abc,def=\"" },
{ "abc,def=%ed%a0%80", "abc,def=%ed%a0%80" } }; { "abc,def=%ed%a0%80", "abc,def=%ed%a0%80" } };
bool bResult = true;
for (int i = 0; i < sizeof aTests / sizeof (Test); ++i) for (int i = 0; i < sizeof aTests / sizeof (Test); ++i)
{ {
bool bValid = false; bool bValid = false;

2
cppuhelper/source/bootstrap.cxx
View File

@@ -273,7 +273,7 @@ OUString findBoostrapArgument(
} }
Reference< registry::XSimpleRegistry > nestRegistries( Reference< registry::XSimpleRegistry > nestRegistries(
const OUString baseDir, const OUString &baseDir,
const Reference< lang::XSingleServiceFactory > & xSimRegFac, const Reference< lang::XSingleServiceFactory > & xSimRegFac,
const Reference< lang::XSingleServiceFactory > & xNesRegFac, const Reference< lang::XSingleServiceFactory > & xNesRegFac,
OUString csl_rdbs, OUString csl_rdbs,

3
cppuhelper/source/propshlp.cxx
View File

@@ -909,11 +909,11 @@ void OPropertySetHelper::firePropertiesChangeEvent(
Sequence<PropertyChangeEvent> aChanges( nFireLen ); Sequence<PropertyChangeEvent> aChanges( nFireLen );
PropertyChangeEvent* pChanges = aChanges.getArray(); PropertyChangeEvent* pChanges = aChanges.getArray();
sal_Int32 nFirePos = 0;
{ {
// must lock the mutex outside the loop. So all values are consistent. // must lock the mutex outside the loop. So all values are consistent.
MutexGuard aGuard( rBHelper.rMutex ); MutexGuard aGuard( rBHelper.rMutex );
Reference < XInterface > xSource( (XPropertySet *)this, UNO_QUERY ); Reference < XInterface > xSource( (XPropertySet *)this, UNO_QUERY );
sal_Int32 nFirePos = 0;
for( i = 0; i < nLen; i++ ) for( i = 0; i < nLen; i++ )
{ {
if( pHandles[i] != -1 ) if( pHandles[i] != -1 )
@@ -946,7 +946,6 @@ PropertyState OPropertySetHelper::getPropertyState( const OUString& PropertyName
Sequence< PropertyState > OPropertySetHelper::getPropertyStates( const Sequence< OUString >& PropertyNames ) Sequence< PropertyState > OPropertySetHelper::getPropertyStates( const Sequence< OUString >& PropertyNames )
{ {
ULONG nNames = PropertyNames.getLength(); ULONG nNames = PropertyNames.getLength();
const OUString* pNames = PropertyNames.getConstArray();
Sequence< PropertyState > aStates( nNames ); Sequence< PropertyState > aStates( nNames );
return aStates; return aStates;

3
cppuhelper/source/tdmgr.cxx
View File

@@ -502,8 +502,7 @@ inline static typelib_TypeDescription * createCTD(
0, 0 ); 0, 0 );
// cleanup // cleanup
if (pRefType) typelib_typedescription_release( pRefType );
typelib_typedescription_release( pRefType );
} }
return pRet; return pRet;
} }

2
cppuhelper/test/testpropshlp.cxx
View File

@@ -461,7 +461,7 @@ public:
{ {
} }
sal_Int32 getRefCount() sal_Int32 getRefCount() const
{ return m_refCount; } { return m_refCount; }
// XEventListener // XEventListener