Do the same for the no-parameter variant of fillTable().

Change-Id: I698e70e7713697e2951c3e6ead4b4d2198d8a6cf

sc/source/core/data/dpcachetable.cxx

@@ -209,14 +209,19 @@ void ScDPCacheTable::fillTable(
 
 void ScDPCacheTable::fillTable()
 {
-   const SCROW  nRowCount = getRowSize();
-   const SCCOL  nColCount = (SCCOL) getColSize();
-   if ( nRowCount <= 0 || nColCount <= 0)
+   SCROW nRowCount = getRowSize();
+   SCCOL nColCount = getColSize();
+   if (nRowCount <= 0 || nColCount <= 0)
         return;
 
     maRowFlags.clear();
     maRowFlags.reserve(nRowCount);
 
+    for (SCROW nRow = 0; nRow < nRowCount; ++nRow)
+    {
+        maRowFlags.push_back(RowFlag());
+        maRowFlags.back().mbShowByFilter = true;
+    }
 
     // Initialize field entries container.
     maFieldEntries.clear();
@@ -227,28 +232,21 @@ void ScDPCacheTable::fillTable()
     {
         maFieldEntries.push_back( vector<SCROW>() );
         SCROW nMemCount = getCache()->GetDimMemberCount( nCol );
-        if ( nMemCount )
-        {
-            std::vector< SCROW > pAdded( nMemCount, -1 );
+        if (!nMemCount)
+            continue;
 
-            for (SCROW nRow = 0; nRow < nRowCount; ++nRow )
-            {
-                SCROW nIndex = getCache()->GetItemDataId( nCol, nRow, false );
-                SCROW nOrder = getOrder( nCol, nIndex );
+        std::vector<SCROW> aAdded(nMemCount, -1);
 
-                if ( nCol == 0 )
+        for (SCROW nRow = 0; nRow < nRowCount; ++nRow)
         {
-                     maRowFlags.push_back(RowFlag());
-                     maRowFlags.back().mbShowByFilter = true;
-                }
-
-                pAdded[nOrder] = nIndex;
+            SCROW nIndex = getCache()->GetItemDataId(nCol, nRow, false);
+            SCROW nOrder = getOrder(nCol, nIndex);
+            aAdded[nOrder] = nIndex;
         }
-            for ( SCROW nRow = 0; nRow < nMemCount; nRow++ )
+        for (SCROW nRow = 0; nRow < nMemCount; ++nRow)
         {
-                if ( pAdded[nRow] != -1 )
-                    maFieldEntries.back().push_back( pAdded[nRow] );
-            }
+            if (aAdded[nRow] != -1)
+                maFieldEntries.back().push_back(aAdded[nRow]);
         }
     }
 }