svx: change instances of maName to more descriptive names

maName is just not descriptive enough and hard to search the source for.

Also in this patch:
- tracked down what ASE, GPL and SOC stand for and added a comment
- found instances of aName, local variables and changed these to
aPaletteName

Change-Id: I4c7eee31b604a8ef2ebef5fd1daa81e193dd7295
Reviewed-on: https://gerrit.libreoffice.org/20958
Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

1 files changed, 9 insertions, 3 deletions

diff --git a/include/svx/Palette.hxx b/include/svx/Palette.hxx
index 9c50899ec687..fa2c1798ff1e 100644
--- a/include/svx/Palette.hxx
+++ b/include/svx/Palette.hxx
@@ -40,11 +40,13 @@ public:
     virtual bool                IsValid() = 0;
 };
 
+// ASE = Adobe Swatch Exchange
+
 class PaletteASE : public Palette
 {
     bool        mbValidPalette;
     OUString    maFPath;
-    OUString    maName;
+    OUString    maASEPaletteName;
     ColorList   maColors;
 
     void        LoadPalette();
@@ -58,13 +60,15 @@ public:
     virtual bool                IsValid() override;
 };
 
+// GPL - this is *not* GNU Public License, but is the Gimp PaLette
+
 class PaletteGPL : public Palette
 {
     bool        mbLoadedPalette;
     bool        mbValidPalette;
     OUString    maFName;
     OUString    maFPath;
-    OUString    maName;
+    OUString    maGPLPaletteName;
     ColorList   maColors;
 
     bool        ReadPaletteHeader(SvFileStream& rFileStream);
@@ -80,11 +84,13 @@ public:
     virtual bool                IsValid() override;
 };
 
+// SOC - Star Office Color-table
+
 class PaletteSOC : public Palette
 {
     bool            mbLoadedPalette;
     OUString        maFPath;
-    OUString        maName;
+    OUString        maSOCPaletteName;
     XColorListRef   mpColorList;
 public:
     PaletteSOC( const OUString &rFPath, const OUString &rFName );