can return a ref, not a pointer, and make local static instead of global leak

vcl/unx/generic/printergfx/psputil.cxx

@@ -28,13 +28,11 @@
 
 // MARKER(update_precomp.py): autogen include statement, do not remove
 #include "precompiled_vcl.hxx"
-
 #include <string.h>
-
+#include <tools/debug.hxx>
+#include <rtl/instance.hxx>
 #include "psputil.hxx"
 
-#include "tools/debug.hxx"
-
 namespace psp {
 
 /*
@@ -254,15 +252,17 @@ ConverterFactory::Convert (const sal_Unicode *pText, int nTextLen,
     return nSize;
 }
 
-ConverterFactory*
-GetConverterFactory ()
+namespace
 {
-    static ConverterFactory* pCvt = NULL;
-
-    if (pCvt == NULL)
-        pCvt = new ConverterFactory;
+    class theConverterFactory
+        : public rtl::Static<ConverterFactory, theConverterFactory>
+    {
+    };
+}
 
-    return pCvt;
+ConverterFactory& GetConverterFactory()
+{
+    return theConverterFactory::get();
 }
 
 

vcl/unx/generic/printergfx/psputil.hxx

@@ -71,7 +71,7 @@ private:
     std::map< rtl_TextEncoding, rtl_UnicodeToTextConverter >        m_aConverters;
 };
 
-ConverterFactory* GetConverterFactory ();
+ConverterFactory& GetConverterFactory ();
 
 }  /* namespace psp */
 

vcl/unx/generic/printergfx/text_gfx.cxx

@@ -583,8 +583,8 @@ PrinterGfx::LicenseWarning(const Point& rPoint, const sal_Unicode* pStr,
     sal_Size  nSize    = 4 * nLen;
     sal_uChar* pBuffer = (sal_uChar*)alloca (nSize* sizeof(sal_uChar));
 
-    ConverterFactory* pCvt = GetConverterFactory ();
-    nSize = pCvt->Convert (pStr, nLen, pBuffer, nSize, RTL_TEXTENCODING_ISO_8859_1);
+    ConverterFactory &rCvt = GetConverterFactory ();
+    nSize = rCvt.Convert (pStr, nLen, pBuffer, nSize, RTL_TEXTENCODING_ISO_8859_1);
 
     PSMoveTo (rPoint);
     PSShowText (pBuffer, nLen, nSize, pDeltaArray);