sw/source/core/unocore/unochart.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit a37018f15d92a5835a93684831111c0aeb4a1654
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Sun Oct 20 11:21:58 2024 +0200
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Sun Oct 20 17:04:54 2024 +0200

    tdf#163486: PVS: fix std:swap arguments
    
    Since
    commit 81c4be45829826ae73ac14e1cb862e38dd73344d
    Author: Caolán McNamara <caolan.mcnam...@collabora.com>
    Date:   Sat Dec 16 20:18:13 2023 +0000
    
        cid#1545561 std::swap COPY_INSTEAD_OF_MOVE
    
    V549    The first argument of 'swap' function is equal to the second 
argument.
    
    Change-Id: Ieeb06744afe4ea09b986c78e8b25a5b5fc87d434
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175240
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>

diff --git a/sw/source/core/unocore/unochart.cxx 
b/sw/source/core/unocore/unochart.cxx
index 7886528cc0d3..6516794113f3 100644
--- a/sw/source/core/unocore/unochart.cxx
+++ b/sw/source/core/unocore/unochart.cxx
@@ -300,7 +300,7 @@ static bool GetTableAndCellsFromRangeRep(
             // need to switch start and end cell ?
             // (does not check for normalization here)
             if (bSortStartEndCells && 1 == sw_CompareCellsByColFirst( 
aStartCell, aEndCell ))
-                std::swap(aStartCell, aStartCell);
+                std::swap(aStartCell, aEndCell);
         }
         else    // a single cell like in "Table1.B3"
         {

Reply via email to