diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-28 11:47:51 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-28 14:51:56 +0200 |
commit | b09deb075319c1c19f91e3a6f64429b61682ebf8 (patch) | |
tree | 9e73d1e63092083a337a4ddf1dfe962b4f6ef4dc /extensions | |
parent | 587d6e13df7bb9c8f78ce605ebb8f8fd34b85a2c (diff) |
loplugin:constparams handle constructors
had to change the structure of the plugin considerably, was too messy to
structure it to do the calculations on a per-function basis
Change-Id: I4edee7735f726101105c607368124a08dba21086
Reviewed-on: https://gerrit.libreoffice.org/40516
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'extensions')
-rw-r--r-- | extensions/source/bibliography/datman.cxx | 4 | ||||
-rw-r--r-- | extensions/source/bibliography/framectr.cxx | 2 | ||||
-rw-r--r-- | extensions/source/config/ldap/ldapuserprofilebe.cxx | 2 | ||||
-rw-r--r-- | extensions/source/config/ldap/ldapuserprofilebe.hxx | 2 | ||||
-rw-r--r-- | extensions/source/update/ui/updatecheckui.cxx | 2 |
5 files changed, 6 insertions, 6 deletions
diff --git a/extensions/source/bibliography/datman.cxx b/extensions/source/bibliography/datman.cxx index 9890daa05654..9ae2804a36b3 100644 --- a/extensions/source/bibliography/datman.cxx +++ b/extensions/source/bibliography/datman.cxx @@ -237,8 +237,8 @@ public: virtual void dispose() override; }; -static sal_uInt16 lcl_FindLogicalName(BibConfig* pConfig , - const OUString& rLogicalColumnName) +static sal_uInt16 lcl_FindLogicalName(BibConfig const * pConfig , + const OUString& rLogicalColumnName) { for(sal_uInt16 i = 0; i < COLUMN_COUNT; i++) { diff --git a/extensions/source/bibliography/framectr.cxx b/extensions/source/bibliography/framectr.cxx index b5cfe93794b7..35e825f567ee 100644 --- a/extensions/source/bibliography/framectr.cxx +++ b/extensions/source/bibliography/framectr.cxx @@ -366,7 +366,7 @@ bool BibFrameController_Impl::SaveModified(const Reference< form::runtime::XForm return bResult; } -static vcl::Window* lcl_GetFocusChild( vcl::Window* pParent ) +static vcl::Window* lcl_GetFocusChild( vcl::Window const * pParent ) { sal_uInt16 nChildren = pParent->GetChildCount(); for( sal_uInt16 nChild = 0; nChild < nChildren; ++nChild) diff --git a/extensions/source/config/ldap/ldapuserprofilebe.cxx b/extensions/source/config/ldap/ldapuserprofilebe.cxx index 54d60e335c50..10f7ae002c47 100644 --- a/extensions/source/config/ldap/ldapuserprofilebe.cxx +++ b/extensions/source/config/ldap/ldapuserprofilebe.cxx @@ -157,7 +157,7 @@ bool LdapUserProfileBe::readLdapConfiguration( bool LdapUserProfileBe::getLdapStringParam( - uno::Reference<container::XNameAccess>& xAccess, + uno::Reference<container::XNameAccess> const & xAccess, const OUString& aLdapSetting, OUString& aServerParameter) { diff --git a/extensions/source/config/ldap/ldapuserprofilebe.hxx b/extensions/source/config/ldap/ldapuserprofilebe.hxx index 74838da874cf..5cca247809b2 100644 --- a/extensions/source/config/ldap/ldapuserprofilebe.hxx +++ b/extensions/source/config/ldap/ldapuserprofilebe.hxx @@ -116,7 +116,7 @@ class LdapUserProfileBe : private LdapProfileMutexHolder, public BackendBase uno::Reference<uno::XComponentContext> const & context, LdapDefinition * definition, OUString * loggedOnUser); - static bool getLdapStringParam(uno::Reference<container::XNameAccess>& xAccess, + static bool getLdapStringParam(uno::Reference<container::XNameAccess> const & xAccess, const OUString& aLdapSetting, OUString& aServerParameter); diff --git a/extensions/source/update/ui/updatecheckui.cxx b/extensions/source/update/ui/updatecheckui.cxx index 2b62b30f1990..bf530dbaffd2 100644 --- a/extensions/source/update/ui/updatecheckui.cxx +++ b/extensions/source/update/ui/updatecheckui.cxx @@ -77,7 +77,7 @@ static OUString getImplementationName() namespace { -Image GetMenuBarIcon( MenuBar* pMBar ) +Image GetMenuBarIcon( MenuBar const * pMBar ) { OUString sResID; vcl::Window *pMBarWin = pMBar->GetWindow(); |