[Libreoffice-commits] core.git: vcl/win

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Sat Oct 27 09:43:20 UTC 2018


 vcl/win/gdi/salfont.cxx |    2 --
 1 file changed, 2 deletions(-)

New commits:
commit 6ef1eac70343d7867e24fa4830081640b0e20351
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Sat Oct 27 09:35:07 2018 +0200
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Sat Oct 27 11:43:01 2018 +0200

    tdf#120703 (PVS): V519 variable is assigned values twice successively
    
    V519 The 'xFontCharMap' variable is assigned values twice successively.
         Perhaps this is a mistake. Check lines: 1673, 1674.
    
    Change-Id: I008c268bd2440e489d4cd50f94b4f04129589448
    Reviewed-on: https://gerrit.libreoffice.org/62408
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>
    Tested-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/vcl/win/gdi/salfont.cxx b/vcl/win/gdi/salfont.cxx
index b5a0efbf047e..82c69c94a2c7 100644
--- a/vcl/win/gdi/salfont.cxx
+++ b/vcl/win/gdi/salfont.cxx
@@ -1658,8 +1658,6 @@ bool WinSalGraphics::CreateFontSubset( const OUString& rToFile,
     if( aRawCffData.get() )
     {
         pWinFontData->UpdateFromHDC( getHDC() );
-        FontCharMapRef xFontCharMap = pWinFontData->GetFontCharMap();
-        xFontCharMap = nullptr;
 
         // provide a font subset from the CFF-table
         FILE* pOutFile = fopen( aToFile.getStr(), "wb" );


More information about the Libreoffice-commits mailing list