[Libreoffice-commits] .: 6 commits - sc/inc sc/source

Christina Rossmanith crossmanith at kemper.freedesktop.org
Sat Jun 4 23:49:22 PDT 2011


 sc/inc/rangelst.hxx                 |    7 --
 sc/source/core/tool/rangelst.cxx    |    7 --
 sc/source/core/tool/rangenam.cxx    |    7 --
 sc/source/filter/excel/excform.cxx  |  104 ++++++++++++++++--------------------
 sc/source/filter/excel/excform8.cxx |   54 ++++++------------
 sc/source/filter/inc/formel.hxx     |   20 ++----
 6 files changed, 81 insertions(+), 118 deletions(-)

New commits:
commit 174ce9bca6eec5034510dd9ba62d38661b5d13f9
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date:   Thu May 26 16:35:04 2011 +0200

    Translated DBG_WARNING messages

diff --git a/sc/source/filter/excel/excform.cxx b/sc/source/filter/excel/excform.cxx
index d575bc4..5e1f53d 100644
--- a/sc/source/filter/excel/excform.cxx
+++ b/sc/source/filter/excel/excform.cxx
@@ -400,9 +400,9 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                     case EXC_BIFF3:
                     case EXC_BIFF4: aIn.Ignore( 10 );   break;
                     case EXC_BIFF5:
-                        DBG_WARNING( "-ExcelToSc::Convert(): 0x1A gibt's nicht in Biff5!" );
+                        DBG_WARNING( "-ExcelToSc::Convert(): 0x1A does not exist in Biff5!" );
                     default:
-                        DBG_WARNING( "-ExcelToSc::Convert(): Ein wenig vergesslich, was?" );
+                        DBG_WARNING( "-ExcelToSc::Convert(): A little oblivious?" );
                 }
                 break;
             case 0x1B: // End External Reference				[330    ]
@@ -412,9 +412,9 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                     case EXC_BIFF3:
                     case EXC_BIFF4: aIn.Ignore( 4 );    break;
                     case EXC_BIFF5:
-                        DBG_WARNING( "-ExcelToSc::Convert(): 0x1B gibt's nicht in Biff5!" );
+                        DBG_WARNING( "-ExcelToSc::Convert(): 0x1B does not exist in Biff5!" );
                     default:
-                        DBG_WARNING( "-ExcelToSc::Convert(): Ein wenig vergesslich, was?" );
+                        DBG_WARNING( "-ExcelToSc::Convert(): A little oblivious?" );
                 }
                 break;
             case 0x1C: // Error Value							[314 266]
@@ -994,8 +994,8 @@ ConvErr ExcelToSc::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sal
                     case EXC_BIFF2: nIgnore = 7;    break;
                     case EXC_BIFF3:
                     case EXC_BIFF4: nIgnore = 10;   break;
-                    case EXC_BIFF5: DBG_WARNING( "-ExcelToSc::Convert(): 0x1A gibt's nicht in Biff5!" );
-                    default:        DBG_WARNING( "-ExcelToSc::Convert(): Ein wenig vergesslich, was?" );
+                    case EXC_BIFF5: DBG_WARNING( "-ExcelToSc::Convert(): 0x1A does not exist in Biff5!" );
+                    default:        DBG_WARNING( "-ExcelToSc::Convert(): A little oblivious?" );
                 }
                 break;
             case 0x1B: // End External Reference				[330    ]
@@ -1004,8 +1004,8 @@ ConvErr ExcelToSc::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sal
                     case EXC_BIFF2: nIgnore = 3;        break;
                     case EXC_BIFF3:
                     case EXC_BIFF4: nIgnore = 4;        break;
-                    case EXC_BIFF5: DBG_WARNING( "-ExcelToSc::Convert(): 0x1B gibt's nicht in Biff5!" );
-                    default:        DBG_WARNING( "-ExcelToSc::Convert(): Ein wenig vergesslich, was?" );
+                    case EXC_BIFF5: DBG_WARNING( "-ExcelToSc::Convert(): 0x1B does not exist in Biff5!" );
+                    default:        DBG_WARNING( "-ExcelToSc::Convert(): A little oblivious?" );
                 }
                 break;
             case 0x1C: // Error Value							[314 266]
