Translate German comments, fix some ws

Change-Id: I3bc8d82c5689b8a2da1374e42dd70191e2fe8ef3
Reviewed-on: https://gerrit.libreoffice.org/11099Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: Thomas Arnhold <thomas@arnhold.org>

sc/source/core/data/attarray.cxx

@@ -889,15 +889,15 @@ static void lcl_MergeDeep( SfxItemSet& rMergeSet, const SfxItemSet& rSource )
                     rMergeSet.InvalidateItem( nId );
             }
         }
-        else if ( eOldState == SFX_ITEM_SET )               // Item gesetzt
+        else if ( eOldState == SFX_ITEM_SET ) // Item set
         {
             SfxItemState eNewState = rSource.GetItemState( nId, true, &pNewItem );
             if ( eNewState == SFX_ITEM_SET )
             {
-                if ( pNewItem != pOldItem )                 // beide gepuhlt
+                if ( pNewItem != pOldItem ) // Both pulled
                     rMergeSet.InvalidateItem( nId );
             }
-            else            // Default
+            else // Default
             {
                 if ( *pOldItem != rSource.GetPool()->GetDefaultItem(nId) )
                     rMergeSet.InvalidateItem( nId );
@@ -923,7 +923,6 @@ void ScAttrArray::MergePatternArea( SCROW nStartRow, SCROW nEndRow,
         do
         {
             // similar patterns must not be repeated
-
             const ScPatternAttr* pPattern = pData[nPos].pPattern;
             if ( pPattern != rState.pOld1 && pPattern != rState.pOld2 )
             {
@@ -1357,7 +1356,9 @@ bool ScAttrArray::IsMerged( SCROW nRow ) const
     return rItem.IsMerged();
 }
 
-// Area around any given summaries expand and adapt any MergeFlag (bRefresh)
+/**
+ * Area around any given summaries expand and adapt any MergeFlag (bRefresh)
+ */
 bool ScAttrArray::ExtendMerge( SCCOL nThisCol, SCROW nStartRow, SCROW nEndRow,
                                 SCCOL& rPaintCol, SCROW& rPaintRow,
                                 bool bRefresh )
@@ -1445,7 +1446,6 @@ bool ScAttrArray::RemoveAreaMerge(SCROW nStartRow, SCROW nEndRow)
             SCROW nMergeEndRow = nThisEnd + nCountY - 1;
 
             // ApplyAttr for areas
-
             for (SCROW nThisRow = nThisStart; nThisRow <= nThisEnd; nThisRow++)
                 pDocument->ApplyAttr( nThisCol, nThisRow, nTab, *pAttr );
 
@@ -1469,8 +1469,9 @@ bool ScAttrArray::RemoveAreaMerge(SCROW nStartRow, SCROW nEndRow)
     return bFound;
 }
 
-// Remove field, but leave MergeFlags
-
+/**
+ * Remove field, but leave MergeFlags
+ */
 void ScAttrArray::DeleteAreaSafe(SCROW nStartRow, SCROW nEndRow)
 {
     SetPatternAreaSafe( nStartRow, nEndRow, pDocument->GetDefPattern(), true );
@@ -1492,7 +1493,7 @@ void ScAttrArray::SetPatternAreaSafe( SCROW nStartRow, SCROW nEndRow,
     while ( nThisRow <= nEndRow )
     {
         pOldPattern = pData[nIndex].pPattern;
-        if (pOldPattern != pWantedPattern)                          //! else-Zweig ?
+        if (pOldPattern != pWantedPattern) // FIXME: else-branch?
         {
             if (nThisRow < nStartRow) nThisRow = nStartRow;
             nRow = pData[nIndex].nRow;
@@ -1652,7 +1653,7 @@ void ScAttrArray::ChangeIndent( SCROW nStartRow, SCROW nEndRow, bool bIncrement
                                ((const SvxHorJustifyItem*)pItem)->GetValue() != SVX_HOR_JUSTIFY_RIGHT ));
         sal_uInt16 nOldValue = ((const SfxUInt16Item&)rOldSet.Get( ATTR_INDENT )).GetValue();
         sal_uInt16 nNewValue = nOldValue;
-        //to keep Increment indent from running outside the cell1659
+        // To keep Increment indent from running outside the cell1659
         long nColWidth = (long)pDocument->GetColWidth(nCol,nTab);
         if ( bIncrement )
         {
@@ -2184,8 +2185,9 @@ void ScAttrArray::DeleteHardAttr(SCROW nStartRow, SCROW nEndRow)
     }
 }
 
-// move within a document
-
+/**
+ * Move within a document
+ */
 void ScAttrArray::MoveTo(SCROW nStartRow, SCROW nEndRow, ScAttrArray& rAttrArray)
 {
     SCROW nStart = nStartRow;
@@ -2202,8 +2204,9 @@ void ScAttrArray::MoveTo(SCROW nStartRow, SCROW nEndRow, ScAttrArray& rAttrArray
     DeleteArea(nStartRow, nEndRow);
 }
 
-// copy between documents (Clipboard)
-
+/**
+ * Copy between documents (Clipboard)
+ */
 void ScAttrArray::CopyArea(
     SCROW nStartRow, SCROW nEndRow, long nDy, ScAttrArray& rAttrArray, sal_Int16 nStripFlags) const
 {
@@ -2267,9 +2270,10 @@ void ScAttrArray::CopyArea(
     }
 }
 
-// leave flags
-// summarized with CopyArea
-
+/**
+ * Leave flags
+ * summarized with CopyArea
+ */
 void ScAttrArray::CopyAreaSafe( SCROW nStartRow, SCROW nEndRow, long nDy, ScAttrArray& rAttrArray )
 {
     nStartRow -= nDy;  // Source

sc/source/core/data/attrib.cxx

@@ -51,8 +51,9 @@ TYPEINIT1(ScDoubleItem,         SfxPoolItem);
 TYPEINIT1(ScPageScaleToItem,    SfxPoolItem);
 TYPEINIT1(ScCondFormatItem,    SfxPoolItem);
 
-//      General Help Function
-
+/**
+ * General Help Function
+ */
 bool ScHasPriority( const ::editeng::SvxBorderLine* pThis, const ::editeng::SvxBorderLine* pOther )
 {
 
@@ -76,15 +77,16 @@ bool ScHasPriority( const ::editeng::SvxBorderLine* pThis, const ::editeng::SvxB
             return false;
         else
         {
-            return true;            //! ???
+            return true; // FIXME: What is this?
         }
     }
 }
 
-//      Item - Implementierungen
-
-// Merge
+/** Item - Implementations */
 
+/**
+ * Merge
+ */
 ScMergeAttr::ScMergeAttr():
     SfxPoolItem(ATTR_MERGE),
     nColMerge(0),
@@ -140,8 +142,9 @@ SfxPoolItem* ScMergeAttr::Create( SvStream& rStream, sal_uInt16 /* nVer */ ) con
     return new ScMergeAttr(static_cast<SCCOL>(nCol),static_cast<SCROW>(nRow));
 }
 
-// MergeFlag
-
+/**
+ * MergeFlag
+ */
 ScMergeFlagAttr::ScMergeFlagAttr():
     SfxInt16Item(ATTR_MERGE_FLAG, 0)
 {
@@ -166,8 +169,9 @@ bool ScMergeFlagAttr::HasPivotPopupButton() const
     return (GetValue() & SC_MF_BUTTON_POPUP) != 0;
 }
 
-// Protection
-
+/**
+ * Protection
+ */
 ScProtectionAttr::ScProtectionAttr():
     SfxPoolItem(ATTR_PROTECTION),
     bProtection(true),
@@ -384,8 +388,9 @@ bool ScProtectionAttr::SetHidePrint( bool bHPrint)
     return true;
 }
 
-//      ScRangeItem - Tabellenbereich
-
+/**
+ * ScRangeItem - Table range
+ */
 bool ScRangeItem::operator==( const SfxPoolItem& rAttr ) const
 {
     OSL_ENSURE( SfxPoolItem::operator==(rAttr), "unequal types" );
@@ -431,8 +436,9 @@ bool ScRangeItem::GetPresentation
     return true;
 }
 
-//      ScTableListItem - List from Tables (-numbers)
-
+/**
+ * ScTableListItem - List from Tables (-numbers)
+ */
 ScTableListItem::ScTableListItem( const ScTableListItem& rCpy )
     :   SfxPoolItem ( rCpy.Which() ),
         nCount      ( rCpy.nCount )
@@ -536,8 +542,9 @@ bool ScTableListItem::GetPresentation
     return false;
 }
 
-//      ScPageHFItem - Dates from the Head and Foot lines
-
+/**
+ * ScPageHFItem - Dates from the Head and Foot lines
+ */
 ScPageHFItem::ScPageHFItem( sal_uInt16 nWhichP )
     :   SfxPoolItem ( nWhichP ),
         pLeftArea   ( NULL ),
@@ -649,7 +656,6 @@ SfxPoolItem* ScPageHFItem::Clone( SfxItemPool* ) const
 static void lcl_SetSpace( OUString& rStr, const ESelection& rSel )
 {
     // Text replaced by a space to ensure they are positions:
-
     sal_Int32 nLen = rSel.nEndPos-rSel.nStartPos;
     rStr = rStr.replaceAt( rSel.nStartPos, nLen, " " );
 }
@@ -717,10 +723,9 @@ SfxPoolItem* ScPageHFItem::Create( SvStream& rStream, sal_uInt16 nVer ) const
          pCenter == NULL || pCenter->GetParagraphCount() == 0 ||
          pRight == NULL  || pRight->GetParagraphCount() == 0 )
     {
-        //  If successfully loaded, each object contains at least one paragraph.
-        //  Excel import in 5.1 created broken TextObjects (#67442#) that are
-        //  corrected here to avoid saving wrong files again (#90487#).
-
+        // If successfully loaded, each object contains at least one paragraph.
+        // Excel import in 5.1 created broken TextObjects (#67442#) that are
+        // corrected here to avoid saving wrong files again (#90487#).
         ScEditEngineDefaulter aEngine( EditEngine::CreatePool(), true );
         if ( pLeft == NULL || pLeft->GetParagraphCount() == 0 )
         {
@@ -739,7 +744,7 @@ SfxPoolItem* ScPageHFItem::Create( SvStream& rStream, sal_uInt16 nVer ) const
         }
     }
 
-    if ( nVer < 1 )             //old field command conversions
+    if ( nVer < 1 ) // old field command conversions
     {
         sal_uInt16 i;
         const OUString& rDel = ScGlobal::GetRscString( STR_HFCMD_DELIMITER );
@@ -775,9 +780,7 @@ SfxPoolItem* ScPageHFItem::Create( SvStream& rStream, sal_uInt16 nVer ) const
             pRight = aEngine.CreateTextObject();
         }
     }
-    else if ( nVer < 2 )
-    {   // not to do, SvxFileField is not exchanged for SvxExtFileField
-    }
+    else if ( nVer < 2 ) {} // nothing to do: SvxFileField is not exchanged for SvxExtFileField
 
     ScPageHFItem* pItem = new ScPageHFItem( Which() );
     pItem->SetArea( pLeft,    SC_HF_LEFTAREA   );
@@ -817,8 +820,9 @@ void ScPageHFItem::SetArea( EditTextObject *pNew, int nArea )
     }
 }
 
-//  ScViewObjectModeItem - Display Mode of View Objects
-
+/**
+ * ScViewObjectModeItem - Display Mode of View Objects
+ */
 ScViewObjectModeItem::ScViewObjectModeItem( sal_uInt16 nWhichP )
     : SfxEnumItem( nWhichP, VOBJ_MODE_SHOW )
 {
@@ -863,11 +867,10 @@ bool ScViewObjectModeItem::GetPresentation
             break;
 
             default:
-            ePres = SFX_ITEM_PRESENTATION_NAMELESS;//this always goes!
+            ePres = SFX_ITEM_PRESENTATION_NAMELESS; // Default setting!
             break;
         }
         /* !!! fall-through !!! */
-
         case SFX_ITEM_PRESENTATION_NAMELESS:
         rText += ScGlobal::GetRscString(STR_VOBJ_MODE_SHOW+GetValue());
         return true;
@@ -923,8 +926,9 @@ SfxPoolItem* ScViewObjectModeItem::Create(
     }
 }
 
-//      double
-
+/**
+ * Double
+ */
 ScDoubleItem::ScDoubleItem( sal_uInt16 nWhichP, double nVal )
     :   SfxPoolItem ( nWhichP ),
         nValue  ( nVal )