sw, writerfilter: avoid some unnecessary value parameters

Change-Id: I409eb644bdf519e097c87911b90f83c5ecd913ee
Reviewed-on: https://gerrit.libreoffice.org/61853
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>

sw/source/filter/ww8/docxsdrexport.cxx

@@ -193,7 +193,7 @@ struct DocxSdrExport::Impl
     bool checkFrameBtlr(SwNode* pStartNode, bool bDML);
 };
 
-DocxSdrExport::DocxSdrExport(DocxExport& rExport, sax_fastparser::FSHelperPtr pSerializer,
+DocxSdrExport::DocxSdrExport(DocxExport& rExport, const sax_fastparser::FSHelperPtr& pSerializer,
                              oox::drawingml::DrawingML* pDrawingML)
     : m_pImpl(o3tl::make_unique<Impl>(*this, rExport, pSerializer, pDrawingML))
 {

sw/source/filter/ww8/docxsdrexport.hxx

@@ -55,7 +55,7 @@ class DocxSdrExport
     struct Impl;
     std::unique_ptr<Impl> m_pImpl;
 public:
-    DocxSdrExport(DocxExport& rExport, sax_fastparser::FSHelperPtr pSerializer, oox::drawingml::DrawingML* pDrawingML);
+    DocxSdrExport(DocxExport& rExport, const sax_fastparser::FSHelperPtr& pSerializer, oox::drawingml::DrawingML* pDrawingML);
     ~DocxSdrExport();
 
     void setSerializer(const sax_fastparser::FSHelperPtr& pSerializer);

writerfilter/source/rtftok/rtfsprm.cxx

@@ -76,7 +76,7 @@ RTFValue::Pointer_t RTFSprms::find(Id nKeyword, bool bFirst, bool bForWrite)
     return pValue;
 }
 
-void RTFSprms::set(Id nKeyword, RTFValue::Pointer_t pValue, RTFOverwrite eOverwrite)
+void RTFSprms::set(Id nKeyword, const RTFValue::Pointer_t& pValue, RTFOverwrite eOverwrite)
 {
     ensureCopyBeforeWrite();
 

writerfilter/source/rtftok/rtfsprm.hxx

@@ -55,7 +55,8 @@ public:
 
     RTFValue::Pointer_t find(Id nKeyword, bool bFirst = true, bool bForWrite = false);
     /// Does the same as ->push_back(), except that it can overwrite or ignore existing entries.
-    void set(Id nKeyword, RTFValue::Pointer_t pValue, RTFOverwrite eOverwrite = RTFOverwrite::YES);
+    void set(Id nKeyword, const RTFValue::Pointer_t& pValue,
+             RTFOverwrite eOverwrite = RTFOverwrite::YES);
     bool erase(Id nKeyword);
     void eraseLast(Id nKeyword);
     /// Removes elements which are already in the reference set.