commit 372130b13689f515c39e5e139967bc2e82dc4802
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date:   Thu May 26 16:34:43 2011 +0200

    __cdecl -> SAL_CALL

diff --git a/sc/inc/rangelst.hxx b/sc/inc/rangelst.hxx
index 6465416..ad5d4a7 100644
--- a/sc/inc/rangelst.hxx
+++ b/sc/inc/rangelst.hxx
@@ -129,11 +129,8 @@ private:
 };
 SV_DECL_IMPL_REF( ScRangePairList );
 
-extern "C" int
-#ifdef WNT
-__cdecl
-#endif
-ScRangePairList_QsortNameCompare( const void*, const void* );
+extern "C"
+int SAL_CALL ScRangePairList_QsortNameCompare( const void*, const void* );
 
 
 #endif
diff --git a/sc/source/core/tool/rangelst.cxx b/sc/source/core/tool/rangelst.cxx
index 85256fa..f83b751 100644
--- a/sc/source/core/tool/rangelst.cxx
+++ b/sc/source/core/tool/rangelst.cxx
@@ -696,11 +696,8 @@ struct ScRangePairNameSort
 };
 
 //-----------------------------------------------------------------------------
-extern "C" int
-#ifdef WNT
-__cdecl
-#endif
-ScRangePairList_QsortNameCompare( const void* p1, const void* p2 )
+extern "C"
+int SAL_CALL ScRangePairList_QsortNameCompare( const void* p1, const void* p2 )
 {
     const ScRangePairNameSort* ps1 = (const ScRangePairNameSort*)p1;
     const ScRangePairNameSort* ps2 = (const ScRangePairNameSort*)p2;
diff --git a/sc/source/core/tool/rangenam.cxx b/sc/source/core/tool/rangenam.cxx
index ea08c19..691eb01 100644
--- a/sc/source/core/tool/rangenam.cxx
+++ b/sc/source/core/tool/rangenam.cxx
@@ -646,11 +646,8 @@ void ScRangeData::ValidateTabRefs()
 }
 
 
