loplugin:countusersofdefaultparams in include/oox..sax

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

16 files changed, 30 insertions, 28 deletions

diff --git a/include/oox/dump/dumperbase.hxx b/include/oox/dump/dumperbase.hxx
index 772197a538ee..048ea78fb88b 100644
--- a/include/oox/dump/dumperbase.hxx
+++ b/include/oox/dump/dumperbase.hxx
@@ -1039,8 +1039,8 @@ private:
 class ItemGuard
 {
 public:
-    explicit            ItemGuard( const OutputRef& rxOut, const String& rName = EMPTY_STRING ) :
-                     mrOut( *rxOut ) { mrOut.startItem( rName ); }
+    explicit            ItemGuard( const OutputRef& rxOut, const String& rName ) :
+                            mrOut( *rxOut ) { mrOut.startItem( rName ); }
                         ~ItemGuard() { mrOut.endItem(); }
     void                cont() { mrOut.contItem(); }
 private:
@@ -1632,7 +1632,7 @@ protected:
                             const OUString& rSysFileName,
                             const BinaryInputStreamRef& rxRecStrm,
                             const String& rRecNames,
-                            const String& rSimpleRecs = EMPTY_STRING );
+                            const String& rSimpleRecs );
 
     sal_Int64           getRecId() const { return mnRecId; }
     sal_Int64           getRecSize() const { return mnRecSize; }
@@ -1675,7 +1675,7 @@ protected:
                             const BinaryInputStreamRef& rxBaseStrm,
                             const OUString& rSysFileName,
                             const String& rRecNames,
-                            const String& rSimpleRecs = EMPTY_STRING );
+                            const String& rSimpleRecs );
 
     virtual bool        implStartRecord( BinaryInputStream& rBaseStrm, sal_Int64& ornRecPos, sal_Int64& ornRecId, sal_Int64& ornRecSize ) override;
     virtual bool        implReadRecordHeader( BinaryInputStream& rBaseStrm, sal_Int64& ornRecId, sal_Int64& ornRecSize ) = 0;
diff --git a/include/oox/dump/oledumper.hxx b/include/oox/dump/oledumper.hxx
index d79e4d545093..ef18258da217 100644
--- a/include/oox/dump/oledumper.hxx
+++ b/include/oox/dump/oledumper.hxx
@@ -50,7 +50,7 @@ protected:
     OUString     dumpAnsiString32( const String& rName );
     OUString     dumpUniString32( const String& rName );
 
-    sal_Int32           dumpStdClipboardFormat( const String& rName = EMPTY_STRING );
+    sal_Int32    dumpStdClipboardFormat( const String& rName );
     OUString     dumpAnsiString32OrStdClip( const String& rName );
     OUString     dumpUniString32OrStdClip( const String& rName );
 
@@ -734,10 +734,9 @@ protected:
     virtual void        implDumpRecordBody() override;
 
 private:
-    OUString     dumpByteString( const String& rName = EMPTY_STRING );
-    OUString     dumpUniString( const String& rName = EMPTY_STRING );
-
-    OUString     dumpByteStringWithLength( const String& rName = EMPTY_STRING );
+    OUString     dumpByteString( const String& rName );
+    OUString     dumpUniString( const String& rName );
+    OUString     dumpByteStringWithLength( const String& rName );
 
 private:
     VbaSharedData&      mrVbaData;
diff --git a/include/oox/export/chartexport.hxx b/include/oox/export/chartexport.hxx
index 528dc84473f5..7f2cb592b20a 100644
--- a/include/oox/export/chartexport.hxx
+++ b/include/oox/export/chartexport.hxx
@@ -207,7 +207,8 @@ private:
 
 public:
 
