From 433d82998088c628ebdff52421113cf195efea45 Mon Sep 17 00:00:00 2001 From: shirahara Date: Sat, 29 Jan 2011 18:12:26 +0900 Subject: [PATCH] easyhacks: remove bogus comments --- sw/source/core/access/acccontext.cxx | 14 ++++++------ sw/source/core/access/accdoc.cxx | 5 +-- sw/source/core/access/accframe.cxx | 1 - sw/source/core/access/accnotextframe.cxx | 14 +++++------- sw/source/core/access/accpara.cxx | 32 ++++++++++++++--------------- sw/source/core/bastyp/calc.cxx | 4 +- sw/source/core/crsr/swcrsr.cxx | 2 +- sw/source/core/doc/docdesc.cxx | 4 +- sw/source/core/doc/docnew.cxx | 2 +- sw/source/core/doc/docredln.cxx | 5 +-- sw/source/core/doc/notxtfrm.cxx | 18 +++++++--------- 11 files changed, 46 insertions(+), 55 deletions(-) diff --git a/sw/source/core/access/acccontext.cxx b/sw/source/core/access/acccontext.cxx index 127820f..9b3c28a 100644 --- a/sw/source/core/access/acccontext.cxx +++ b/sw/source/core/access/acccontext.cxx @@ -399,7 +399,7 @@ void SwAccessibleContext::ScrolledOut( const SwRect& rOldVisArea ) Dispose( sal_True ); } -// --> OD 2005-12-12 #i27301# - use new type definition for <_nStates> +// --> #i27301# - use new type definition for <_nStates> void SwAccessibleContext::InvalidateChildrenStates( const SwFrm* _pFrm, tAccessibleStates _nStates ) { @@ -1246,7 +1246,7 @@ void SwAccessibleContext::InvalidateChildPosOrSize( ::rtl::Reference< ::accessibility::AccessibleShape > xAccImpl = GetMap()->GetContextImpl( rChildFrmOrObj.GetDrawObject(), this, sal_True ); - // --> OD 2004-11-29 #i37790# + // -->#i37790# if ( xAccImpl.is() ) { ScrolledInShape( rChildFrmOrObj.GetDrawObject(), @@ -1323,7 +1323,7 @@ void SwAccessibleContext::InvalidateFocus() _InvalidateFocus(); } -// --> OD 2005-12-12 #i27301# - use new type definition for <_nStates> +// -->#i27301# - use new type definition for <_nStates> void SwAccessibleContext::InvalidateStates( tAccessibleStates _nStates ) { if( GetMap() ) @@ -1376,7 +1376,7 @@ void SwAccessibleContext::InvalidateRelation( sal_uInt16 nType ) /** text selection has changed - OD 2005-12-14 #i27301# + OD #i27301# @author OD */ @@ -1390,7 +1390,7 @@ void SwAccessibleContext::InvalidateTextSelection() /** attributes has changed - OD 2009-01-06 #i88069# + OD #i88069# @author OD */ @@ -1504,7 +1504,7 @@ bool SwAccessibleContext::HasAdditionalAccessibleChildren() } /** get additional accessible child by index - OD 2010-01-27 #i88070# + OD #i88070# @author OD */ @@ -1527,7 +1527,7 @@ Window* SwAccessibleContext::GetAdditionalAccessibleChild( const sal_Int32 nInde /** get all additional accessible children - OD 2010-01-27 #i88070# + OD #i88070# @author OD */ diff --git a/sw/source/core/access/accdoc.cxx b/sw/source/core/access/accdoc.cxx index e7b8e8b..a045a93 100644 --- a/sw/source/core/access/accdoc.cxx +++ b/sw/source/core/access/accdoc.cxx @@ -86,8 +86,7 @@ void SwAccessibleDocumentBase::SetVisArea() if( aOldVisArea != rNewVisArea ) { SwAccessibleFrame::SetVisArea( GetMap()->GetVisArea() ); - // --> OD 2007-12-07 #i58139# - // showing state of document view needs also be updated. + // --> #i58139# showing state of document view needs also be updated. // Thus, call method instead of // ChildrenScrolled( GetFrm(), aOldVisArea ); Scrolled( aOldVisArea ); @@ -504,7 +503,7 @@ uno::Reference SwAccessibleDocument::getSelectedAccessibleChild( return maSelectionHelper.getSelectedAccessibleChild(nSelectedChildIndex); } -// --> OD 2004-11-16 #111714# - index has to be treated as global child index. +// -->index has to be treated as global child index. void SwAccessibleDocument::deselectAccessibleChild( sal_Int32 nChildIndex ) throw ( lang::IndexOutOfBoundsException, diff --git a/sw/source/core/access/accframe.cxx b/sw/source/core/access/accframe.cxx index a00fd79..29dfc11 100644 --- a/sw/source/core/access/accframe.cxx +++ b/sw/source/core/access/accframe.cxx @@ -396,7 +396,6 @@ sal_Bool SwAccessibleFrame::IsOpaque( ViewShell *pVSh ) const rBack.GetGraphicPos() != GPOS_NONE ) return sal_True; - /// OD 20.08.2002 #99657# /// If a fly frame has a transparent background color, we have /// to consider the background. /// But a background color "no fill"/"auto fill" has *not* to be considered. diff --git a/sw/source/core/access/accnotextframe.cxx b/sw/source/core/access/accnotextframe.cxx index 8eaf921..b2b612d 100644 --- a/sw/source/core/access/accnotextframe.cxx +++ b/sw/source/core/access/accnotextframe.cxx @@ -40,7 +40,7 @@ #include #include #include -// --> OD 2009-07-14 #i73249# +// --> #i73249# #include // <-- #include "accnotextframe.hxx" @@ -73,8 +73,7 @@ SwAccessibleNoTextFrame::SwAccessibleNoTextFrame( msDesc() { const SwNoTxtNode* pNd = GetNoTxtNode(); - // --> OD 2009-07-14 #i73249# - // consider new attributes Title and Description + // --> #i73249# consider new attributes Title and Description if( pNd ) { msTitle = pNd->GetTitle(); @@ -96,9 +95,8 @@ SwAccessibleNoTextFrame::~SwAccessibleNoTextFrame() void SwAccessibleNoTextFrame::Modify( SfxPoolItem *pOld, SfxPoolItem *pNew) { const sal_uInt16 nWhich = pOld ? pOld->Which() : pNew ? pNew->Which() : 0 ; - // --> OD 2009-07-14 #i73249# - // suppress handling of RES_NAME_CHANGED in case that attribute Title is - // used as the accessible name. + // -->#i73249# suppress handling of RES_NAME_CHANGED + // in case that attribute Title is used as the accessible name. if ( nWhich != RES_NAME_CHANGED || msTitle.getLength() == 0 ) { @@ -109,7 +107,7 @@ void SwAccessibleNoTextFrame::Modify( SfxPoolItem *pOld, SfxPoolItem *pNew) OSL_ENSURE( pNd == aDepend.GetRegisteredIn(), "invalid frame" ); switch( nWhich ) { - // --> OD 2009-07-14 #i73249# + // --> #i73249# case RES_TITLE_CHANGED: { const String& sOldTitle( @@ -171,7 +169,7 @@ void SwAccessibleNoTextFrame::Dispose( sal_Bool bRecursive ) SwAccessibleFrameBase::Dispose( bRecursive ); } -// --> OD 2009-07-14 #i73249# +// --> #i73249# OUString SAL_CALL SwAccessibleNoTextFrame::getAccessibleName (void) throw (uno::RuntimeException) { diff --git a/sw/source/core/access/accpara.cxx b/sw/source/core/access/accpara.cxx index f3c2c16..286a6bd 100644 --- a/sw/source/core/access/accpara.cxx +++ b/sw/source/core/access/accpara.cxx @@ -78,11 +78,11 @@ #include #include #include -// --> OD 2010-02-22 #i10825# +// -->#i10825# #include #include // <-- -// --> OD 2010-03-08 #i92233# +// -->#i92233# #include // <-- @@ -128,7 +128,7 @@ const SwTxtNode* SwAccessibleParagraph::GetTxtNode() const ::rtl::OUString SwAccessibleParagraph::GetDescription() { - // --> OD 2004-09-29 #117933# - provide empty description for paragraphs + // -->provide empty description for paragraphs return ::rtl::OUString(); // <-- } @@ -138,7 +138,7 @@ sal_Int32 SwAccessibleParagraph::GetCaretPos() sal_Int32 nRet = -1; // get the selection's point, and test whether it's in our node - // --> OD 2005-12-20 #i27301# - consider adjusted method signature + // -->#i27301# - consider adjusted method signature SwPaM* pCaret = GetCursor( false ); // caret is first PaM in PaM-ring // <-- if( pCaret != NULL ) @@ -155,7 +155,6 @@ sal_Int32 SwAccessibleParagraph::GetCaretPos() if( GetPortionData().IsValidCorePosition( nIndex ) ) { // Yes, it's us! - // --> OD 2006-10-19 #70538# // consider that cursor/caret is in front of the list label if ( pCaret->IsInFrontOfLabel() ) { @@ -188,7 +187,7 @@ sal_Bool SwAccessibleParagraph::GetSelection( nEnd = -1; // get the selection, and test whether it affects our text node - // --> OD 2005-12-20 #i27301# - consider adjusted method signature + // -->#i27301# - consider adjusted method signature SwPaM* pCrsr = GetCursor( true ); // <-- if( pCrsr != NULL ) @@ -309,16 +308,15 @@ sal_Bool SwAccessibleParagraph::GetSelection( return bRet; } -// --> OD 2005-12-20 #i27301# - new parameter <_bForSelection> +// --> #i27301# - new parameter <_bForSelection> SwPaM* SwAccessibleParagraph::GetCursor( const bool _bForSelection ) { // get the cursor shell; if we don't have any, we don't have a // cursor/selection either SwPaM* pCrsr = NULL; SwCrsrShell* pCrsrShell = SwAccessibleParagraph::GetCrsrShell(); - // --> OD 2005-12-20 #i27301# - // - if cursor is retrieved for selection, the cursors for a table selection - // has to be returned. + // --> #i27301# - if cursor is retrieved for selection, + // the cursors for a table selection has to be returned. if ( pCrsrShell != NULL && ( _bForSelection || !pCrsrShell->IsTableMode() ) ) // <-- @@ -360,7 +358,7 @@ void SwAccessibleParagraph::GetStates( rStateSet.AddState( AccessibleStateType::FOCUSABLE ); // FOCUSED (simulates node index of cursor) - // --> OD 2005-12-20 #i27301# - consider adjusted method signature + // --> #i27301# - consider adjusted method signature SwPaM* pCaret = GetCursor( false ); // <-- const SwTxtNode* pTxtNd = GetTxtNode(); @@ -494,7 +492,7 @@ void SwAccessibleParagraph::_InvalidateFocus() SwAccessibleParagraph::SwAccessibleParagraph( SwAccessibleMap& rInitMap, const SwTxtFrm& rTxtFrm ) - // --> OD 2010-02-24 #i108125# + // --> #i108125# : SwClient( const_cast(rTxtFrm.GetTxtNode()) ) // <-- , SwAccessibleContext( &rInitMap, AccessibleRole::PARAGRAPH, &rTxtFrm ) @@ -504,14 +502,14 @@ SwAccessibleParagraph::SwAccessibleParagraph( , nOldCaretPos( -1 ) , bIsHeading( sal_False ) , aSelectionHelper( *this ) - // --> OD 2010-02-19 #i108125# + // -->#i108125# , mpParaChangeTrackInfo( new SwParaChangeTrackingInfo( rTxtFrm ) ) // <-- { SolarMutexGuard aGuard; bIsHeading = IsHeading(); - // --> OD 2004-09-27 #117970# - set an empty accessibility name for paragraphs + // -->#117970# - set an empty accessibility name for paragraphs SetName( ::rtl::OUString() ); // <-- @@ -525,7 +523,7 @@ SwAccessibleParagraph::~SwAccessibleParagraph() delete pPortionData; delete pHyperTextData; - // --> OD 2010-02-22 #i108125# + // --> #i108125# delete mpParaChangeTrackInfo; // <-- } @@ -864,7 +862,7 @@ lang::Locale SAL_CALL SwAccessibleParagraph::getLocale (void) /** paragraphs are in relation CONTENT_FLOWS_FROM and/or CONTENT_FLOWS_TO - OD 2005-12-02 #i27138# + OD #i27138# @author OD */ @@ -914,7 +912,7 @@ void SAL_CALL SwAccessibleParagraph::grabFocus() // get cursor shell SwCrsrShell *pCrsrSh = GetCrsrShell(); - // --> OD 2005-12-20 #i27301# - consider new method signature + // -->#i27301# - consider new method signature SwPaM *pCrsr = GetCursor( false ); // <-- const SwTxtFrm *pTxtFrm = static_cast( GetFrm() ); diff --git a/sw/source/core/bastyp/calc.cxx b/sw/source/core/bastyp/calc.cxx index 00bd963..0c37188 100644 --- a/sw/source/core/bastyp/calc.cxx +++ b/sw/source/core/bastyp/calc.cxx @@ -721,8 +721,8 @@ void SwCalc::Pop( const VoidPtr ) SwCalcOper SwCalc::GetToken() { #if OSL_DEBUG_LEVEL > 1 -//JP 25.01.2001: static for switch back to the "old" implementation of the -// calculator, which don't use the I18N routines. +//static for switch back to the "old" implementation of the +//calculator, which don't use the I18N routines. static int nUseOld = 0; if( !nUseOld ) { diff --git a/sw/source/core/crsr/swcrsr.cxx b/sw/source/core/crsr/swcrsr.cxx index 3842154..afdcc92 100644 --- a/sw/source/core/crsr/swcrsr.cxx +++ b/sw/source/core/crsr/swcrsr.cxx @@ -2045,7 +2045,7 @@ BOOL SwCursor::GotoTblBox( const String& rName ) BOOL SwCursor::MovePara(SwWhichPara fnWhichPara, SwPosPara fnPosPara ) { - //JP 28.8.2001: for optimization test something before + //for optimization test something before const SwNode* pNd = &GetPoint()->nNode.GetNode(); bool bShortCut = false; if ( fnWhichPara == fnParaCurr ) diff --git a/sw/source/core/doc/docdesc.cxx b/sw/source/core/doc/docdesc.cxx index f8379e1..674e740 100644 --- a/sw/source/core/doc/docdesc.cxx +++ b/sw/source/core/doc/docdesc.cxx @@ -94,8 +94,8 @@ static void lcl_DefaultPageFmt( sal_uInt16 nPoolFmtId, //Raender haben eine defaultmaessige Mindestgroesse. //wenn der Drucker einen groesseren Rand vorgibt, so //ist mir dass auch recht. - // MIB 06/25/2002, #99397#: The HTML page desc had A4 as page size - // always. This has been changed to take the page size from the printer. + // The HTML page desc had A4 as page size always. + // This has been changed to take the page size from the printer. // Unfortunately, the margins of the HTML page desc are smaller than // the margins used here in general, so one extra case is required. // In the long term, this needs to be changed to always keep the diff --git a/sw/source/core/doc/docnew.cxx b/sw/source/core/doc/docnew.cxx index dc11d47..dc01eea 100644 --- a/sw/source/core/doc/docnew.cxx +++ b/sw/source/core/doc/docnew.cxx @@ -874,7 +874,7 @@ SwModify* SwDoc::GetUnoCallBack() const return pUnoCallBack; } -/*-----------------28.5.2001 10:06------------------ + * SwDoc: * Reading and writing of the layout cache. *--------------------------------------------------*/ diff --git a/sw/source/core/doc/docredln.cxx b/sw/source/core/doc/docredln.cxx index 6567351..d422972 100644 --- a/sw/source/core/doc/docredln.cxx +++ b/sw/source/core/doc/docredln.cxx @@ -3122,9 +3122,8 @@ BOOL SwRedline::HasValidRange() const * pMkNd = &GetMark()->nNode.GetNode(); if( pPtNd->StartOfSectionNode() == pMkNd->StartOfSectionNode() && !pPtNd->StartOfSectionNode()->IsTableNode() && - // JP 18.5.2001: Bug 87222 - invalid if points on the end of content - // DVO 25.03.2002: #96530# end-of-content only invalid if no content - // index exists + // invalid if points on the end of content + // end-of-content only invalid if no content index exists ( pPtNd != pMkNd || GetContentIdx() != NULL || pPtNd != &pPtNd->GetNodes().GetEndOfContent() ) ) diff --git a/sw/source/core/doc/notxtfrm.cxx b/sw/source/core/doc/notxtfrm.cxx index 62a036b..af7ca81 100644 --- a/sw/source/core/doc/notxtfrm.cxx +++ b/sw/source/core/doc/notxtfrm.cxx @@ -323,7 +323,7 @@ void SwNoTxtFrm::Paint( const SwRect &rRect, const SwPrtOptions * /*pPrintData*/ if ( bClip ) pOut->IntersectClipRegion( aPaintArea.SVRect() ); - /// OD 25.09.2002 #99739# - delete unused 3rd parameter + /// delete unused 3rd parameter PaintPicture( pOut, aGrfArea ); } else @@ -378,9 +378,8 @@ void lcl_CalcRect( Point& rPt, Size& rDim, USHORT nMirror ) void SwNoTxtFrm::GetGrfArea( SwRect &rRect, SwRect* pOrigRect, BOOL ) const { - // JP 23.01.2001: currently only used for scaling, cropping and mirroring - // the contour of graphics! - // all other is handled by the GraphicObject + //currently only used for scaling, cropping and mirroring the contour of graphics! + //all other is handled by the GraphicObject //In rRect wird das sichbare Rechteck der Grafik gesteckt. //In pOrigRect werden Pos+Size der Gesamtgrafik gesteck. @@ -731,7 +730,7 @@ void SwNoTxtFrm::Modify( SfxPoolItem* pOld, SfxPoolItem* pNew ) else if ( pSh->VisArea().IsOver( aRect ) && OUTDEV_WINDOW == pSh->GetOut()->GetOutDevType() ) { - // OD 27.11.2002 #105519# - invalidate instead of painting + //invalidate instead of painting pSh->GetWin()->Invalidate( aRect.SVRect() ); } @@ -784,9 +783,9 @@ void lcl_correctlyAlignRect( SwRect& rAlignedGrfArea, const SwRect& rInArea, Out // Ausgabe der Grafik. Hier wird entweder eine QuickDraw-Bmp oder // eine Grafik vorausgesetzt. Ist nichts davon vorhanden, wird // eine Ersatzdarstellung ausgegeben. -/// OD 25.09.2002 #99739# - delete unused 3rd parameter. -/// OD 25.09.2002 #99739# - use aligned rectangle for drawing graphic. -/// OD 25.09.2002 #99739# - pixel-align coordinations for drawing graphic. +/// delete unused 3rd parameter. +/// use aligned rectangle for drawing graphic. +/// pixel-align coordinations for drawing graphic. void SwNoTxtFrm::PaintPicture( OutputDevice* pOut, const SwRect &rGrfArea ) const { ViewShell* pShell = GetShell(); @@ -800,7 +799,7 @@ void SwNoTxtFrm::PaintPicture( OutputDevice* pOut, const SwRect &rGrfArea ) cons const bool bIsChart = pOLENd && ChartPrettyPainter::IsChart( pOLENd->GetOLEObj().GetObject() ); - /// OD 25.09.2002 #99739# - calculate aligned rectangle from parameter . + /// calculate aligned rectangle from parameter . /// Use aligned rectangle instead of in /// the following code. SwRect aAlignedGrfArea = rGrfArea; @@ -808,7 +807,6 @@ void SwNoTxtFrm::PaintPicture( OutputDevice* pOut, const SwRect &rGrfArea ) cons if( !bIsChart ) { - /// OD 25.09.2002 #99739# /// Because for drawing a graphic left-top-corner and size coordinations are /// used, these coordinations have to be determined on pixel level. ::SwAlignGrfRect( &aAlignedGrfArea, *pOut ); -- 1.7.3.1.msysgit.0