From d6d04400095c2663ff5e1f58018d76df78121704 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Wed, 25 Sep 2019 13:29:19 +0200 Subject: loplugin:constmethod in comphelper..desktop Change-Id: I3dbdc72b05c9d01cce279c783c60d191900f8b7c Reviewed-on: https://gerrit.libreoffice.org/79538 Tested-by: Jenkins Reviewed-by: Noel Grandin --- desktop/source/deployment/gui/dp_gui_extlistbox.hxx | 4 ++-- desktop/source/deployment/manager/dp_properties.cxx | 4 ++-- desktop/source/deployment/manager/dp_properties.hxx | 4 ++-- desktop/source/deployment/registry/executable/dp_executable.cxx | 4 ++-- desktop/source/deployment/registry/package/dp_package.cxx | 4 ++-- desktop/source/deployment/registry/sfwk/dp_parceldesc.hxx | 4 ++-- 6 files changed, 12 insertions(+), 12 deletions(-) (limited to 'desktop/source') diff --git a/desktop/source/deployment/gui/dp_gui_extlistbox.hxx b/desktop/source/deployment/gui/dp_gui_extlistbox.hxx index 5cb6edf744cc..7621807f1ef6 100644 --- a/desktop/source/deployment/gui/dp_gui_extlistbox.hxx +++ b/desktop/source/deployment/gui/dp_gui_extlistbox.hxx @@ -173,9 +173,9 @@ public: virtual void SetDrawingArea(weld::DrawingArea* pDrawingArea) override; TEntry_Impl const & GetEntryData( long nPos ) { return m_vEntries[ nPos ]; } - long GetEntryCount() { return static_cast(m_vEntries.size()); } + long GetEntryCount() const { return static_cast(m_vEntries.size()); } tools::Rectangle GetEntryRect( const long nPos ) const; - bool HasActive() { return m_bHasActive; } + bool HasActive() const { return m_bHasActive; } long PointToPos( const Point& rPos ); virtual void RecalcAll(); void RemoveUnlocked(); diff --git a/desktop/source/deployment/manager/dp_properties.cxx b/desktop/source/deployment/manager/dp_properties.cxx index 3ba2e3e9e89f..46535e0ee0e1 100644 --- a/desktop/source/deployment/manager/dp_properties.cxx +++ b/desktop/source/deployment/manager/dp_properties.cxx @@ -122,7 +122,7 @@ void ExtensionProperties::write() contentProps.writeStream( xData, true /* replace existing */ ); } -bool ExtensionProperties::isSuppressedLicense() +bool ExtensionProperties::isSuppressedLicense() const { bool ret = false; if (m_prop_suppress_license) @@ -133,7 +133,7 @@ bool ExtensionProperties::isSuppressedLicense() return ret; } -bool ExtensionProperties::isExtensionUpdate() +bool ExtensionProperties::isExtensionUpdate() const { bool ret = false; if (m_prop_extension_update) diff --git a/desktop/source/deployment/manager/dp_properties.hxx b/desktop/source/deployment/manager/dp_properties.hxx index 4bc3309c3e36..f6edd8476629 100644 --- a/desktop/source/deployment/manager/dp_properties.hxx +++ b/desktop/source/deployment/manager/dp_properties.hxx @@ -51,9 +51,9 @@ public: void write(); - bool isSuppressedLicense(); + bool isSuppressedLicense() const; - bool isExtensionUpdate(); + bool isExtensionUpdate() const; }; } diff --git a/desktop/source/deployment/registry/executable/dp_executable.cxx b/desktop/source/deployment/registry/executable/dp_executable.cxx index 95ba4e96e222..0ef1d3e2d0a3 100644 --- a/desktop/source/deployment/registry/executable/dp_executable.cxx +++ b/desktop/source/deployment/registry/executable/dp_executable.cxx @@ -61,7 +61,7 @@ class BackendImpl : public ::dp_registry::backend::PackageRegistryBackend Reference const & xCmdEnv ) override; bool getFileAttributes(sal_uInt64& out_Attributes); - bool isUrlTargetInExtension(); + bool isUrlTargetInExtension() const; public: ExecutablePackageImpl( @@ -260,7 +260,7 @@ void BackendImpl::ExecutablePackageImpl::processPackage_( //But we can check if we are within $UNO_USER_PACKAGES_CACHE etc. //Done for security reasons. For example an extension manifest could contain a path to //an executable outside the extension. -bool BackendImpl::ExecutablePackageImpl::isUrlTargetInExtension() +bool BackendImpl::ExecutablePackageImpl::isUrlTargetInExtension() const { bool bSuccess = false; OUString sExtensionDir; diff --git a/desktop/source/deployment/registry/package/dp_package.cxx b/desktop/source/deployment/registry/package/dp_package.cxx index 38f809c6a40f..712742ed93c5 100644 --- a/desktop/source/deployment/registry/package/dp_package.cxx +++ b/desktop/source/deployment/registry/package/dp_package.cxx @@ -148,7 +148,7 @@ class BackendImpl : public ImplBaseT const Reference< ucb::XCommandEnvironment >& xCmdEnv, const OUString& licenseUrl); - DescriptionInfoset getDescriptionInfoset(); + DescriptionInfoset getDescriptionInfoset() const; // Package virtual beans::Optional< beans::Ambiguous > isRegistered_( @@ -561,7 +561,7 @@ OUString BackendImpl::PackageImpl::getTextFromURL( } -DescriptionInfoset BackendImpl::PackageImpl::getDescriptionInfoset() +DescriptionInfoset BackendImpl::PackageImpl::getDescriptionInfoset() const { return dp_misc::getDescriptionInfoset(m_url_expanded); } diff --git a/desktop/source/deployment/registry/sfwk/dp_parceldesc.hxx b/desktop/source/deployment/registry/sfwk/dp_parceldesc.hxx index 620e01b24c3c..001e8b2e156f 100644 --- a/desktop/source/deployment/registry/sfwk/dp_parceldesc.hxx +++ b/desktop/source/deployment/registry/sfwk/dp_parceldesc.hxx @@ -45,8 +45,8 @@ private: sal_Int32 skipIndex; public: ParcelDescDocHandler():m_bIsParsed( false ), skipIndex( 0 ){} - const OUString& getParcelLanguage() { return m_sLang; } - bool isParsed() { return m_bIsParsed; } + const OUString& getParcelLanguage() const { return m_sLang; } + bool isParsed() const { return m_bIsParsed; } // XDocumentHandler virtual void SAL_CALL startDocument() override; -- cgit