-    ChartExport( sal_Int32 nXmlNamespace, ::sax_fastparser::FSHelperPtr pFS, css::uno::Reference< css::frame::XModel >& xModel, ::oox::core::XmlFilterBase* pFB = nullptr, DocumentType eDocumentType = DOCUMENT_PPTX );
+    ChartExport( sal_Int32 nXmlNamespace, ::sax_fastparser::FSHelperPtr pFS, css::uno::Reference< css::frame::XModel >& xModel,
+                 ::oox::core::XmlFilterBase* pFB, DocumentType eDocumentType );
     virtual ~ChartExport() {}
 
     sal_Int32           GetChartID( );
diff --git a/include/oox/export/shapes.hxx b/include/oox/export/shapes.hxx
index fad36643b4c6..5c1d7d860ca7 100644
--- a/include/oox/export/shapes.hxx
+++ b/include/oox/export/shapes.hxx
@@ -126,7 +126,10 @@ private:
 
 public:
 
-    ShapeExport( sal_Int32 nXmlNamespace, ::sax_fastparser::FSHelperPtr pFS, ShapeHashMap* pShapeMap = nullptr, ::oox::core::XmlFilterBase* pFB = nullptr, DocumentType eDocumentType = DOCUMENT_PPTX, DMLTextExport* pTextExport = nullptr );
+    ShapeExport( sal_Int32 nXmlNamespace, ::sax_fastparser::FSHelperPtr pFS,
+                 ShapeHashMap* pShapeMap, ::oox::core::XmlFilterBase* pFB,
+                 DocumentType eDocumentType = DOCUMENT_PPTX,
+                 DMLTextExport* pTextExport = nullptr );
     virtual ~ShapeExport() {}
 
     void SetURLTranslator(const std::shared_ptr<URLTransformer>& pTransformer);
diff --git a/include/oox/helper/binaryinputstream.hxx b/include/oox/helper/binaryinputstream.hxx
index 2e952d3ba9e5..bbc2550cb2c6 100644
--- a/include/oox/helper/binaryinputstream.hxx
+++ b/include/oox/helper/binaryinputstream.hxx
@@ -152,7 +152,7 @@ public:
             True = NUL characters are inserted into the imported string.
             False = NUL characters are replaced by question marks (default).
      */
-    OString      readCharArray( sal_Int32 nChars, bool bAllowNulChars = false );
+    OString      readCharArray( sal_Int32 nChars, bool bAllowNulChars );
 
     /** Reads a byte character array and returns a Unicode string.
 
@@ -351,7 +351,7 @@ public:
      */
     explicit            RelativeInputStream(
                             BinaryInputStream& rInStrm,
-                            sal_Int64 nSize = SAL_MAX_INT64 );
+                            sal_Int64 nSize );
 
     /** Returns the size of the data block in the wrapped stream offered by
         this wrapper. */
diff --git a/include/oox/helper/binarystreambase.hxx b/include/oox/helper/binarystreambase.hxx
index d5f2387872f8..9868fc542400 100644
--- a/include/oox/helper/binarystreambase.hxx
+++ b/include/oox/helper/binarystreambase.hxx
@@ -100,7 +100,7 @@ public:
         @param nAnchorPos
             Position in the stream the data blocks are aligned to.
      */
-    void                alignToBlock( sal_Int32 nBlockSize, sal_Int64 nAnchorPos = 0 );
+    void                alignToBlock( sal_Int32 nBlockSize, sal_Int64 nAnchorPos );
 
 protected:
     explicit            BinaryStreamBase( bool bSeekable ) : mbEof( false ), mbSeekable( bSeekable ) {}
diff --git a/include/oox/helper/containerhelper.hxx b/include/oox/helper/containerhelper.hxx
index 016df0d60d3f..9c0632a53aa0 100644
--- a/include/oox/helper/containerhelper.hxx
+++ b/include/oox/helper/containerhelper.hxx
@@ -44,7 +44,7 @@ struct ValueRange
     sal_Int32           mnFirst;
     sal_Int32           mnLast;
 
