tdf#62525 vcl: use cow_wrapper for lineinfo

Change-Id: I7fe29813e78c23b043b98f212a6ccd092def78b7
Reviewed-on: https://gerrit.libreoffice.org/24437
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>

2 files changed, 27 insertions, 78 deletions

diff --git a/include/vcl/lineinfo.hxx b/include/vcl/lineinfo.hxx
index 82a7e8cc28c1..e5d8fc9855ec 100644
--- a/include/vcl/lineinfo.hxx
+++ b/include/vcl/lineinfo.hxx
@@ -25,14 +25,13 @@
 #include <vcl/vclenum.hxx>
 #include <basegfx/vector/b2enums.hxx>
 #include <com/sun/star/drawing/LineCap.hpp>
-
+#include <o3tl/cow_wrapper.hxx>
 
 class SvStream;
 namespace basegfx { class B2DPolyPolygon; }
 
 struct ImplLineInfo
 {
-    sal_uInt32              mnRefCount;
     LineStyle               meStyle;
     long                    mnWidth;
     sal_uInt16              mnDashCount;
@@ -48,22 +47,12 @@ struct ImplLineInfo
                         ImplLineInfo( const ImplLineInfo& rImplLineInfo );
 
     bool operator==( const ImplLineInfo& ) const;
-
-    friend SvStream&    ReadImplLineInfo( SvStream& rIStm, ImplLineInfo& rImplLineInfo );
-    friend SvStream&    WriteImplLineInfo( SvStream& rOStm, const ImplLineInfo& rImplLineInfo );
 };
 
 
 class VCL_DLLPUBLIC LineInfo
 {
-private:
-
-    ImplLineInfo*   mpImplLineInfo;
-
-    SAL_DLLPRIVATE void ImplMakeUnique();
-
 public:
-
                     LineInfo( LineStyle eLineStyle = LINE_SOLID, long nWidth = 0L );
                     LineInfo( const LineInfo& rLineInfo );
                     ~LineInfo();
@@ -113,6 +102,9 @@ public:
     void applyToB2DPolyPolygon(
         basegfx::B2DPolyPolygon& io_rLinePolyPolygon,
         basegfx::B2DPolyPolygon& o_rFillPolyPolygon) const;
+
+private:
+    o3tl::cow_wrapper< ImplLineInfo >          mpImplLineInfo;
 };
 
 #endif // INCLUDED_VCL_LINEINFO_HXX
diff --git a/vcl/source/gdi/lineinfo.cxx b/vcl/source/gdi/lineinfo.cxx
index fc5878399235..2b71a285e294 100644
--- a/vcl/source/gdi/lineinfo.cxx
+++ b/vcl/source/gdi/lineinfo.cxx
@@ -28,7 +28,6 @@
 
 
 ImplLineInfo::ImplLineInfo() :
-    mnRefCount  ( 1 ),
     meStyle     ( LINE_SOLID ),
     mnWidth     ( 0 ),
     mnDashCount ( 0 ),
@@ -42,7 +41,6 @@ ImplLineInfo::ImplLineInfo() :
 }
 
 ImplLineInfo::ImplLineInfo( const ImplLineInfo& rImplLineInfo ) :
-    mnRefCount  ( 1 ),
     meStyle     ( rImplLineInfo.meStyle ),
     mnWidth     ( rImplLineInfo.mnWidth ),
     mnDashCount ( rImplLineInfo.mnDashCount ),
@@ -69,94 +67,63 @@ inline bool ImplLineInfo::operator==( const ImplLineInfo& rB ) const
 }
 
 
-LineInfo::LineInfo( LineStyle eStyle, long nWidth )
+LineInfo::LineInfo( LineStyle eStyle, long nWidth ) : mpImplLineInfo()
 {
-    mpImplLineInfo = new ImplLineInfo;
     mpImplLineInfo->meStyle = eStyle;
     mpImplLineInfo->mnWidth = nWidth;
 }
 
