[Libreoffice-commits] core.git: 6 commits - desktop/source editeng/source svl/source svtools/source ucb/source vcl/source

Caolán McNamara caolanm at redhat.com
Mon Dec 12 09:48:11 UTC 2016


 desktop/source/deployment/manager/dp_manager.cxx |    2 +-
 desktop/source/splash/splash.cxx                 |    2 +-
 editeng/source/uno/unoipset.cxx                  |    2 +-
 svl/source/items/itemset.cxx                     |    2 +-
 svtools/source/svrtf/rtfout.cxx                  |    2 +-
 ucb/source/core/FileAccess.cxx                   |   12 ++++++------
 vcl/source/edit/vclmedit.cxx                     |    2 +-
 7 files changed, 12 insertions(+), 12 deletions(-)

New commits:
commit 7803f6ba1903428a97a3b171260ec0145bf644c8
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Mon Dec 12 09:45:38 2016 +0000

    coverity#1397042 Unchecked return value
    
    Change-Id: Ibb24dea2464c2e0bba289f08bba053efe6fe57fc

diff --git a/desktop/source/deployment/manager/dp_manager.cxx b/desktop/source/deployment/manager/dp_manager.cxx
index cdb60f0..70a441d 100644
--- a/desktop/source/deployment/manager/dp_manager.cxx
+++ b/desktop/source/deployment/manager/dp_manager.cxx
@@ -711,7 +711,7 @@ Reference<deployment::XPackage> PackageManagerImpl::addPackage(
 
     try {
         ::ucbhelper::Content sourceContent;
-        create_ucb_content( &sourceContent, url, xCmdEnv ); // throws exc
+        (void)create_ucb_content( &sourceContent, url, xCmdEnv ); // throws exc
         const OUString title( StrTitle::getTitle( sourceContent ) );
         const OUString title_enc( ::rtl::Uri::encode(
                                       title, rtl_UriCharClassPchar,
commit c43c50723d6e395184214b56aa9964125622d9eb
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Mon Dec 12 09:44:21 2016 +0000

    coverity#1397041 Unchecked return value
    
    Change-Id: Ia8c601475bea142faad9c1f1e9b797d0ebc13679

diff --git a/ucb/source/core/FileAccess.cxx b/ucb/source/core/FileAccess.cxx
index 4c2859d..545785c 100644
--- a/ucb/source/core/FileAccess.cxx
+++ b/ucb/source/core/FileAccess.cxx
@@ -276,12 +276,12 @@ void OFileAccess::transferImpl( const OUString& rSource,
 
     try
     {
-        aDestPath.transferContent( aSrc,
-                                   bMoveData
-                                    ? ucbhelper::InsertOperation_MOVE
-                                    : ucbhelper::InsertOperation_COPY,
-                                   aName,
-                                   css::ucb::NameClash::OVERWRITE );
+        (void)aDestPath.transferContent(aSrc,
+                                        bMoveData
+                                         ? ucbhelper::InsertOperation_MOVE
+                                         : ucbhelper::InsertOperation_COPY,
+                                        aName,
+                                        css::ucb::NameClash::OVERWRITE);
     }
     catch ( css::ucb::CommandFailedException const & )
     {
commit 510f497f05cbf65e2ee425a8af43f47406dbfcc5
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Mon Dec 12 09:41:17 2016 +0000

    coverity#1397045 Unchecked return value
    
    and
    
    coverity#1397044 Unchecked return value
    
    Change-Id: I193bd676e417c35450efa91cb5773c78c998b63f

diff --git a/editeng/source/uno/unoipset.cxx b/editeng/source/uno/unoipset.cxx
index 60bac11..d02c64c 100644
--- a/editeng/source/uno/unoipset.cxx
+++ b/editeng/source/uno/unoipset.cxx
@@ -99,7 +99,7 @@ uno::Any SvxItemPropertySet::getPropertyValue( const SfxItemPropertySimpleEntry*
 
     const SfxPoolItem* pItem = nullptr;
     SfxItemPool* pPool = rSet.GetPool();
-    rSet.GetItemState( pMap->nWID, bSearchInParent, &pItem );
+    (void)rSet.GetItemState( pMap->nWID, bSearchInParent, &pItem );
     if( nullptr == pItem && pPool )
         pItem = &(pPool->GetDefaultItem( pMap->nWID ));
 
diff --git a/svl/source/items/itemset.cxx b/svl/source/items/itemset.cxx
index 4f6465e..9996e93 100644
--- a/svl/source/items/itemset.cxx
+++ b/svl/source/items/itemset.cxx
@@ -1283,7 +1283,7 @@ void SfxItemSet::MergeValues( const SfxItemSet& rSet )
         while( 0 != ( nWhich = aIter.NextWhich() ) )
         {
             const SfxPoolItem* pItem = nullptr;
-            rSet.GetItemState( nWhich, true, &pItem );
+            (void)rSet.GetItemState( nWhich, true, &pItem );
             if( !pItem )
             {
                 // Not set, so default
commit c5ba9161a8fb45f86c4d6cf030afa0a20b42cd3f
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Mon Dec 12 09:39:54 2016 +0000

    coverity#1397046 Unchecked return value
    
    Change-Id: I7f8988a7cf5f3d815a20cac249b3753d732994e1

diff --git a/svtools/source/svrtf/rtfout.cxx b/svtools/source/svrtf/rtfout.cxx
index cb722db..605fad7 100644
--- a/svtools/source/svrtf/rtfout.cxx
+++ b/svtools/source/svrtf/rtfout.cxx
@@ -141,7 +141,7 @@ SvStream& Out_Char(SvStream& rStream, sal_Unicode c,
                                             || (RTL_TEXTENCODING_UTF8==eDestEnc); // #i43933# do not export UTF-8 chars in RTF;
                     if (bWriteAsUnicode)
                     {
-                        sBuf.convertToString(&sConverted,
+                        (void)sBuf.convertToString(&sConverted,
                             eDestEnc, OUSTRING_TO_OSTRING_CVTFLAGS);
                     }
                     const sal_Int32 nLen = sConverted.getLength();
commit cc777d31c1508ef74551532b553374f5db87bca9
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Mon Dec 12 09:37:17 2016 +0000

    coverity#1397048 Unchecked return value
    
    Change-Id: Ie33ef6565cadcddb59c1d78a63aec666e8693f87

diff --git a/desktop/source/splash/splash.cxx b/desktop/source/splash/splash.cxx
index a7385ae..13c59c8 100644
--- a/desktop/source/splash/splash.cxx
+++ b/desktop/source/splash/splash.cxx
@@ -537,7 +537,7 @@ void SplashScreen::SetScreenBitmap(BitmapEx &rBitmap)
     if (Application::LoadBrandBitmap (aResBuf.makeStringAndClear().getStr(), rBitmap))
         return;
 
-    Application::LoadBrandBitmap ("intro", rBitmap);
+    (void)Application::LoadBrandBitmap ("intro", rBitmap);
 }
 
 void SplashScreen::determineProgressRatioValues(
commit d6b094bc8f23cbdad4da646320f3e6809fa42c5d
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Mon Dec 12 09:35:41 2016 +0000

    coverity#1397043 Unchecked return value
    
    Change-Id: I37c49aa568646e44c05f5feec537d344a2440b91

diff --git a/vcl/source/edit/vclmedit.cxx b/vcl/source/edit/vclmedit.cxx
index 0367839..8280878 100644
--- a/vcl/source/edit/vclmedit.cxx
+++ b/vcl/source/edit/vclmedit.cxx
@@ -696,7 +696,7 @@ bool ImpVclMEdit::HandleCommand( const CommandEvent& rCEvt )
          ( rCEvt.GetCommand() == CommandEventId::StartAutoScroll ) ||
          ( rCEvt.GetCommand() == CommandEventId::AutoScroll ) )
     {
-        mpTextWindow->HandleScrollCommand( rCEvt, mpHScrollBar, mpVScrollBar );
+        (void)mpTextWindow->HandleScrollCommand( rCEvt, mpHScrollBar, mpVScrollBar );
         bDone = true;
     }
     return bDone;


More information about the Libreoffice-commits mailing list