[PATCH] Calc sort: Use method GetSortKeyCount from SortParam

Albert Thuswaldner (via Code Review) gerrit at gerrit.libreoffice.org
Tue May 7 11:20:34 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3816

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/16/3816/1

Calc sort: Use method GetSortKeyCount from SortParam

Change-Id: I4500ba71abeba5fe8293cea22b10fd910e46059f
---
M sc/source/ui/dbgui/tpsort.cxx
1 file changed, 3 insertions(+), 3 deletions(-)



diff --git a/sc/source/ui/dbgui/tpsort.cxx b/sc/source/ui/dbgui/tpsort.cxx
index 4499600..1479313 100644
--- a/sc/source/ui/dbgui/tpsort.cxx
+++ b/sc/source/ui/dbgui/tpsort.cxx
@@ -169,13 +169,13 @@
     if ( aSortData.maKeyState[0].bDoSort )
     {
         // Make sure that the all sort keys are reset
-        for ( sal_uInt16 i=nSortKeyCount; i<aSortData.maKeyState.size(); i++ )
+        for ( sal_uInt16 i=nSortKeyCount; i<aSortData.GetSortKeyCount(); i++ )
         {
             maSortKeyCtrl.AddSortKey(i+1);
             maSortKeyItems[i].m_pLbSort->SetSelectHdl( LINK( this,
                                  ScTabPageSortFields, SelectHdl ) );
         }
-        nSortKeyCount = aSortData.maKeyState.size();
+        nSortKeyCount = aSortData.GetSortKeyCount();
         FillFieldLists(0);
 
         for ( sal_uInt16 i=0; i<nSortKeyCount; i++ )
@@ -265,7 +265,7 @@
         if ( nSortPos[i] == LISTBOX_ENTRY_NOTFOUND ) nSortPos[i] = 0;
     }
 
-    if( nSortKeyCount >= aNewSortData.maKeyState.size() )
+    if( nSortKeyCount >= aNewSortData.GetSortKeyCount() )
         aNewSortData.maKeyState.resize(nSortKeyCount);
 
     if ( nSortPos[0] > 0 )

-- 
To view, visit https://gerrit.libreoffice.org/3816
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4500ba71abeba5fe8293cea22b10fd910e46059f
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Albert Thuswaldner <albert.thuswaldner at gmail.com>



More information about the LibreOffice mailing list