-LineInfo::LineInfo( const LineInfo& rLineInfo )
+LineInfo::LineInfo( const LineInfo& rLineInfo ) : mpImplLineInfo(rLineInfo.mpImplLineInfo)
 {
-    mpImplLineInfo = rLineInfo.mpImplLineInfo;
-    mpImplLineInfo->mnRefCount++;
 }
 
 LineInfo::~LineInfo()
 {
-    if( !( --mpImplLineInfo->mnRefCount ) )
-        delete mpImplLineInfo;
 }
 
 LineInfo& LineInfo::operator=( const LineInfo& rLineInfo )
 {
-
-    rLineInfo.mpImplLineInfo->mnRefCount++;
-
-    if( !( --mpImplLineInfo->mnRefCount ) )
-        delete mpImplLineInfo;
-
     mpImplLineInfo = rLineInfo.mpImplLineInfo;
     return *this;
 }
 
 bool LineInfo::operator==( const LineInfo& rLineInfo ) const
 {
-
-    return( mpImplLineInfo == rLineInfo.mpImplLineInfo ||
-           *mpImplLineInfo == *rLineInfo.mpImplLineInfo );
-}
-
-void LineInfo::ImplMakeUnique()
-{
-    if( mpImplLineInfo->mnRefCount != 1 )
-    {
-        if( mpImplLineInfo->mnRefCount )
-            mpImplLineInfo->mnRefCount--;
-
-        mpImplLineInfo = new ImplLineInfo( *mpImplLineInfo );
-    }
+    return mpImplLineInfo == rLineInfo.mpImplLineInfo;
 }
 
 void LineInfo::SetStyle( LineStyle eStyle )
 {
-    ImplMakeUnique();
     mpImplLineInfo->meStyle = eStyle;
 }
 
 void LineInfo::SetWidth( long nWidth )
 {
-    ImplMakeUnique();
     mpImplLineInfo->mnWidth = nWidth;
 }
 
 void LineInfo::SetDashCount( sal_uInt16 nDashCount )
 {
-    ImplMakeUnique();
     mpImplLineInfo->mnDashCount = nDashCount;
 }
 
 void LineInfo::SetDashLen( long nDashLen )
 {
-    ImplMakeUnique();
     mpImplLineInfo->mnDashLen = nDashLen;
 }
 
 void LineInfo::SetDotCount( sal_uInt16 nDotCount )
 {
-    ImplMakeUnique();
     mpImplLineInfo->mnDotCount = nDotCount;
 }
 
 void LineInfo::SetDotLen( long nDotLen )
 {
-    ImplMakeUnique();
     mpImplLineInfo->mnDotLen = nDotLen;
 }
 
 void LineInfo::SetDistance( long nDistance )
 {
-    ImplMakeUnique();
     mpImplLineInfo->mnDistance = nDistance;
 }
 
@@ -165,7 +132,6 @@ void LineInfo::SetLineJoin(basegfx::B2DLineJoin eLineJoin)
 
     if(eLineJoin != mpImplLineInfo->meLineJoin)
     {
-        ImplMakeUnique();
         mpImplLineInfo->meLineJoin = eLineJoin;
     }
 }
@@ -174,7 +140,6 @@ void LineInfo::SetLineCap(css::drawing::LineCap eLineCap)
 {
     if(eLineCap != mpImplLineInfo->meLineCap)
     {
-        ImplMakeUnique();
         mpImplLineInfo->meLineCap = eLineCap;
     }
 }
@@ -186,74 +151,66 @@ bool LineInfo::IsDefault() const
         && ( css::drawing::LineCap_BUTT == mpImplLineInfo->meLineCap));
 }
 
