On Wed, Feb 16, 2011 at 07:56:06AM +0100, David Tardon wrote:
On Sun, Feb 13, 2011 at 08:42:42PM +0100, Christina Roßmanith wrote:
diff --git a/svtools/source/config/menuoptions.cxx b/svtools/source/config/menuoptions.cxx
index a12f1e0..4e814a7 100644
--- a/svtools/source/config/menuoptions.cxx
+++ b/svtools/source/config/menuoptions.cxx
@@ -62,13 +62,13 @@ using namespace ::com::sun::star::uno ;
-#define PROPERTYNAME_SHOWICONSINMENUES
OUString(RTL_CONSTASCII_USTRINGPARAM("ShowIconsInMenues" ))
-#define PROPERTYNAME_SYSTEMICONSINMENUES
OUString(RTL_CONSTASCII_USTRINGPARAM("IsSystemIconsInMenus" ))
+#define PROPERTYNAME_SHOWICONSINMENUS
OUString(RTL_CONSTASCII_USTRINGPARAM("ShowIconsInMenus" ))
+#define PROPERTYNAME_SYSTEMICONSINMENUS
OUString(RTL_CONSTASCII_USTRINGPARAM("IsSystemIconsInMenus" ))
I have removed the changes related to these properties. The property
names are documented and used at other places, so we cannot just change
them. As for the handles, they can be changed, as they are only used
locally, but IMHO it is better if the handle's name do not differ from
the property's name.
Actually, this is not exactly true. These are not properties defined by
UNO api, they are just configuration keys. I think if we allow reading
of the old-spelled key for some time for compatibility and only write
the new-spelled key, noone can want more.
D.
Context
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.