loplugin:passstuffbyref improved return in emfio,writerfilter

Change-Id: I237936d62d0f1b17574dd88b5c9de932dc03238e
Reviewed-on: https://gerrit.libreoffice.org/47214Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

compilerplugins/clang/passstuffbyref.cxx

@@ -250,7 +250,8 @@ void PassStuffByRef::checkReturnValue(const FunctionDecl * functionDecl, const C
         || (dc.MemberFunction().Class("Submission").Namespace("xforms")
             .GlobalNamespace())
         || (dc.Function("TopLeft").Class("SwRect").GlobalNamespace())
-        || (dc.Function("ConvDicList_CreateInstance").GlobalNamespace()))
+        || (dc.Function("ConvDicList_CreateInstance").GlobalNamespace())
+        || (dc.Function("Create").Class("OUnoAutoPilot").Namespace("dbp").GlobalNamespace()))
     {
         return;
     }
@@ -349,6 +350,9 @@ bool PassStuffByRef::isReturnExprDisqualified(const Expr* expr)
         if (isa<CXXBindTemporaryExpr>(expr)) {
             return true;
         }
+        if (isa<InitListExpr>(expr)) {
+            return true;
+        }
         expr = expr->IgnoreParenCasts();
         if (auto childExpr = dyn_cast<ArraySubscriptExpr>(expr)) {
             expr = childExpr->getLHS();

desktop/source/app/cmdlineargs.hxx

@@ -115,7 +115,7 @@ class CommandLineArgs
         std::vector< OUString > GetConversionList() const;
         const OUString&         GetConversionParams() const { return m_conversionparams;}
         OUString                GetConversionOut() const;
-        OUString                GetImageConversionType() const { return m_convertimages; }
+        OUString const &        GetImageConversionType() const { return m_convertimages; }
         const OUString&         GetPidfileName() const { return m_pidfile;}
 
         // Special analyzed states (does not match directly to a command line parameter!)

emfio/inc/mtftools.hxx

@@ -280,7 +280,7 @@ namespace emfio
 
         bool        isEmpty() const { return maClip.isCleared(); }
 
-        basegfx::B2DPolyPolygon getClipPath() const;
+        basegfx::B2DPolyPolygon const & getClipPath() const;
 
         bool        operator==(const WinMtfClipPath& rPath) const
         {

emfio/source/reader/mtftools.cxx

@@ -115,7 +115,7 @@ namespace emfio
         maClip = basegfx::utils::B2DClipState();
     }
 
-    basegfx::B2DPolyPolygon WinMtfClipPath::getClipPath() const
+    basegfx::B2DPolyPolygon const & WinMtfClipPath::getClipPath() const
     {
         return maClip.getClipPoly();
     }

extensions/source/propctrlr/eventhandler.cxx

@@ -337,7 +337,7 @@ namespace pcr
         virtual ~EventHolder( ) override;
 
     private:
-        ScriptEventDescriptor impl_getDescriptor_throw( const OUString& _rEventName ) const;
+        ScriptEventDescriptor const & impl_getDescriptor_throw( const OUString& _rEventName ) const;
     };
 
 
@@ -364,7 +364,7 @@ namespace pcr
         return impl_getDescriptor_throw( _rEventName );
     }
 
-    ScriptEventDescriptor EventHolder::impl_getDescriptor_throw( const OUString& _rEventName ) const
+    ScriptEventDescriptor const & EventHolder::impl_getDescriptor_throw( const OUString& _rEventName ) const
     {
         EventMap::const_iterator pos = m_aEventNameAccess.find( _rEventName );
         if ( pos == m_aEventNameAccess.end() )

extensions/source/propctrlr/propcontroller.cxx

@@ -923,7 +923,7 @@ namespace pcr
     }
 
 
