From abe39f7781f59b96c5a8d3dd5b41c60fdf04ad84 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Fri, 6 Mar 2020 14:40:40 +0200 Subject: improve loplugin:unusedfields noticed something that wasn't being picked up, wrote some tests, and found an unhandled case in Plugin::getParentFunctionDecl Change-Id: I52b4ea273be6614e197392dfc4d6053bbc1704de Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90141 Tested-by: Jenkins Reviewed-by: Noel Grandin --- jvmfwk/inc/fwkbase.hxx | 1 - jvmfwk/source/fwkbase.cxx | 6 +++--- 2 files changed, 3 insertions(+), 4 deletions(-) (limited to 'jvmfwk') diff --git a/jvmfwk/inc/fwkbase.hxx b/jvmfwk/inc/fwkbase.hxx index a729f6ad52e4..6428083e20d0 100644 --- a/jvmfwk/inc/fwkbase.hxx +++ b/jvmfwk/inc/fwkbase.hxx @@ -33,7 +33,6 @@ struct VersionInfo; class VendorSettings { - OUString m_xmlDocVendorSettingsFileUrl; CXmlDocPtr m_xmlDocVendorSettings; CXPathContextPtr m_xmlPathContextVendorSettings; diff --git a/jvmfwk/source/fwkbase.cxx b/jvmfwk/source/fwkbase.cxx index a064b93f80be..332d84ebb07b 100644 --- a/jvmfwk/source/fwkbase.cxx +++ b/jvmfwk/source/fwkbase.cxx @@ -82,11 +82,11 @@ OUString getParamFirstUrl(OUString const & name) }//blind namespace -VendorSettings::VendorSettings(): - m_xmlDocVendorSettingsFileUrl(BootParams::getVendorSettings()) +VendorSettings::VendorSettings() { + OUString xmlDocVendorSettingsFileUrl(BootParams::getVendorSettings()); //Prepare the xml document and context - OString sSettingsPath = getVendorSettingsPath(m_xmlDocVendorSettingsFileUrl); + OString sSettingsPath = getVendorSettingsPath(xmlDocVendorSettingsFileUrl); if (sSettingsPath.isEmpty()) { OString sMsg("[Java framework] A vendor settings file was not specified." -- cgit