[Libreoffice-commits] core.git: 3 commits - desktop/source filter/source idl/source

[Caolán McNamara (via logerrit)]

 desktop/source/deployment/misc/lockfile.cxx |    2 +-
 filter/source/flash/swfwriter1.cxx          |    4 ----
 idl/source/prj/svidl.cxx                    |    4 ++--
 3 files changed, 3 insertions(+), 7 deletions(-)

New commits:
commit 99242d6ca22eaad663e1326e5eb402ded3521e24
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Thu Jul 18 10:41:39 2019 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Thu Jul 18 21:15:42 2019 +0200

    cid#1448337 Unchecked return value from library
    
    Change-Id: Ie43cc1888179340318ddefb7fa4ba5f0bc1dc896
    Reviewed-on: https://gerrit.libreoffice.org/75879
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/filter/source/flash/swfwriter1.cxx b/filter/source/flash/swfwriter1.cxx
index bab56d749bc6..7967468650c3 100644
--- a/filter/source/flash/swfwriter1.cxx
+++ b/filter/source/flash/swfwriter1.cxx
@@ -740,14 +740,10 @@ sal_uInt16 Writer::defineBitmap( const BitmapEx &bmpSource, sal_Int32 nJPEGQuali
         alpha_compressed_size = uLongf(width * height + static_cast<sal_uInt32>(raw_size/100) + 12);
         pAlphaCompressed.reset(new sal_uInt8[ compressed_size ]);
 
-#ifdef DBG_UTIL
         if(compress2(pAlphaCompressed.get(), &alpha_compressed_size, aAlphaData.data(), width * height, Z_BEST_COMPRESSION) != Z_OK)
         {
             SAL_WARN( "filter.flash", "compress2 failed!" ); ((void)0);
         }
-#else
-        compress2(pAlphaCompressed.get(), &alpha_compressed_size, aAlphaData.data(), width * height, Z_BEST_COMPRESSION);
-#endif
     }
 
     // clear these early for less peak memory usage
commit dcfc5ecf93203e697df2290fa045f6a2fc7e00cd
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Thu Jul 18 10:35:26 2019 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Thu Jul 18 21:15:24 2019 +0200

    cid#1448323 Unchecked return value
    
    Change-Id: I2d1d3d2ce7302fa895820981e0c15b59d491ddc4
    Reviewed-on: https://gerrit.libreoffice.org/75876
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/idl/source/prj/svidl.cxx b/idl/source/prj/svidl.cxx
index d76b16a66835..fcfebed79816 100644
--- a/idl/source/prj/svidl.cxx
+++ b/idl/source/prj/svidl.cxx
@@ -120,8 +120,8 @@ int main ( int argc, char ** argv)
     {
         osl::DirectoryItem aDI;
         osl::FileStatus fileStatus( osl_FileStatus_Mask_FileName );
-        osl::DirectoryItem::get( aCommand.aExportFile, aDI );
-        aDI.getFileStatus(fileStatus);
+        (void)osl::DirectoryItem::get( aCommand.aExportFile, aDI );
+        (void)aDI.getFileStatus(fileStatus);
         pDataBase->SetExportFile( fileStatus.getFileName() );
     }
 
commit 53d78a7410dadb62be23c209b35ca1422a896307
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Thu Jul 18 10:34:02 2019 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Thu Jul 18 21:15:07 2019 +0200

    cid#1448296 Unchecked return value
    
    Change-Id: I09268bc22f4fb34160975063e38ea60a68123400
    Reviewed-on: https://gerrit.libreoffice.org/75875
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/desktop/source/deployment/misc/lockfile.cxx b/desktop/source/deployment/misc/lockfile.cxx
index ff216ff7f9dc..0cda82e6b3c3 100644
--- a/desktop/source/deployment/misc/lockfile.cxx
+++ b/desktop/source/deployment/misc/lockfile.cxx
@@ -130,7 +130,7 @@ namespace desktop {
                 // remove file and create new
                 File::remove( m_aLockname );
                 File aFile(m_aLockname);
-                aFile.open( osl_File_OpenFlag_Create );
+                (void)aFile.open( osl_File_OpenFlag_Create );
                 aFile.close( );
                 syncToFile( );
                 m_bRemove = true;