leave this Clone as inline

so code isn't generated for it unless the new_clone is required,
which it isn't under gcc.

Change-Id: I07e0b3e4b2e07625509b51ba7d37e24440f56cfe

include/svl/zforlist.hxx

@@ -255,7 +255,7 @@ public:
                                          sal_Unicode cZero );
                         ~NfCurrencyEntry() {}
 
-    NfCurrencyEntry* Clone() const;
+    inline NfCurrencyEntry* Clone() const;
 
                         /// Symbols and language identical
     bool                operator==( const NfCurrencyEntry& r ) const;
@@ -305,9 +305,15 @@ public:
 };
 
 /**
- * Necessary for ptr_vector on Windows. Please don't remove this, or at
- * least check it on Windows before attempting to remove it.
+ * Necessary for ptr_vector on Windows. Please don't remove these, or at
+ * least check it on Windows before attempting to remove them.
  */
+NfCurrencyEntry* NfCurrencyEntry::Clone() const
+{
+    return new NfCurrencyEntry(
+       aSymbol, aBankSymbol, eLanguage, nPositiveFormat, nNegativeFormat, nDigits, cZeroChar);
+}
+
 inline NfCurrencyEntry* new_clone( const NfCurrencyEntry& r )
 {
     return r.Clone();

svl/source/numbers/zforlist.cxx

@@ -3837,12 +3837,6 @@ NfCurrencyEntry::NfCurrencyEntry(
     nPositiveFormat(nPositiveFmt), nNegativeFormat(nNegativeFmt),
     nDigits(nDig), cZeroChar(cZero) {}
 
-NfCurrencyEntry* NfCurrencyEntry::Clone() const
-{
-    return new NfCurrencyEntry(
-       aSymbol, aBankSymbol, eLanguage, nPositiveFormat, nNegativeFormat, nDigits, cZeroChar);
-}
-
 bool NfCurrencyEntry::operator==( const NfCurrencyEntry& r ) const
 {
     return aSymbol      == r.aSymbol

unusedcode.easy

@@ -8,7 +8,6 @@ GDriveSession::GDriveSession()
 ImpBitmap::ImplCrop(Rectangle const&)
 ImpBitmap::ImplErase(Color const&)
 ImpBitmap::ImplSetSalBitmap(SalBitmap*)
-NfCurrencyEntry::Clone() const
 OpenGLContext::init(_XDisplay*, unsigned long, unsigned int, unsigned int, int)
 OpenGLContext::renderToFile()
 OpenGLContext::requestSingleBufferedRendering()