[Libreoffice-commits] core.git: svl/qa svl/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Mon Oct 8 08:25:52 UTC 2018


 svl/qa/unit/svl.cxx              |    8 ++++----
 svl/source/crypto/cryptosign.cxx |    6 +++---
 svl/source/fsstor/fsstorage.cxx  |    4 ++--
 svl/source/items/style.cxx       |    2 +-
 svl/source/misc/gridprinter.cxx  |    2 +-
 svl/source/undo/undo.cxx         |    6 +++---
 svl/source/uno/pathservice.cxx   |    2 +-
 7 files changed, 15 insertions(+), 15 deletions(-)

New commits:
commit 5181253946ca1877cc42050452aa6d733d6da3f1
Author:     Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Mon Oct 8 08:27:51 2018 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Mon Oct 8 10:25:31 2018 +0200

    loplugin:constfields in svl
    
    Change-Id: I18183c5c257cbe69bd067d4e74c50483ae683cf3
    Reviewed-on: https://gerrit.libreoffice.org/61509
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/svl/qa/unit/svl.cxx b/svl/qa/unit/svl.cxx
index 5c14f6539baa..76730bec7127 100644
--- a/svl/qa/unit/svl.cxx
+++ b/svl/qa/unit/svl.cxx
@@ -233,9 +233,9 @@ void Test::testNumberFormat()
     };
 
     struct {
-        NfIndexTableOffset eStart;
-        NfIndexTableOffset eEnd;
-        size_t nSize;
+        NfIndexTableOffset const eStart;
+        NfIndexTableOffset const eEnd;
+        size_t const nSize;
         const char** pCodes;
     } aTests[] = {
         { NF_NUMBER_START, NF_NUMBER_END, 6, pNumber },
@@ -1101,7 +1101,7 @@ void Test::testIsNumberFormat()
     struct NumberFormatData
     {
         const char* pFormat;
-        bool bIsNumber;
+        bool const bIsNumber;
     } aTests[] = {
         { "20.3", true },
         { "2", true },
diff --git a/svl/source/crypto/cryptosign.cxx b/svl/source/crypto/cryptosign.cxx
index 0e86664c28c1..57da138391ed 100644
--- a/svl/source/crypto/cryptosign.cxx
+++ b/svl/source/crypto/cryptosign.cxx
@@ -118,9 +118,9 @@ Extension  ::=  SEQUENCE  {
 */
 
 typedef struct {
-    SECItem extnID;
-    SECItem critical;
-    SECItem extnValue;
+    SECItem const extnID;
+    SECItem const critical;
+    SECItem const extnValue;
 } Extension;
 
 /*
diff --git a/svl/source/fsstor/fsstorage.cxx b/svl/source/fsstor/fsstorage.cxx
index c05c788b1bd8..360083cf386a 100644
--- a/svl/source/fsstor/fsstorage.cxx
+++ b/svl/source/fsstor/fsstorage.cxx
@@ -67,10 +67,10 @@ using namespace ::com::sun::star;
 
 struct FSStorage_Impl
 {
-    OUString m_aURL;
+    OUString const m_aURL;
 
     ::ucbhelper::Content m_aContent;
-    sal_Int32 m_nMode;
+    sal_Int32 const m_nMode;
 
     std::unique_ptr<::comphelper::OInterfaceContainerHelper2> m_pListenersContainer; // list of listeners
     std::unique_ptr<::cppu::OTypeCollection> m_pTypeCollection;
diff --git a/svl/source/items/style.cxx b/svl/source/items/style.cxx
index 3517c68cd718..acfa74f8e910 100644
--- a/svl/source/items/style.cxx
+++ b/svl/source/items/style.cxx
@@ -388,7 +388,7 @@ struct DoesStyleMatchStyleSheetPredicate final : public svl::StyleSheetPredicate
         return bMatches;
     }
 
-    SfxStyleSheetIterator *mIterator;
+    SfxStyleSheetIterator * const mIterator;
 };
 
 }
diff --git a/svl/source/misc/gridprinter.cxx b/svl/source/misc/gridprinter.cxx
index fc89fdea4688..9ed7f43b41d5 100644
--- a/svl/source/misc/gridprinter.cxx
+++ b/svl/source/misc/gridprinter.cxx
@@ -51,7 +51,7 @@ typedef mdds::multi_type_matrix<matrix_trait> MatrixImplType;
 struct GridPrinter::Impl
 {
     MatrixImplType maMatrix;
-    bool mbPrint;
+    bool const mbPrint;
 
     Impl( size_t nRows, size_t nCols, bool bPrint ) :
         maMatrix(nRows, nCols, OUString()), mbPrint(bPrint) {}
diff --git a/svl/source/undo/undo.cxx b/svl/source/undo/undo.cxx
index cc7cb8157ff8..3b2085108126 100644
--- a/svl/source/undo/undo.cxx
+++ b/svl/source/undo/undo.cxx
@@ -1298,11 +1298,11 @@ void SfxUndoManager::EmptyActionsChanged()
 
 struct SfxListUndoAction::Impl
 {
-    sal_uInt16 mnId;
-    ViewShellId mnViewShellId;
+    sal_uInt16 const mnId;
+    ViewShellId const mnViewShellId;
 
     OUString maComment;
-    OUString maRepeatComment;
+    OUString const maRepeatComment;
 
     Impl( sal_uInt16 nId, ViewShellId nViewShellId, const OUString& rComment, const OUString& rRepeatComment ) :
         mnId(nId), mnViewShellId(nViewShellId), maComment(rComment), maRepeatComment(rRepeatComment) {}
diff --git a/svl/source/uno/pathservice.cxx b/svl/source/uno/pathservice.cxx
index 8e49057f1e9c..90aeeddfc3ed 100644
--- a/svl/source/uno/pathservice.cxx
+++ b/svl/source/uno/pathservice.cxx
@@ -33,7 +33,7 @@ namespace com { namespace sun { namespace star { namespace uno {
 
 class PathService : public ::cppu::WeakImplHelper< css::frame::XConfigManager, css::lang::XServiceInfo >
 {
-    SvtPathOptions m_aOptions;
+    SvtPathOptions const m_aOptions;
 
 public:
     PathService()


More information about the Libreoffice-commits mailing list