-    OPropertyBrowserController::PropertyHandlerRef OPropertyBrowserController::impl_getHandlerForProperty_throw( const OUString& _rPropertyName ) const
+    OPropertyBrowserController::PropertyHandlerRef const & OPropertyBrowserController::impl_getHandlerForProperty_throw( const OUString& _rPropertyName ) const
     {
         PropertyHandlerRepository::const_iterator handlerPos = m_aPropertyHandlers.find( _rPropertyName );
         if ( handlerPos == m_aPropertyHandlers.end() )

extensions/source/propctrlr/propcontroller.hxx

@@ -318,7 +318,7 @@ namespace pcr
             @return
                 the handler which is responsible for the given property
         */
-        PropertyHandlerRef
+        PropertyHandlerRef const &
             impl_getHandlerForProperty_throw( const OUString& _rPropertyName ) const;
 
         /** determines whether we have a handler for the given property

writerfilter/source/dmapper/DomainMapper.cxx

@@ -3764,7 +3764,7 @@ bool DomainMapper::IsRTFImport() const
     return m_pImpl->IsRTFImport();
 }
 
-uno::Reference < lang::XMultiServiceFactory > DomainMapper::GetTextFactory() const
+uno::Reference < lang::XMultiServiceFactory > const & DomainMapper::GetTextFactory() const
 {
     return m_pImpl->GetTextFactory();
 }
@@ -3780,7 +3780,7 @@ OUString DomainMapper::getOrCreateCharStyle( PropertyValueVector_t& rCharPropert
     return pStyleSheets->getOrCreateCharStyle( rCharProperties, bAlwaysCreate );
 }
 
-StyleSheetTablePtr DomainMapper::GetStyleSheetTable( )
+StyleSheetTablePtr const & DomainMapper::GetStyleSheetTable( )
 {
     return m_pImpl->GetStyleSheetTable( );
 }

writerfilter/source/dmapper/DomainMapper.hxx

@@ -96,11 +96,11 @@ public:
 
     bool IsOOXMLImport() const;
     bool IsRTFImport() const;
-    css::uno::Reference<css::lang::XMultiServiceFactory> GetTextFactory() const;
+    css::uno::Reference<css::lang::XMultiServiceFactory> const & GetTextFactory() const;
     css::uno::Reference<css::text::XTextRange> GetCurrentTextRange();
 
     OUString getOrCreateCharStyle( PropertyValueVector_t& rCharProperties, bool bAlwaysCreate );
-    std::shared_ptr< StyleSheetTable > GetStyleSheetTable( );
+    std::shared_ptr< StyleSheetTable > const & GetStyleSheetTable( );
     GraphicZOrderHelper* graphicZOrderHelper();
     GraphicNamingHelper& GetGraphicNamingHelper();
 

writerfilter/source/dmapper/DomainMapperTableManager.cxx

@@ -381,17 +381,17 @@ bool DomainMapperTableManager::sprm(Sprm & rSprm)
     return bRet;
 }
 
-std::shared_ptr< vector<sal_Int32> > DomainMapperTableManager::getCurrentGrid( )
+std::shared_ptr< vector<sal_Int32> > const & DomainMapperTableManager::getCurrentGrid( )
 {
     return m_aTableGrid.back( );
 }
 
-std::shared_ptr< vector< sal_Int32 > > DomainMapperTableManager::getCurrentSpans( )
+std::shared_ptr< vector< sal_Int32 > > const & DomainMapperTableManager::getCurrentSpans( )
 {
     return m_aGridSpans.back( );
 }
 
-std::shared_ptr< vector< sal_Int32 > > DomainMapperTableManager::getCurrentCellWidths( )
+std::shared_ptr< vector< sal_Int32 > > const & DomainMapperTableManager::getCurrentCellWidths( )
 {
     return m_aCellWidths.back( );
 }

writerfilter/source/dmapper/DomainMapperTableManager.hxx

@@ -94,9 +94,9 @@ public:
     virtual void endOfCellAction() override;
     virtual void endOfRowAction() override;
 
-    IntVectorPtr getCurrentGrid( );
-    IntVectorPtr getCurrentSpans( );
-    IntVectorPtr getCurrentCellWidths( );
+    IntVectorPtr const & getCurrentGrid( );
+    IntVectorPtr const & getCurrentSpans( );
+    IntVectorPtr const & getCurrentCellWidths( );
 
     /// Turn the attributes collected so far in m_aTableLook into a property and clear the container.
     void finishTableLook();

writerfilter/source/dmapper/DomainMapper_Impl.cxx

@@ -581,13 +581,13 @@ PropertyMapPtr DomainMapper_Impl::GetTopContextOfType(ContextType eId)
 }
 
 
-uno::Reference< text::XTextAppend >  DomainMapper_Impl::GetTopTextAppend()
+uno::Reference< text::XTextAppend > const &  DomainMapper_Impl::GetTopTextAppend()
 {
     OSL_ENSURE(!m_aTextAppendStack.empty(), "text append stack is empty" );
     return m_aTextAppendStack.top().xTextAppend;
 }
 
-FieldContextPtr  DomainMapper_Impl::GetTopFieldContext()
+FieldContextPtr const &  DomainMapper_Impl::GetTopFieldContext()
 {
     SAL_WARN_IF(m_aFieldStack.empty(), "writerfilter.dmapper", "Field stack is empty");
     return m_aFieldStack.top();

writerfilter/source/dmapper/DomainMapper_Impl.hxx

@@ -167,7 +167,7 @@ public:
     const OUString&  GetCommand() const {return m_sCommand; }
 
     void SetFieldId(FieldId eFieldId ) { m_eFieldId = eFieldId; }
-    boost::optional<FieldId> GetFieldId() const { return m_eFieldId; }
+    boost::optional<FieldId> const & GetFieldId() const { return m_eFieldId; }
 
     void AppendResult(OUString const& rResult) { m_sResult += rResult; }
     const OUString&  GetResult() const { return m_sResult; }
@@ -635,8 +635,8 @@ public:
     }
     PropertyMapPtr GetTopContextOfType(ContextType eId);
 
-    css::uno::Reference<css::text::XTextAppend> GetTopTextAppend();
-    FieldContextPtr GetTopFieldContext();
+    css::uno::Reference<css::text::XTextAppend> const & GetTopTextAppend();
+    FieldContextPtr const & GetTopFieldContext();
 
     FontTablePtr const & GetFontTable()
     {

writerfilter/source/dmapper/OLEHandler.cxx

@@ -253,7 +253,7 @@ OUString OLEHandler::getCLSID(const uno::Reference<uno::XComponentContext>& xCom
     return aRet;
 }
 
-OUString OLEHandler::GetDrawAspect() const
+OUString const & OLEHandler::GetDrawAspect() const
 {
     return m_sDrawAspect;
 }

writerfilter/source/dmapper/OLEHandler.hxx

@@ -90,7 +90,7 @@ public:
     /// Get the CLSID of the OLE object, in case we can find one based on m_sProgId.
     OUString getCLSID(const css::uno::Reference<css::uno::XComponentContext>& xComponentContext) const;
 
-    OUString GetDrawAspect() const;
+    OUString const & GetDrawAspect() const;
 
     OUString copyOLEOStream(css::uno::Reference<css::text::XTextDocument> const& xTextDocument);
 

writerfilter/source/dmapper/SettingsTable.cxx

@@ -562,7 +562,7 @@ bool SettingsTable::GetProtectForm() const
 {
     return m_pImpl->m_bProtectForm;
 }
-uno::Sequence<beans::PropertyValue> SettingsTable::GetThemeFontLangProperties() const
+uno::Sequence<beans::PropertyValue> const & SettingsTable::GetThemeFontLangProperties() const
 {
     return m_pImpl->m_pThemeFontLangProps;
 }

writerfilter/source/dmapper/SettingsTable.hxx

@@ -75,7 +75,7 @@ class SettingsTable : public LoggedProperties, public LoggedTable
     bool GetNoColumnBalance() const;
     bool GetProtectForm() const;
 
-    css::uno::Sequence<css::beans::PropertyValue> GetThemeFontLangProperties() const;
+    css::uno::Sequence<css::beans::PropertyValue> const & GetThemeFontLangProperties() const;
 
     css::uno::Sequence<css::beans::PropertyValue> GetCompatSettings() const;
 

writerfilter/source/dmapper/StyleSheetTable.cxx

@@ -395,7 +395,7 @@ StyleSheetTable::~StyleSheetTable()
 {
 }
 
-PropertyMapPtr StyleSheetTable::GetDefaultCharProps()
+PropertyMapPtr const & StyleSheetTable::GetDefaultCharProps()
 {
     return m_pImpl->m_pDefaultCharProps;
 }

writerfilter/source/dmapper/StyleSheetTable.hxx

@@ -103,7 +103,7 @@ public:
     OUString getOrCreateCharStyle( PropertyValueVector_t& rCharProperties, bool bAlwaysCreate );
 
     /// Returns the default character properties.
-    PropertyMapPtr GetDefaultCharProps();
+    PropertyMapPtr const & GetDefaultCharProps();
 
 private:
     // Properties

writerfilter/source/dmapper/TableData.hxx

@@ -218,7 +218,7 @@ public:
 
        @param i      index of the cell
      */
-    TablePropertyMapPtr getCellProperties(unsigned int i) const
+    TablePropertyMapPtr const & getCellProperties(unsigned int i) const
     {
         return mCells[i]->getProperties();
     }

writerfilter/source/dmapper/TableManager.hxx

@@ -196,13 +196,13 @@ class TableManager
     TableManagerState mState;
 
 protected:
-    TablePropertyMapPtr getCellProps()
+    TablePropertyMapPtr const & getCellProps()
     {
         return mState.getCellProps();
     }
 
 public:
-    TablePropertyMapPtr getRowProps()
+    TablePropertyMapPtr const & getRowProps()
     {
         return mState.getRowProps();
     }