diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-11-24 13:08:01 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-11-27 10:44:25 +0100 |
| commit | 2e5508a17660401e1b4b489dbc9f70b978745b75 (patch) | |
| tree | 881afcf9d8b9f0a653df0c94a8a2deefdf85670e /sfx2 | |
| parent | 470682b3abf0622f5e9663d62d8641f63ceb6f30 (diff) | |
loplugin:unnecessaryparen check for (f1()).f2
Change-Id: I93257b0ddd41c649875124d6d5c5faeaa431bae3
Reviewed-on: https://gerrit.libreoffice.org/45218
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sfx2')
| -rw-r--r-- | sfx2/source/control/templatelocalview.cxx | 4 | ||||
| -rw-r--r-- | sfx2/source/doc/objmisc.cxx | 4 | ||||
| -rw-r--r-- | sfx2/source/view/viewprn.cxx | 2 |
3 files changed, 5 insertions, 5 deletions
diff --git a/sfx2/source/control/templatelocalview.cxx b/sfx2/source/control/templatelocalview.cxx index a58e86d875a2..958429f6e005 100644 --- a/sfx2/source/control/templatelocalview.cxx +++ b/sfx2/source/control/templatelocalview.cxx @@ -706,8 +706,8 @@ bool TemplateLocalView::copyFrom (TemplateContainerItem *pItem, const OUString & if (!pItem->maTemplates.empty()) { - nId = (pItem->maTemplates.back()).nId+1; - nDocId = (pItem->maTemplates.back()).nDocId+1; + nId = pItem->maTemplates.back().nId+1; + nDocId = pItem->maTemplates.back().nDocId+1; } if (mpDocTemplates->CopyFrom(nRegionId,nDocId,aPath)) diff --git a/sfx2/source/doc/objmisc.cxx b/sfx2/source/doc/objmisc.cxx index a5e62da8c0a2..ff504a1d448b 100644 --- a/sfx2/source/doc/objmisc.cxx +++ b/sfx2/source/doc/objmisc.cxx @@ -513,7 +513,7 @@ bool SfxObjectShell::SwitchToShared( bool bShared, bool bSave ) { const OUString aTempFileURL = pMedium->GetURLObject().GetMainURL( INetURLObject::DecodeMechanism::NONE ); GetMedium()->SwitchDocumentToFile( GetSharedFileURL() ); - (pImpl->m_aSharedFileURL).clear(); + pImpl->m_aSharedFileURL.clear(); // now remove the temporary file the document was based on ::utl::UCBContentHelper::Kill( aTempFileURL ); @@ -581,7 +581,7 @@ void SfxObjectShell::FreeSharedFile( const OUString& aTempFileURL ) // now remove the temporary file the document is based currently on ::utl::UCBContentHelper::Kill( aTempFileURL ); - (pImpl->m_aSharedFileURL).clear(); + pImpl->m_aSharedFileURL.clear(); } } diff --git a/sfx2/source/view/viewprn.cxx b/sfx2/source/view/viewprn.cxx index 2456829d52bb..39c5dd2b75aa 100644 --- a/sfx2/source/view/viewprn.cxx +++ b/sfx2/source/view/viewprn.cxx @@ -617,7 +617,7 @@ const std::shared_ptr< vcl::PrinterController >& SfxViewShell::GetPrinterControl Printer* SfxViewShell::GetActivePrinter() const { - return (pImpl->m_xPrinterController) + return pImpl->m_xPrinterController ? pImpl->m_xPrinterController->getPrinter().get() : nullptr; } |