-    explicit     ValueRange( sal_Int32 nValue = 0 ) : mnFirst( nValue ), mnLast( nValue ) {}
+    explicit     ValueRange( sal_Int32 nValue ) : mnFirst( nValue ), mnLast( nValue ) {}
     explicit     ValueRange( sal_Int32 nFirst, sal_Int32 nLast ) : mnFirst( nFirst ), mnLast( nLast ) {}
 
     bool         operator==( const ValueRange& rRange ) const { return (mnFirst == rRange.mnFirst) && (mnLast == rRange.mnLast); }
diff --git a/include/oox/helper/graphichelper.hxx b/include/oox/helper/graphichelper.hxx
index 6301332fc7eb..c2773ae6af0e 100644
--- a/include/oox/helper/graphichelper.hxx
+++ b/include/oox/helper/graphichelper.hxx
@@ -133,7 +133,7 @@ public:
         @return  The URL of the created and internally cached graphic object. */
     OUString     importGraphicObject(
                             const css::uno::Reference< css::io::XInputStream >& rxInStrm,
-                            const WMF_EXTERNALHEADER* pExtHeader = nullptr ) const;
+                            const WMF_EXTERNALHEADER* pExtHeader ) const;
 
     /** Creates a persistent graphic object from the passed binary memory block.
         @return  The URL of the created and internally cached graphic object. */
diff --git a/include/oox/mathml/importutils.hxx b/include/oox/mathml/importutils.hxx
index bdfdf7cd2fa1..8481cb9a32b7 100644
--- a/include/oox/mathml/importutils.hxx
+++ b/include/oox/mathml/importutils.hxx
@@ -120,7 +120,7 @@ public:
     struct OOX_DLLPUBLIC AttributeList
     {
         OUString& operator[] (int token);
-        OUString attribute( int token, const OUString& def = OUString()) const;
+        OUString attribute( int token, const OUString& def) const;
         bool attribute( int token, bool def ) const;
         sal_Unicode attribute( int token, sal_Unicode def ) const;
         // when adding more attribute() overloads, add also to XmlStream itself
diff --git a/include/oox/ole/olehelper.hxx b/include/oox/ole/olehelper.hxx
index 4e2a2d8e62fc..76528379053e 100644
--- a/include/oox/ole/olehelper.hxx
+++ b/include/oox/ole/olehelper.hxx
@@ -105,7 +105,7 @@ namespace OleHelper
     OOX_DLLPUBLIC sal_Int32    decodeOleColor(
                             const GraphicHelper& rGraphicHelper,
                             sal_uInt32 nOleColor,
-                            bool bDefaultColorBgr = true );
+                            bool bDefaultColorBgr );
 
     /** Returns the OLE color from the passed UNO RGB color.
      */
diff --git a/include/oox/ole/vbacontrol.hxx b/include/oox/ole/vbacontrol.hxx
index f7e22cf68eda..8a54c6d3ba18 100644
--- a/include/oox/ole/vbacontrol.hxx
+++ b/include/oox/ole/vbacontrol.hxx
@@ -191,7 +191,7 @@ public:
                             const css::uno::Reference< css::uno::XComponentContext >& rxContext,
                             const css::uno::Reference< css::frame::XModel >& rxDocModel,
                             const GraphicHelper& rGraphicHelper,
-                            bool bDefaultColorBgr = true );
+                            bool bDefaultColorBgr );
 
     /** Imports the form and its embedded controls, and inserts the form with
         all its controls into the passed dialog library. */
diff --git a/include/oox/ppt/pptshape.hxx b/include/oox/ppt/pptshape.hxx
index ffd1cf9fda48..5ae47157ab2a 100644
--- a/include/oox/ppt/pptshape.hxx
+++ b/include/oox/ppt/pptshape.hxx
@@ -54,7 +54,7 @@ class PPTShape : public oox::drawingml::Shape
 public:
 
     PPTShape( const oox::ppt::ShapeLocation eShapeLocation,
-                const sal_Char* pServiceType = nullptr );
+                const sal_Char* pServiceType );
     virtual ~PPTShape() override;
 
     using oox::drawingml::Shape::addShape;
