core.git: sw/source

[Caolán McNamara (via logerrit)]

 sw/source/core/bastyp/calc.cxx |   18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

New commits:
commit 76d762fea934df31cd81768d467d5d40d439cb67
Author:     Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Sun May 5 16:50:03 2024 +0100
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Mon May 6 12:22:59 2024 +0200

    do the casts just once so this is readable
    
    Change-Id: Idb9eefa31c084604a134f28892b333c56f6d6fc2
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167161
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>

diff --git a/sw/source/core/bastyp/calc.cxx b/sw/source/core/bastyp/calc.cxx
index 1633090972d0..fbfc4b701457 100644
--- a/sw/source/core/bastyp/calc.cxx
+++ b/sw/source/core/bastyp/calc.cxx
@@ -119,22 +119,22 @@ const sal_Int32 coContFlags =
         & ~i18n::KParseTokens::IGNORE_LEADING_WS;
 
 extern "C" {
-static int OperatorCompare( const void *pFirst, const void *pSecond)
+static int OperatorCompare(const void *pA, const void *pB)
 {
+    const CalcOp *pFirst = static_cast<const CalcOp*>(pA);
+    const CalcOp *pSecond = static_cast<const CalcOp*>(pB);
+
     int nRet = 0;
-    if( CALC_NAME == static_cast<const CalcOp*>(pFirst)->eOp )
+    if( CALC_NAME == pFirst->eOp )
     {
-        nRet = static_cast<const CalcOp*>(pFirst)->aName.compareTo(
-                static_cast<const CalcOp*>(pSecond)->aName );
+        nRet = pFirst->aName.compareTo(pSecond->aName );
     }
     else
     {
-        if( CALC_NAME == static_cast<const CalcOp*>(pSecond)->eOp )
-            nRet = -1 * static_cast<const CalcOp*>(pSecond)->aName.compareTo(
-                        static_cast<const CalcOp*>(pFirst)->aName );
+        if( CALC_NAME == pSecond->eOp )
+            nRet = -1 * pSecond->aName.compareTo(pFirst->aName);
         else
-            nRet = static_cast<const CalcOp*>(pFirst)->aName.compareTo(
-                     static_cast<const CalcOp*>(pSecond)->aName );
+            nRet = pFirst->aName.compareTo(pSecond->aName);
     }
     return nRet;
 }