diff options
| author | Noel Grandin <noel@peralex.com> | 2016-02-17 13:14:50 +0200 |
|---|---|---|
| committer | Noel Grandin <noel@peralex.com> | 2016-02-23 08:08:56 +0200 |
| commit | c45d3badc96481db093560b94d8bf51ead6bd17c (patch) | |
| tree | 4bb6c9220678a12b327e46ca2acd01e77fc8e2c4 /sfx2/source | |
| parent | 003d0ccf902d2449320dd24119564565a384f365 (diff) | |
new loplugin: commaoperator
Change-Id: I03f24e61f696b7619855e3c7010aa0d874e5a4ff
Diffstat (limited to 'sfx2/source')
| -rw-r--r-- | sfx2/source/appl/appquit.cxx | 3 | ||||
| -rw-r--r-- | sfx2/source/appl/module.cxx | 3 | ||||
| -rw-r--r-- | sfx2/source/bastyp/mieclip.cxx | 5 | ||||
| -rw-r--r-- | sfx2/source/dialog/filedlghelper.cxx | 2 | ||||
| -rw-r--r-- | sfx2/source/doc/sfxbasemodel.cxx | 3 |
5 files changed, 11 insertions, 5 deletions
diff --git a/sfx2/source/appl/appquit.cxx b/sfx2/source/appl/appquit.cxx index cc891da5fd8d..a97ae379116e 100644 --- a/sfx2/source/appl/appquit.cxx +++ b/sfx2/source/appl/appquit.cxx @@ -101,7 +101,8 @@ void SfxApplication::Deinitialize() DBG_ASSERT( pAppData_Impl->pViewFrame == nullptr, "active foreign ViewFrame" ); - delete[] pAppData_Impl->pInterfaces, pAppData_Impl->pInterfaces = nullptr; + delete[] pAppData_Impl->pInterfaces; + pAppData_Impl->pInterfaces = nullptr; // free administration managers DELETEZ(pAppData_Impl->pAppDispat); diff --git a/sfx2/source/appl/module.cxx b/sfx2/source/appl/module.cxx index 6ddcf09c77c7..bf7df421db97 100644 --- a/sfx2/source/appl/module.cxx +++ b/sfx2/source/appl/module.cxx @@ -341,7 +341,8 @@ void SfxModule::DestroyModules_Impl() SfxModule* pMod = rModules[nPos]; delete pMod; } - delete pModules, pModules = nullptr; + delete pModules; + pModules = nullptr; } } diff --git a/sfx2/source/bastyp/mieclip.cxx b/sfx2/source/bastyp/mieclip.cxx index cbbb348ba2bc..00d1e52f1bae 100644 --- a/sfx2/source/bastyp/mieclip.cxx +++ b/sfx2/source/bastyp/mieclip.cxx @@ -34,7 +34,10 @@ SvStream* MSE40HTMLClipFormatObj::IsValid( SvStream& rStream ) { bool bRet = false; if( pStrm ) - delete pStrm, pStrm = nullptr; + { + delete pStrm; + pStrm = nullptr; + } OString sLine, sVersion; sal_Int32 nStt = -1, nEnd = -1, nFragStart = -1, nFragEnd = -1; diff --git a/sfx2/source/dialog/filedlghelper.cxx b/sfx2/source/dialog/filedlghelper.cxx index 3314c97dfd5d..ed5b0556c60b 100644 --- a/sfx2/source/dialog/filedlghelper.cxx +++ b/sfx2/source/dialog/filedlghelper.cxx @@ -1894,7 +1894,7 @@ static void SetToken( OUString& rOrigStr, sal_Int32 nToken, sal_Unicode cTok, co } } - ++pStr, + ++pStr; ++i; } diff --git a/sfx2/source/doc/sfxbasemodel.cxx b/sfx2/source/doc/sfxbasemodel.cxx index 1b342e9d340c..e510588ab1b5 100644 --- a/sfx2/source/doc/sfxbasemodel.cxx +++ b/sfx2/source/doc/sfxbasemodel.cxx @@ -521,7 +521,8 @@ SfxBaseModel::~SfxBaseModel() { //In SvxDrawingLayerImport when !xTargetDocument the fallback SvxUnoDrawingModel created there //never gets disposed called on it, so m_pData leaks. - delete m_pData, m_pData = nullptr; + delete m_pData; + m_pData = nullptr; } // XInterface |