basegfx/source/polygon/b2dpolygontools.cxx    |    1 -
 basegfx/source/polygon/b3dpolygontools.cxx    |    1 -
 sax/source/fastparser/fastparser.cxx          |    2 +-
 ucb/source/cacher/contentresultsetwrapper.cxx |    3 ---
 vcl/source/filter/ipict/ipict.cxx             |    2 +-
 vcl/source/outdev/bitmapex.cxx                |    2 --
 vcl/unx/gtk3/fpicker/SalGtkFilePicker.cxx     |    2 +-
 7 files changed, 3 insertions(+), 10 deletions(-)

New commits:
commit 363512d39ba1543b7e91af1d59dadb5378da7649
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Mon Mar 6 15:09:25 2023 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Mon Mar 6 16:48:15 2023 +0000

    clang-tidy dead-store
    
    Change-Id: I842114880c43dfcc342b6255b7d17befb905bccb
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148345
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/basegfx/source/polygon/b2dpolygontools.cxx 
b/basegfx/source/polygon/b2dpolygontools.cxx
index 03f5299ca505..900ab735a1e0 100644
--- a/basegfx/source/polygon/b2dpolygontools.cxx
+++ b/basegfx/source/polygon/b2dpolygontools.cxx
@@ -1255,7 +1255,6 @@ namespace basegfx::utils
                 // to enlarge these as needed
                 const double fFactor(fCandidateLength / fAllowedLength);
                 std::for_each(aDotDashArray.begin(), aDotDashArray.end(), 
[&fFactor](double &f){ f *= fFactor; });
-                fDotDashLength *= fFactor;
             }
 
             // prepare current edge's start
diff --git a/basegfx/source/polygon/b3dpolygontools.cxx 
b/basegfx/source/polygon/b3dpolygontools.cxx
index a1f65bef2f48..3b3779d7460a 100644
--- a/basegfx/source/polygon/b3dpolygontools.cxx
+++ b/basegfx/source/polygon/b3dpolygontools.cxx
@@ -219,7 +219,6 @@ namespace basegfx::utils
                 // to enlarge these as needed
                 const double fFactor(fCandidateLength / fAllowedLength);
                 std::for_each(aDotDashArray.begin(), aDotDashArray.end(), 
[&fFactor](double &f){ f *= fFactor; });
-                fDotDashLength *= fFactor;
             }
 
             // prepare current edge's start
diff --git a/sax/source/fastparser/fastparser.cxx 
b/sax/source/fastparser/fastparser.cxx
index aae3c1162d47..06e2fb1e3e92 100644
--- a/sax/source/fastparser/fastparser.cxx
+++ b/sax/source/fastparser/fastparser.cxx
@@ -1565,7 +1565,7 @@ static void NormalizeURI( OUString& rName )
     // try OASIS + W3 URI normalization
     bool bSuccess = NormalizeOasisURN( rName );
     if( ! bSuccess )
-        bSuccess = NormalizeW3URI( rName );
+        NormalizeW3URI( rName );
 }
 
 constexpr OUStringLiteral XML_URI_W3_PREFIX(u"http://www.w3.org/";);
diff --git a/ucb/source/cacher/contentresultsetwrapper.cxx 
b/ucb/source/cacher/contentresultsetwrapper.cxx
index f6e7c0058a65..72388222f12d 100644
--- a/ucb/source/cacher/contentresultsetwrapper.cxx
+++ b/ucb/source/cacher/contentresultsetwrapper.cxx
@@ -275,10 +275,7 @@ void SAL_CALL ContentResultSetWrapper::dispose()
     }
 
     if (isCleared)
-    {
         aGuard.lock();
-        isCleared = false;
-    }
     if( m_aDisposeEventListeners.getLength(aGuard) )
     {
         EventObject aEvt;
diff --git a/vcl/source/filter/ipict/ipict.cxx 
b/vcl/source/filter/ipict/ipict.cxx
index fcdef08e3592..2d9be6f8cd9c 100644
--- a/vcl/source/filter/ipict/ipict.cxx
+++ b/vcl/source/filter/ipict/ipict.cxx
@@ -783,7 +783,7 @@ sal_uInt64 PictReader::ReadPixMapEtc( BitmapEx &rBitmap, 
bool bBaseAddr, bool bC
     else
     {
         nRowBytes &= 0x3fff;
-        nPixelSize = nCmpCount = nCmpSize = 1;
+        nPixelSize = nCmpCount = 1;
         nDataSize += 10;
         aPalette.resize(2);
         aPalette[0] = Color(0xff, 0xff, 0xff);
diff --git a/vcl/source/outdev/bitmapex.cxx b/vcl/source/outdev/bitmapex.cxx
index 57851f943f51..9935a18bdee6 100644
--- a/vcl/source/outdev/bitmapex.cxx
+++ b/vcl/source/outdev/bitmapex.cxx
@@ -522,8 +522,6 @@ void OutputDevice::DrawTransformedBitmapEx(
             aAlpha.BlendWith( bitmapEx.GetAlphaMask());
         bitmapEx = BitmapEx( bitmapEx.GetBitmap(), aAlpha );
     }
-    if(rtl::math::approxEqual( fAlpha, 1.0 ))
-        fAlpha = 1.0; // avoid the need for approxEqual in backends
 
     // If the backend's implementation is known to not need any optimizations 
here, pass to it directly.
     // With most backends it's more performant to try to simplify to 
DrawBitmapEx() first.
diff --git a/vcl/unx/gtk3/fpicker/SalGtkFilePicker.cxx 
b/vcl/unx/gtk3/fpicker/SalGtkFilePicker.cxx
index 9afb55f0eb21..36278074c446 100644
--- a/vcl/unx/gtk3/fpicker/SalGtkFilePicker.cxx
+++ b/vcl/unx/gtk3/fpicker/SalGtkFilePicker.cxx
@@ -1677,7 +1677,7 @@ void SalGtkFilePicker::impl_initialize(GtkWidget* 
pParentWidget, sal_Int16 templ
     GtkFileChooserAction eAction = GTK_FILE_CHOOSER_ACTION_OPEN;
     OString sOpen = getOpenText();
     OString sSave = getSaveText();
-    const gchar *first_button_text = sOpen.getStr();
+    const gchar *first_button_text;
 
     SolarMutexGuard g;
 

Reply via email to