-extern "C" int
-#ifdef WNT
-__cdecl
-#endif
-ScRangeData_QsortNameCompare( const void* p1, const void* p2 )
+extern "C"
+int SAL_CALL ScRangeData_QsortNameCompare( const void* p1, const void* p2 )
 {
     return (int) ScGlobal::GetCollator()->compareString(
             (*(const ScRangeData**)p1)->GetName(),
commit ef578f44a6ffb5a4e9afb07141b5338b21b4280d
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date:   Sun May 15 20:28:05 2011 +0200

    Translated comments

diff --git a/sc/source/filter/inc/formel.hxx b/sc/source/filter/inc/formel.hxx
index 9c902c6..0cb0c3a 100644
--- a/sc/source/filter/inc/formel.hxx
+++ b/sc/source/filter/inc/formel.hxx
@@ -50,10 +50,10 @@ struct ScComplexRefData;
 enum ConvErr
 {
     ConvOK = 0,
-    ConvErrNi,		// nicht implemntierter/unbekannter Opcode aufgetreten
-    ConvErrNoMem,	// Fehler beim Speicheranfordern
-    ConvErrExternal,// Add-Ins aus Excel werden nicht umgesetzt
-    ConvErrCount	// Nicht alle Bytes der Formel 'erwischt'
+    ConvErrNi,		// unimplemented/unknown opcode occurred
+    ConvErrNoMem,	// alloc error
+    ConvErrExternal,// excel add-ins are not converted
+    ConvErrCount	// did not get all bytes of formula
 };
 
 enum FORMULA_TYPE
@@ -87,12 +87,12 @@ public:
 class ConverterBase
 {
 protected:
-    TokenPool			aPool;			// User Token + Predefined Token
+    TokenPool			aPool;			// user token + predefined token
     TokenStack			aStack;
     ScAddress           aEingPos;
     ConvErr             eStatus;
-    sal_Char*			pBuffer;		// Universal-Puffer
-    sal_uInt16				nBufferSize;	// ...und seine Groesse
+    sal_Char*			pBuffer;		// universal buffer
+    sal_uInt16				nBufferSize;	// ...and its size
 
                         ConverterBase( sal_uInt16 nNewBuffer );
     virtual				~ConverterBase();
commit 8ad640da24370d7f52082e11911e4d3e5274bb6e
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date:   Sun May 15 13:04:15 2011 +0200

    Removed unused methods from ConvertBase

diff --git a/sc/source/filter/inc/formel.hxx b/sc/source/filter/inc/formel.hxx
index 94c56c5..9c902c6 100644
--- a/sc/source/filter/inc/formel.hxx
+++ b/sc/source/filter/inc/formel.hxx
@@ -98,12 +98,6 @@ protected:
     virtual				~ConverterBase();
 
     void				Reset();
-
-public:
-    inline SCCOL		GetEingabeCol( void ) const	{ return aEingPos.Col(); }
-    inline SCROW		GetEingabeRow( void ) const	{ return aEingPos.Row(); }
-    inline SCTAB		GetEingabeTab( void ) const	{ return aEingPos.Tab(); }
-    inline ScAddress	GetEingPos( void ) const	{ return aEingPos; }
 };
 
 
commit ce621b9389842df677983af6b47f495cabf15ce4
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date:   Sat May 14 21:03:22 2011 +0200

    Translated comments

diff --git a/sc/source/filter/excel/excform.cxx b/sc/source/filter/excel/excform.cxx
index 698b48d..d575bc4 100644
--- a/sc/source/filter/excel/excform.cxx
+++ b/sc/source/filter/excel/excform.cxx
@@ -121,7 +121,7 @@ void ImportExcel::Formula( const XclAddress& rXclPos,
     ScAddress aScPos( ScAddress::UNINITIALIZED );
     if( GetAddressConverter().ConvertAddress( aScPos, rXclPos, GetCurrScTab(), true ) )
     {
-        // jetzt steht Lesemarke auf Formel, Laenge in nFormLen
+        // Formula will be read next, length in nFormLen
         const ScTokenArray* pErgebnis = 0;
         sal_Bool				bConvert;
 
@@ -238,7 +238,7 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
         aSRD.InitFlags();
         aCRD.InitFlags();
 
-        switch( nOp )	//								Buch Seite:
+        switch( nOp )	//								book page:
         {			//										SDK4 SDK5
             case 0x01: // Array Formula							[325    ]
                        // Array Formula or Shared Formula		[    277]
@@ -258,7 +258,7 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                 aPool >> aStack;
                 break;
             case 0x04: // Subtraction							[313 264]
-                // SECOMD-TOP minus TOP
+                // SECOND-TOP minus TOP
                 aStack >> nMerk0;
                 aPool << aStack << ocSub << nMerk0;
                 aPool >> aStack;
@@ -274,7 +274,7 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                 aPool << aStack << ocDiv << nMerk0;
                 aPool >> aStack;
                 break;
-            case 0x07: // Exponetiation							[313 265]
+            case 0x07: // Exponentiation							[313 265]
                 // raise SECOND-TOP to power of TOP
                 aStack >> nMerk0;
                 aPool << aStack << ocPow << nMerk0;
@@ -305,19 +305,19 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                 aPool >> aStack;
                 break;
             case 0x0C: // Greater Than or Equal					[313 265]
-                // SECOND-TOP == TOP
+                // SECOND-TOP >= TOP
                 aStack >> nMerk0;
                 aPool << aStack << ocGreaterEqual << nMerk0;
                 aPool >> aStack;
                 break;
             case 0x0D: // Greater Than							[313 265]
-                // SECOND-TOP == TOP
+                // SECOND-TOP > TOP
                 aStack >> nMerk0;
                 aPool << aStack << ocGreater << nMerk0;
                 aPool >> aStack;
                 break;
             case 0x0E: // Not Equal								[313 265]
-                // SECOND-TOP == TOP
+                // SECOND-TOP != TOP
                 aStack >> nMerk0;
                 aPool << aStack << ocNotEqual << nMerk0;
                 aPool >> aStack;
@@ -328,7 +328,7 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                 aPool >> aStack;
                 break;
             case 0x10: // Union									[314 265]
-                // ocSep behelfsweise statt 'ocUnion'
+                // ocSep instead of 'ocUnion'
                 aStack >> nMerk0;
                 aPool << aStack << ocSep << nMerk0;
                     // doesn't fit exactly, but is more Excel-like
@@ -385,7 +385,7 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                 }
 
                 if( nOpt & 0x04 )
-                {// nFakt -> Bytes oder Words ueberlesen	AttrChoose
+                {// nFakt -> skip bytes or words	AttrChoose
                     nData++;
                     aIn.Ignore( nData * nFakt );
                 }
@@ -516,7 +516,7 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                     case EXC_BIFF5: aIn.Ignore( 12 );   break;
                     default:
                         OSL_FAIL(
-                        "-ExcelToSc::Convert(): Ein wenig vergesslich, was?" );
+                        "-ExcelToSc::Convert(): A little oblivious?" );
                 }
                 const XclImpName* pName = GetNameManager().GetName( nUINT16 );
                 if(pName && !pName->GetScRangeData())
@@ -721,22 +721,22 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                 aIn >> nTabFirst >> nTabLast >> nRow >> nCol;
 
                 if( nExtSheet >= 0 )
-                {	// von extern
+                {	// from external
                     if( rR.pExtSheetBuff->GetScTabIndex( nExtSheet, nTabLast ) )
                     {
                         nTabFirst = nTabLast;
-                        nExtSheet = 0;		// gefunden
+                        nExtSheet = 0;		// found
                     }
                     else
                     {
                         aPool << ocBad;
                         aPool >> aStack;
-                        nExtSheet = 1;		// verhindert Erzeugung einer SingleRef
+                        nExtSheet = 1;		// don't create a SingleRef
                     }
                 }
 
                 if( nExtSheet <= 0 )
-                {	// in aktuellem Workbook
+                {	// in current Workbook
                     aSRD.nTab = static_cast<SCTAB>(nTabFirst);
                     aSRD.SetFlag3D( sal_True );
                     aSRD.SetTabRel( false );
@@ -790,19 +790,19 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                     if( rR.pExtSheetBuff->GetScTabIndex( nExtSheet, nTabLast ) )
                     {
                         nTabFirst = nTabLast;
-                        nExtSheet = 0;		// gefunden
+                        nExtSheet = 0;		// found
                     }
                     else
                     {
                         aPool << ocBad;
                         aPool >> aStack;
-                        nExtSheet = 1;		// verhindert Erzeugung einer CompleteRef
+                        nExtSheet = 1;		// don't create a CompleteRef
                     }
                 }
 
                 if( nExtSheet <= 0 )
-                {// in aktuellem Workbook
-                    // erster Teil des Bereichs
+                {// in current Workbook
+                    // first part of range
                     ScSingleRefData&	rR1 = aCRD.Ref1;
                     ScSingleRefData&	rR2 = aCRD.Ref2;
 
@@ -838,7 +838,7 @@ ConvErr ExcelToSc::Convert( const ScTokenArray*& pErgebnis, XclImpStream& aIn, s
                         rR2.SetTabDeleted( sal_True );
 
                     aStack << aPool.Store( aCRD );
-                }//ENDE in aktuellem Workbook
+                }//END in current Workbook
             }
                 break;
             default: bError = sal_True;
@@ -928,7 +928,7 @@ ConvErr ExcelToSc::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sal
         aSRD.InitFlags();
         aCRD.InitFlags();
 
-        switch( nOp )	//								Buch Seite:
+        switch( nOp )	//								book page:
         {			//										SDK4 SDK5
             case 0x01: // Array Formula							[325    ]
                        // Array Formula or Shared Formula		[    277]
@@ -982,7 +982,7 @@ ConvErr ExcelToSc::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sal
                 }
 
                 if( nOpt & 0x04 )
-                {// nFakt -> Bytes oder Words ueberlesen	AttrChoose
+                {// nFakt -> skip bytes or words 	AttrChoose
                     nData++;
                     aIn.Ignore( nData * nFakt );
                 }
@@ -1042,7 +1042,7 @@ ConvErr ExcelToSc::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sal
                     case EXC_BIFF3:
                     case EXC_BIFF4: nIgnore = 10;   break;
                     case EXC_BIFF5: nIgnore = 14;   break;
-                    default:        OSL_FAIL( "-ExcelToSc::Convert(): Ein wenig vergesslich, was?" );
+                    default:        OSL_FAIL( "-ExcelToSc::Convert(): A little oblivious?" );
                 }
                 break;
             case 0x44:
@@ -1184,23 +1184,23 @@ ConvErr ExcelToSc::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sal
                 aIn >> nTabFirst >> nTabLast >> nRow >> nCol;
 
                 if( nExtSheet >= 0 )
-                    // von extern
+                    // from external
                 {
                     if( rR.pExtSheetBuff->GetScTabIndex( nExtSheet, nTabLast ) )
                     {
                         nTabFirst = nTabLast;
-                        nExtSheet = 0;		// gefunden
+                        nExtSheet = 0;		// found
                     }
                     else
                     {
                         aPool << ocBad;
                         aPool >> aStack;
-                        nExtSheet = 1;		// verhindert Erzeugung einer SingleRef
+                        nExtSheet = 1;		// don't create a SingleRef
                     }
                 }
 
                 if( nExtSheet <= 0 )
-                {// in aktuellem Workbook
+                {// in current Workbook
                     sal_Bool b3D = ( static_cast<SCTAB>(nTabFirst) != aEingPos.Tab() ) || bRangeName;
                     aSRD.nTab = static_cast<SCTAB>(nTabFirst);
                     aSRD.SetFlag3D( b3D );
@@ -1239,24 +1239,24 @@ ConvErr ExcelToSc::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sal
                     >> nColFirst >> nColLast;
 
                 if( nExtSheet >= 0 )
-                    // von extern
+                    // from external
                 {
                     if( rR.pExtSheetBuff->GetScTabIndex( nExtSheet, nTabLast ) )
                     {
                         nTabFirst = nTabLast;
-                        nExtSheet = 0;		// gefunden
+                        nExtSheet = 0;		// found
                     }
                     else
                     {
                         aPool << ocBad;
                         aPool >> aStack;
-                        nExtSheet = 1;		// verhindert Erzeugung einer CompleteRef
+                        nExtSheet = 1;		// don't create a CompleteRef
                     }
                 }
 
                 if( nExtSheet <= 0 )
-                {// in aktuellem Workbook
-                    // erster Teil des Bereichs
+                {// in current Workbook
+                    // first part of range
                     ScSingleRefData	&rR1 = aCRD.Ref1;
                     ScSingleRefData	&rR2 = aCRD.Ref2;
 
@@ -1276,7 +1276,7 @@ ConvErr ExcelToSc::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sal
                         SetComplRow( aCRD );
 
                     rRangeList.Append( aCRD, nTab );
-                }//ENDE in aktuellem Workbook
+                }//END in current Workbook
             }
                 break;
             case 0x5C:
@@ -1396,7 +1396,6 @@ sal_Bool ExcelToSc::GetAbsRefs( ScRangeList& rRangeList, XclImpStream& rStrm, sa
                 goto _common;
     _common:
                 // do not check abs/rel flags, linked controls have set them!
-//               if( !(( nCol1 & 0xC000 ) || ( nCol2 & 0xC000 )) )
                 {
                     ScRange aScRange;
                     nRow1 &= 0x3FFF;
@@ -1532,7 +1531,7 @@ void ExcelToSc::DoMulArgs( DefTokenId eId, sal_uInt8 nAnz )
 
     if( eId == ocCeil || eId == ocFloor )
     {
-        aStack << aPool.Store( 1.0 );	// default, da in Excel nicht vorhanden
+        aStack << aPool.Store( 1.0 );	// default, because not present in Excel
         nAnz++;
     }
 
@@ -1570,11 +1569,11 @@ void ExcelToSc::DoMulArgs( DefTokenId eId, sal_uInt8 nAnz )
 
         sal_Int16 nLast = nAnz - 1;
 
-        // Funktionen, bei denen Parameter wegfallen muessen
+        // functions for which parameters have to be skipped
         if( eId == ocPercentrank && nAnz == 3 )
-            nSkipEnd = 0;		// letzten Parameter bei Bedarf weglassen
+            nSkipEnd = 0;		// skip last parameter if necessary
 
-        // Joost-Spezialfaelle
+        // Joost special cases
         else if( eId == ocIf )
         {
             sal_uInt16			nNullParam = 0;
@@ -1626,11 +1625,11 @@ void ExcelToSc::ExcRelToScRel( sal_uInt16 nRow, sal_uInt8 nCol, ScSingleRefData
         if( nRow & 0x8000 )
         {//															rel Row
             rSRD.SetRowRel( sal_True );
-            if( nRow & 0x2000 )	// Bit 13 gesetzt?
-                //												-> Row negativ
+            if( nRow & 0x2000 )	// Bit 13 set?
+                //												-> Row negative
                 rSRD.nRelRow = static_cast<SCsROW>(static_cast<sal_Int16>(nRow | 0xC000));
             else
-                //												-> Row positiv
+                //												-> Row positive
                 rSRD.nRelRow = static_cast<SCsROW>(nRow & nRowMask);
         }
         else
@@ -1763,8 +1762,7 @@ void ExcelToSc::SetError( ScFormulaCell &rCell, const ConvErr eErr )
         case ConvErrNoMem:		nInd = errCodeOverflow; break;
         case ConvErrExternal:	nInd = errNoName; break;
         case ConvErrCount:		nInd = errCodeOverflow; break;
-        default:				nInd = errNoCode;	// hier fiel mir nichts
-                                                    //  Besseres ein...
+        default:				nInd = errNoCode;	// I had no better idea
     }
 
     rCell.SetErrCode( nInd );
@@ -1792,8 +1790,6 @@ void ExcelToSc::SetComplRow( ScComplexRefData &rCRD )
 
 void ExcelToSc::ReadExtensionArray( unsigned int n, XclImpStream& aIn )
 {
-    // printf( "inline array;\n" );
-
     sal_uInt8        nByte;
     sal_uInt16      nUINT16;
     double      fDouble;
@@ -1896,8 +1892,6 @@ void ExcelToSc::ReadExtensionArray( unsigned int n, XclImpStream& aIn )
 
 void ExcelToSc::ReadExtensionNlr( XclImpStream& aIn )
 {
-    // printf( "natural lang fmla;\n" );
-
     sal_uInt32 nFlags;
     aIn >> nFlags;
 
@@ -1907,8 +1901,6 @@ void ExcelToSc::ReadExtensionNlr( XclImpStream& aIn )
 
 void ExcelToSc::ReadExtensionMemArea( XclImpStream& aIn )
 {
-    // printf( "mem area;\n" );
-
     sal_uInt16 nCount;
     aIn >> nCount;
 
diff --git a/sc/source/filter/excel/excform8.cxx b/sc/source/filter/excel/excform8.cxx
index d00ce1b..11e34ea 100644
--- a/sc/source/filter/excel/excform8.cxx
+++ b/sc/source/filter/excel/excform8.cxx
@@ -192,7 +192,7 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
         aSRD.InitFlags();
         aCRD.InitFlags();
 
-        switch( nOp )	//								Buch Seite:
+        switch( nOp )	//								book page:
         {			//										SDK4 SDK5
             case 0x01: // Array Formula							[325    ]
                        // Array Formula or Shared Formula		[    277]
@@ -207,7 +207,7 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
                 aPool >> aStack;
                 break;
             case 0x04: // Subtraction							[313 264]
-                // SECOMD-TOP minus TOP
+                // SECOND-TOP minus TOP
                 aStack >> nMerk0;
                 aPool << aStack << ocSub << nMerk0;
                 aPool >> aStack;
@@ -223,7 +223,7 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
                 aPool << aStack << ocDiv << nMerk0;
                 aPool >> aStack;
                 break;
-            case 0x07: // Exponetiation							[313 265]
+            case 0x07: // Exponentiation							[313 265]
                 // raise SECOND-TOP to power of TOP
                 aStack >> nMerk0;
                 aPool << aStack << ocPow << nMerk0;
@@ -254,19 +254,19 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
                 aPool >> aStack;
                 break;
             case 0x0C: // Greater Than or Equal					[313 265]
-                // SECOND-TOP == TOP
+                // SECOND-TOP >= TOP
                 aStack >> nMerk0;
                 aPool << aStack << ocGreaterEqual << nMerk0;
                 aPool >> aStack;
                 break;
             case 0x0D: // Greater Than							[313 265]
-                // SECOND-TOP == TOP
+                // SECOND-TOP > TOP
                 aStack >> nMerk0;
                 aPool << aStack << ocGreater << nMerk0;
                 aPool >> aStack;
                 break;
             case 0x0E: // Not Equal								[313 265]
-                // SECOND-TOP == TOP
+                // SECOND-TOP != TOP
                 aStack >> nMerk0;
                 aPool << aStack << ocNotEqual << nMerk0;
                 aPool >> aStack;
@@ -277,7 +277,7 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
                 aPool >> aStack;
                 break;
             case 0x10: // Union									[314 265]
-                // ocSep behelfsweise statt 'ocUnion'
+                // ocSep instead of 'ocUnion'
                 aStack >> nMerk0;
                 aPool << aStack << ocSep << nMerk0;
                     // doesn't fit exactly, but is more Excel-like
@@ -392,7 +392,7 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
                 nFakt = 2;
 
                 if( nOpt & 0x04 )
-                {// nFakt -> Bytes oder Words ueberlesen	AttrChoose
+                {// nFakt -> skip bytes or words	AttrChoose
                     nData++;
                     aIn.Ignore( nData * nFakt );
                 }
@@ -576,22 +576,22 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
             case 0x66:
             case 0x26: // Constant Reference Subexpression		[321 271]
                 aExtensions.push_back( EXTENSION_MEMAREA );
-                aIn.Ignore( 6 );       // mehr steht da nicht!
+                aIn.Ignore( 6 );       // There isn't any more
                 break;
             case 0x47:
             case 0x67:
             case 0x27: // Erroneous Constant Reference Subexpr.	[322 272]
-                aIn.Ignore( 6 );   // mehr steht da nicht!
+                aIn.Ignore( 6 );   // There isn't any more
                 break;
             case 0x48:
             case 0x68:
             case 0x28: // Incomplete Constant Reference Subexpr.[331 281]
-                aIn.Ignore( 6 );   // mehr steht da nicht!
+                aIn.Ignore( 6 );   // There isn't any more
                 break;
             case 0x49:
             case 0x69:
             case 0x29: // Variable Reference Subexpression		[331 281]
-                aIn.Ignore( 2 );   // mehr steht da nicht!
+                aIn.Ignore( 2 );   // There isn't any more
                 break;
             case 0x4C:
             case 0x6C:
@@ -640,12 +640,12 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
             case 0x4E:
             case 0x6E:
             case 0x2E: // Reference Subexpression Within a Name	[332 282]
-                aIn.Ignore( 2 );   // mehr steht da nicht!
+                aIn.Ignore( 2 );   // There isn't any more
                 break;
             case 0x4F:
             case 0x6F:
             case 0x2F: // Incomplete Reference Subexpression...	[332 282]
-                aIn.Ignore( 2 );   // mehr steht da nicht!
+                aIn.Ignore( 2 );   // There isn't any more
                 break;
             case 0x58:
             case 0x78:
@@ -945,7 +945,7 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
 ConvErr ExcelToSc8::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sal_Size nFormulaLen,
                               SCsTAB nTab, const FORMULA_TYPE eFT )
 {
-    sal_uInt8					nOp, nLen;//, nByte;
+    sal_uInt8					nOp, nLen;
     sal_Bool					bError = false;
     const sal_Bool				bRangeName = eFT == FT_RangeName;
     const sal_Bool				bSharedFormula = eFT == FT_SharedFormula;
@@ -975,7 +975,7 @@ ConvErr ExcelToSc8::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sa
         aSRD.InitFlags();
         aCRD.InitFlags();
 
-        switch( nOp )	//								Buch Seite:
+        switch( nOp )	//								book page:
         {			//										SDK4 SDK5
             case 0x01: // Array Formula							[325    ]
                        // Array Formula or Shared Formula		[    277]
@@ -1019,7 +1019,7 @@ ConvErr ExcelToSc8::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sa
                 nFakt = 2;
 
                 if( nOpt & 0x04 )
-                {// nFakt -> Bytes oder Words ueberlesen	AttrChoose
+                {// nFakt -> skip bytes or words 	AttrChoose
                     nData++;
                     aIn.Ignore( nData * nFakt );
                 }
@@ -1111,12 +1111,12 @@ ConvErr ExcelToSc8::Convert( _ScRangeListTabs& rRangeList, XclImpStream& aIn, sa
             case 0x48:
             case 0x68:
             case 0x28: // Incomplete Constant Reference Subexpr.[331 281]
-                aIn.Ignore( 6 );   // mehr steht da nicht!
+                aIn.Ignore( 6 );   // There isn't any more
                 break;
             case 0x49:
             case 0x69:
             case 0x29: // Variable Reference Subexpression		[331 281]
-                aIn.Ignore( 2 );   // mehr steht da nicht!
+                aIn.Ignore( 2 );   // There isn't any more
                 break;
             case 0x4A:
             case 0x6A:
@@ -1649,7 +1649,7 @@ sal_Bool ExcelToSc8::GetAbsRefs( ScRangeList& r, XclImpStream& aIn, sal_Size nLe
                 sal_uInt8	nOpt;
                 aIn >> nOpt >> nData;
                 if( nOpt & 0x04 )
-                {// nFakt -> Bytes oder Words ueberlesen	AttrChoose
+                {// nFakt -> skip bytes or words 	AttrChoose
                     nData++;
                     nSeek = nData * 2;
             }
commit 349f92c9c3bf91b2c56fb2b69ad3f22d0247988d
Author: Christina Rossmanith <ChrRossmanith at web.de>
Date:   Sat May 14 20:35:55 2011 +0200

    Removed commented code

diff --git a/sc/source/filter/excel/excform8.cxx b/sc/source/filter/excel/excform8.cxx
index f26a987..d00ce1b 100644
--- a/sc/source/filter/excel/excform8.cxx
+++ b/sc/source/filter/excel/excform8.cxx
@@ -582,15 +582,11 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
             case 0x67:
             case 0x27: // Erroneous Constant Reference Subexpr.	[322 272]
                 aIn.Ignore( 6 );   // mehr steht da nicht!
-//               aPool << ocBad;
-//               aPool >> aStack;
                 break;
             case 0x48:
             case 0x68:
             case 0x28: // Incomplete Constant Reference Subexpr.[331 281]
                 aIn.Ignore( 6 );   // mehr steht da nicht!
-//               aPool << ocBad;
-//               aPool >> aStack;
                 break;
             case 0x49:
             case 0x69:
@@ -645,15 +641,11 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
             case 0x6E:
             case 0x2E: // Reference Subexpression Within a Name	[332 282]
                 aIn.Ignore( 2 );   // mehr steht da nicht!
-//               aPool << ocBad;
-//               aPool >> aStack;
                 break;
             case 0x4F:
             case 0x6F:
             case 0x2F: // Incomplete Reference Subexpression...	[332 282]
                 aIn.Ignore( 2 );   // mehr steht da nicht!
-//               aPool << ocBad;
-//               aPool >> aStack;
                 break;
             case 0x58:
             case 0x78:
@@ -769,7 +761,6 @@ ConvErr ExcelToSc8::Convert( const ScTokenArray*& rpTokArray, XclImpStream& aIn,
                 }
                 else
                 {
-                    //aStack << ocNoName;
                     aPool << ocBad;
                     aPool >> aStack;
                 }
@@ -1566,7 +1557,6 @@ sal_Bool ExcelToSc8::GetAbsRefs( ScRangeList& r, XclImpStream& aIn, sal_Size nLe
                 goto _common;
     _common:
                 // do not check abs/rel flags, linked controls have set them!
-//               if( !(( nCol1 & 0xC000 ) || ( nCol2 & 0xC000 )) )
                 {
                     ScRange aScRange;
                     nCol1 &= 0x3FFF;
@@ -1674,8 +1664,4 @@ sal_Bool ExcelToSc8::GetAbsRefs( ScRangeList& r, XclImpStream& aIn, sal_Size nLe
     return !r.empty();
 }
 
-
-
-
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */


More information about the Libreoffice-commits mailing list