-SvStream& ReadImplLineInfo( SvStream& rIStm, ImplLineInfo& rImplLineInfo )
+SvStream& ReadLineInfo( SvStream& rIStm, LineInfo& rLineInfo )
 {
     VersionCompat   aCompat( rIStm, StreamMode::READ );
     sal_uInt16          nTmp16(0);
     sal_Int32       nTmp32(0);
 
-    rIStm.ReadUInt16( nTmp16 ); rImplLineInfo.meStyle = (LineStyle) nTmp16;
+    rIStm.ReadUInt16( nTmp16 ); rLineInfo.mpImplLineInfo->meStyle = (LineStyle) nTmp16;
     rIStm.ReadInt32( nTmp32 );
-    rImplLineInfo.mnWidth = nTmp32;
+    rLineInfo.mpImplLineInfo->mnWidth = nTmp32;
 
     if( aCompat.GetVersion() >= 2 )
     {
         // version 2
-        rIStm.ReadUInt16( rImplLineInfo.mnDashCount ).ReadInt32( nTmp32 );
-        rImplLineInfo.mnDashLen = nTmp32;
-        rIStm.ReadUInt16( rImplLineInfo.mnDotCount ).ReadInt32( nTmp32 );
-        rImplLineInfo.mnDotLen = nTmp32;
+        rIStm.ReadUInt16( rLineInfo.mpImplLineInfo->mnDashCount ).ReadInt32( nTmp32 );
+        rLineInfo.mpImplLineInfo->mnDashLen = nTmp32;
+        rIStm.ReadUInt16( rLineInfo.mpImplLineInfo->mnDotCount ).ReadInt32( nTmp32 );
+        rLineInfo.mpImplLineInfo->mnDotLen = nTmp32;
         rIStm.ReadInt32( nTmp32 );
-        rImplLineInfo.mnDistance = nTmp32;
+        rLineInfo.mpImplLineInfo->mnDistance = nTmp32;
     }
 
     if( aCompat.GetVersion() >= 3 )
     {
         // version 3
-        rIStm.ReadUInt16( nTmp16 ); rImplLineInfo.meLineJoin = (basegfx::B2DLineJoin) nTmp16;
+        rIStm.ReadUInt16( nTmp16 ); rLineInfo.mpImplLineInfo->meLineJoin = (basegfx::B2DLineJoin) nTmp16;
     }
 
     if( aCompat.GetVersion() >= 4 )
     {
         // version 4
-        rIStm.ReadUInt16( nTmp16 ); rImplLineInfo.meLineCap = (css::drawing::LineCap) nTmp16;
+        rIStm.ReadUInt16( nTmp16 ); rLineInfo.mpImplLineInfo->meLineCap = (css::drawing::LineCap) nTmp16;
     }
 
     return rIStm;
 }
 
-SvStream& WriteImplLineInfo( SvStream& rOStm, const ImplLineInfo& rImplLineInfo )
+SvStream& WriteLineInfo( SvStream& rOStm, const LineInfo& rLineInfo )
 {
     VersionCompat aCompat( rOStm, StreamMode::WRITE, 4 );
 
     // version 1
-    rOStm.WriteUInt16( rImplLineInfo.meStyle ).WriteInt32( rImplLineInfo.mnWidth );
+    rOStm.WriteUInt16( rLineInfo.mpImplLineInfo->meStyle )
+         .WriteInt32( rLineInfo.mpImplLineInfo->mnWidth );
 
     // since version2
-    rOStm.WriteUInt16( rImplLineInfo.mnDashCount ).WriteInt32( rImplLineInfo.mnDashLen );
-    rOStm.WriteUInt16( rImplLineInfo.mnDotCount ).WriteInt32( rImplLineInfo.mnDotLen );
-    rOStm.WriteInt32( rImplLineInfo.mnDistance );
+    rOStm.WriteUInt16( rLineInfo.mpImplLineInfo->mnDashCount )
+         .WriteInt32( rLineInfo.mpImplLineInfo->mnDashLen );
+    rOStm.WriteUInt16( rLineInfo.mpImplLineInfo->mnDotCount )
+         .WriteInt32( rLineInfo.mpImplLineInfo->mnDotLen );
+    rOStm.WriteInt32( rLineInfo.mpImplLineInfo->mnDistance );
 
     // since version3
-    rOStm.WriteUInt16( static_cast<sal_uInt16>(rImplLineInfo.meLineJoin) );
+    rOStm.WriteUInt16( static_cast<sal_uInt16>(rLineInfo.mpImplLineInfo->meLineJoin) );
 
     // since version4
-    rOStm.WriteUInt16( rImplLineInfo.meLineCap );
+    rOStm.WriteUInt16( rLineInfo.mpImplLineInfo->meLineCap );
 
     return rOStm;
 }
 
-SvStream& ReadLineInfo( SvStream& rIStm, LineInfo& rLineInfo )
-{
-    rLineInfo.ImplMakeUnique();
-    return ReadImplLineInfo( rIStm, *rLineInfo.mpImplLineInfo );
-}
-
-SvStream& WriteLineInfo( SvStream& rOStm, const LineInfo& rLineInfo )
-{
-    return WriteImplLineInfo( rOStm, *rLineInfo.mpImplLineInfo );
-}
-
 void LineInfo::applyToB2DPolyPolygon(
     basegfx::B2DPolyPolygon& io_rLinePolyPolygon,
     basegfx::B2DPolyPolygon& o_rFillPolyPolygon) const