@@ -65,7 +65,7 @@ public:
             const oox::drawingml::Theme* pTheme,
             const css::uno::Reference< css::drawing::XShapes >& rxShapes,
             basegfx::B2DHomMatrix& aTransformation,
-            ::oox::drawingml::ShapeIdMap* pShapeMap = nullptr );
+            ::oox::drawingml::ShapeIdMap* pShapeMap );
 
     ShapeLocation getShapeLocation() const { return meShapeLocation; };
     void setReferenced( bool bReferenced ){ mbReferenced = bReferenced; };
diff --git a/include/package/Inflater.hxx b/include/package/Inflater.hxx
index 2abfad209341..a974534bc901 100644
--- a/include/package/Inflater.hxx
+++ b/include/package/Inflater.hxx
@@ -39,7 +39,7 @@ protected:
     sal_Int32   doInflateBytes (css::uno::Sequence < sal_Int8 > &rBuffer, sal_Int32 nNewOffset, sal_Int32 nNewLength);
 
 public:
-    Inflater(bool bNoWrap = false);
+    Inflater(bool bNoWrap);
     ~Inflater();
     void SAL_CALL setInput( const css::uno::Sequence< sal_Int8 >& rBuffer );
     bool SAL_CALL needsDictionary(  ) { return bNeedDict;}
diff --git a/include/sax/fastattribs.hxx b/include/sax/fastattribs.hxx
index 1dcbbf491869..0349ebdc6f17 100644
--- a/include/sax/fastattribs.hxx
+++ b/include/sax/fastattribs.hxx
@@ -65,10 +65,9 @@ class SAX_DLLPUBLIC FastTokenHandlerBase
      * @return Tokenized form of pStr
      */
     static sal_Int32 getTokenFromChars(
-                         const css::uno::Reference<
-                             css::xml::sax::XFastTokenHandler > &xTokenHandler,
+                         const css::uno::Reference<css::xml::sax::XFastTokenHandler > &xTokenHandler,
                          FastTokenHandlerBase *pTokenHandler /* can be NULL */,
-                         const char *pStr, size_t nLength = 0 );
+                         const char *pStr, size_t nLength );
 };
 
 class SAX_DLLPUBLIC FastAttributeList : public ::cppu::WeakImplHelper1< css::xml::sax::XFastAttributeList >
diff --git a/include/sax/fshelper.hxx b/include/sax/fshelper.hxx
index fd9b85661c91..60c0b87b6b56 100644
--- a/include/sax/fshelper.hxx
+++ b/include/sax/fshelper.hxx
@@ -70,7 +70,7 @@ class SAX_DLLPUBLIC FastSerializerHelper
 {
 public:
 
-    FastSerializerHelper( const css::uno::Reference< css::io::XOutputStream >& xOutputStream, bool bWriteHeader = true );
+    FastSerializerHelper( const css::uno::Reference< css::io::XOutputStream >& xOutputStream, bool bWriteHeader );
 
     ~FastSerializerHelper();
 
diff --git a/include/sax/tools/converter.hxx b/include/sax/tools/converter.hxx
index 072c3b438d6e..77e5a3839e7b 100644
--- a/include/sax/tools/converter.hxx
+++ b/include/sax/tools/converter.hxx
@@ -67,8 +67,8 @@ public:
     /** convert measure to string */
     static void convertMeasure( OUStringBuffer& rBuffer,
                                 sal_Int32 nMeasure,
-                                sal_Int16 SourceUnit = css::util::MeasureUnit::MM_100TH,
-                                sal_Int16 nTargetUnit = css::util::MeasureUnit::INCH  );
+                                sal_Int16 SourceUnit,
+                                sal_Int16 nTargetUnit  );
 
     /** convert string to boolean */
     static bool convertBool( bool& rBool,