[PATCH] sw: convert some usage of Point::X/Y to Point::set/getX/Y, t...

Miklos Vajna (via Code Review) gerrit at gerrit.libreoffice.org
Fri May 17 00:18:58 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3929

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/29/3929/1

sw: convert some usage of Point::X/Y to Point::set/getX/Y, take two

69c52bc5b7973a2e414209470f5bf9b66dcc4f30 had some typos, and got
reverted in 78123a504b5215d061935d7b2771ba51e68cae0e, this does the
same, except those typos corrected, so vertical layout is still OK.

Change-Id: I79fe610e4cc8bbbddbd8415198d6309a93f4ba18
---
M sw/source/core/access/acccontext.cxx
M sw/source/core/access/accdoc.cxx
M sw/source/core/access/accpara.cxx
M sw/source/core/access/acctable.cxx
M sw/source/core/bastyp/swrect.cxx
M sw/source/core/crsr/crsrsh.cxx
M sw/source/core/crsr/crstrvl.cxx
M sw/source/core/crsr/swcrsr.cxx
M sw/source/core/crsr/trvlcol.cxx
M sw/source/core/crsr/trvlfnfl.cxx
M sw/source/core/crsr/viscrs.cxx
M sw/source/core/doc/docdraw.cxx
M sw/source/core/doc/docfly.cxx
M sw/source/core/doc/notxtfrm.cxx
M sw/source/core/docnode/ndnotxt.cxx
M sw/source/core/draw/dflyobj.cxx
M sw/source/core/draw/dpage.cxx
M sw/source/core/draw/dview.cxx
M sw/source/core/frmedt/fecopy.cxx
M sw/source/core/frmedt/fefly1.cxx
M sw/source/core/frmedt/feshview.cxx
M sw/source/core/frmedt/fetab.cxx
M sw/source/core/frmedt/fews.cxx
M sw/source/core/frmedt/tblsel.cxx
M sw/source/core/layout/anchoreddrawobject.cxx
M sw/source/core/layout/calcmove.cxx
M sw/source/core/layout/fly.cxx
M sw/source/core/layout/flycnt.cxx
M sw/source/core/layout/flyincnt.cxx
M sw/source/core/layout/frmtool.cxx
M sw/source/core/layout/pagechg.cxx
M sw/source/core/layout/paintfrm.cxx
32 files changed, 291 insertions(+), 282 deletions(-)



diff --git a/sw/source/core/access/acccontext.cxx b/sw/source/core/access/acccontext.cxx
index e80e97b9..8753289 100644
--- a/sw/source/core/access/acccontext.cxx
+++ b/sw/source/core/access/acccontext.cxx
@@ -770,8 +770,8 @@
     {
         SwRect aLogBounds( GetBounds( *(GetMap()), GetFrm() ) ); // twip rel to doc root
         Point aPixPos( GetMap()->CoreToPixel( aLogBounds.SVRect() ).TopLeft() );
-        aPixPoint.X() += aPixPos.X();
-        aPixPoint.Y() += aPixPos.Y();
+        aPixPoint.setX(aPixPoint.getX() + aPixPos.getX());
+        aPixPoint.setY(aPixPoint.getY() + aPixPos.getY());
     }
 
     const SwAccessibleChild aChild( GetChildAtPixel( aPixPoint, *(GetMap()) ) );
@@ -844,7 +844,7 @@
         {
             SwRect aParentLogBounds( GetBounds( *(GetMap()), pParent ) ); // twip rel to doc root
             Point aParentPixPos( GetMap()->CoreToPixel( aParentLogBounds.SVRect() ).TopLeft() );
-            aPixBounds.Move( -aParentPixPos.X(), -aParentPixPos.Y() );
+            aPixBounds.Move( -aParentPixPos.getX(), -aParentPixPos.getY() );
         }
     }
 
@@ -881,7 +881,7 @@
 
     /* getBoundsImpl already checked that GetWindow returns valid pointer. */
     aPixPos = GetWindow()->OutputToAbsoluteScreenPixel(aPixPos);
-    awt::Point aPoint(aPixPos.X(), aPixPos.Y());
+    awt::Point aPoint(aPixPos.getX(), aPixPos.getY());
 
     return aPoint;
 }
diff --git a/sw/source/core/access/accdoc.cxx b/sw/source/core/access/accdoc.cxx
index 78981f85..746146b 100644
--- a/sw/source/core/access/accdoc.cxx
+++ b/sw/source/core/access/accdoc.cxx
@@ -202,7 +202,7 @@
     CHECK_FOR_WINDOW( XAccessibleComponent, pWin )
 
     Point aPixPos( pWin->GetWindowExtentsRelative( pWin->GetAccessibleParentWindow() ).TopLeft() );
-    awt::Point aLoc( aPixPos.X(), aPixPos.Y() );
+    awt::Point aLoc( aPixPos.getX(), aPixPos.getY() );
 
     return aLoc;
 }
@@ -218,7 +218,7 @@
     CHECK_FOR_WINDOW( XAccessibleComponent, pWin )
 
     Point aPixPos( pWin->GetWindowExtentsRelative( 0 ).TopLeft() );
-    awt::Point aLoc( aPixPos.X(), aPixPos.Y() );
+    awt::Point aLoc( aPixPos.getX(), aPixPos.getY() );
 
     return aLoc;
 }
diff --git a/sw/source/core/access/accpara.cxx b/sw/source/core/access/accpara.cxx
index 5dba314..3b6ef33 100644
--- a/sw/source/core/access/accpara.cxx
+++ b/sw/source/core/access/accpara.cxx
@@ -1625,7 +1625,7 @@
     SwRect aFrmLogBounds( GetBounds( *(GetMap()) ) ); // twip rel to doc root
 
     Point aFrmPixPos( GetMap()->CoreToPixel( aFrmLogBounds.SVRect() ).TopLeft() );
-    aScreenRect.Move( -aFrmPixPos.X(), -aFrmPixPos.Y() );
+    aScreenRect.Move( -aFrmPixPos.getX(), -aFrmPixPos.getY() );
 
     // convert into AWT Rectangle
     return awt::Rectangle(
@@ -1662,8 +1662,8 @@
     Point aPoint( rPoint.X, rPoint.Y );
     SwRect aLogBounds( GetBounds( *(GetMap()), GetFrm() ) ); // twip rel to doc root
     Point aPixPos( GetMap()->CoreToPixel( aLogBounds.SVRect() ).TopLeft() );
-    aPoint.X() += aPixPos.X();
-    aPoint.Y() += aPixPos.Y();
+    aPoint.setX(aPoint.getX() + aPixPos.getX());
+    aPoint.setY(aPoint.getY() + aPixPos.getY());
     MapMode aMapMode = pWin->GetMapMode();
     Point aCorePoint( GetMap()->PixelToCore( aPoint ) );
     if( !aLogBounds.IsInside( aCorePoint ) )
@@ -1700,17 +1700,17 @@
         bool bVert = pFrm->IsVertical();
         bool bR2L = pFrm->IsRightToLeft();
 
-        if ( (!bVert && aResultRect.Pos().X() > aCorePoint.X()) ||
-             ( bVert && aResultRect.Pos().Y() > aCorePoint.Y()) ||
-             ( bR2L  && aResultRect.Right()   < aCorePoint.X()) )
+        if ( (!bVert && aResultRect.Pos().getX() > aCorePoint.getX()) ||
+             ( bVert && aResultRect.Pos().getY() > aCorePoint.getY()) ||
+             ( bR2L  && aResultRect.Right()   < aCorePoint.getX()) )
         {
             SwIndex aIdxPrev( pNode, nIndex - 1);
             SwPosition aPosPrev( *pNode, aIdxPrev );
             SwRect aResultRectPrev;
             pFrm->GetCharRect( aResultRectPrev, aPosPrev );
-            if ( (!bVert && aResultRectPrev.Pos().X() < aCorePoint.X() && aResultRect.Pos().Y() == aResultRectPrev.Pos().Y()) ||
-                 ( bVert && aResultRectPrev.Pos().Y() < aCorePoint.Y() && aResultRect.Pos().X() == aResultRectPrev.Pos().X()) ||
-                 (  bR2L && aResultRectPrev.Right()   > aCorePoint.X() && aResultRect.Pos().Y() == aResultRectPrev.Pos().Y()) )
+            if ( (!bVert && aResultRectPrev.Pos().getX() < aCorePoint.getX() && aResultRect.Pos().getY() == aResultRectPrev.Pos().getY()) ||
+                 ( bVert && aResultRectPrev.Pos().getY() < aCorePoint.getY() && aResultRect.Pos().getX() == aResultRectPrev.Pos().getX()) ||
+                 (  bR2L && aResultRectPrev.Right()   > aCorePoint.getX() && aResultRect.Pos().getY() == aResultRectPrev.Pos().getY()) )
                 aPos = aPosPrev;
         }
     }
@@ -2535,7 +2535,7 @@
 
                 SwRect aFrmLogBounds( GetBounds( *(GetMap()) ) ); // twip rel to doc root
                 Point aFrmPixPos( GetMap()->CoreToPixel( aFrmLogBounds.SVRect() ).TopLeft() );
-                aScreenRect.Move( -aFrmPixPos.X(), -aFrmPixPos.Y() );
+                aScreenRect.Move( -aFrmPixPos.getX(), -aFrmPixPos.getY() );
 
                 // convert into AWT Rectangle
                 const awt::Rectangle aCursorRect( aScreenRect.Left(),
diff --git a/sw/source/core/access/acctable.cxx b/sw/source/core/access/acctable.cxx
index 8992690..f744730 100644
--- a/sw/source/core/access/acctable.cxx
+++ b/sw/source/core/access/acctable.cxx
@@ -154,14 +154,14 @@
                 // #i77106#
                 if ( IncludeRow( *pLower ) )
                 {
-                    maRows.insert( pLower->Frm().Top() - maTabFrmPos.Y() );
+                    maRows.insert( pLower->Frm().Top() - maTabFrmPos.getY() );
                     CollectData( pLower );
                 }
             }
             else if( pLower->IsCellFrm() &&
                      rLower.IsAccessible( mbIsInPagePreview ) )
             {
-                maColumns.insert( pLower->Frm().Left() - maTabFrmPos.X() );
+                maColumns.insert( pLower->Frm().Left() - maTabFrmPos.getX() );
             }
             else
             {
@@ -352,9 +352,9 @@
         ::std::advance( aStt,
             static_cast< Int32Set_Impl::difference_type >( nStart ) );
         if( bColumns )
-            aArea.Left( *aStt + aPos.X() );
+            aArea.Left( *aStt + aPos.getX() );
         else
-            aArea.Top( *aStt + aPos.Y() );
+            aArea.Top( *aStt + aPos.getY() );
     }
     if( nEnd < static_cast< sal_Int32 >( rRowsOrColumns.size() ) )
     {
@@ -362,9 +362,9 @@
         ::std::advance( aEnd,
             static_cast< Int32Set_Impl::difference_type >( nEnd ) );
         if( bColumns )
-            aArea.Right( *aEnd + aPos.X() - 1 );
+            aArea.Right( *aEnd + aPos.getX() - 1 );
         else
-            aArea.Bottom( *aEnd + aPos.Y() - 1 );
+            aArea.Bottom( *aEnd + aPos.getY() - 1 );
     }
 
     GetSelection( aPos, aArea, rSelBoxes, mpTabFrm, rSelHdl, bColumns );
diff --git a/sw/source/core/bastyp/swrect.cxx b/sw/source/core/bastyp/swrect.cxx
index 40db66c..a403c46 100644
--- a/sw/source/core/bastyp/swrect.cxx
+++ b/sw/source/core/bastyp/swrect.cxx
@@ -140,12 +140,12 @@
 {
     if ( m_Size.getHeight() < 0 )
     {
-        m_Point.Y() += m_Size.getHeight() + 1;
+        m_Point.setY(m_Point.getY() + m_Size.getHeight() + 1);
         m_Size.setHeight(-m_Size.getHeight());
     }
     if ( m_Size.getWidth() < 0 )
     {
-        m_Point.X() += m_Size.getWidth() + 1;
+        m_Point.setX(m_Point.getX() + m_Size.getWidth() + 1);
         m_Size.setWidth(-m_Size.getWidth());
     }
 }
@@ -167,9 +167,9 @@
 
 void SwRect::AddWidth( const long nAdd ) { m_Size.Width() += nAdd; }
 void SwRect::AddHeight( const long nAdd ) { m_Size.Height() += nAdd; }
-void SwRect::SubLeft( const long nSub ){ m_Size.Width() += nSub; m_Point.X() -= nSub; }
+void SwRect::SubLeft( const long nSub ){ m_Size.Width() += nSub; m_Point.setX(m_Point.getX() - nSub); }
 void SwRect::AddRight( const long nAdd ){ m_Size.Width() += nAdd; }
-void SwRect::SubTop( const long nSub ){ m_Size.Height() += nSub; m_Point.Y() -= nSub; }
+void SwRect::SubTop( const long nSub ){ m_Size.Height() += nSub; m_Point.setY(m_Point.getY() - nSub); }
 void SwRect::AddBottom( const long nAdd ){ m_Size.Height() += nAdd; }
 void SwRect::SetPosX( const long nNew ){ m_Point.setX(nNew); }
 void SwRect::SetPosY( const long nNew ){ m_Point.setY(nNew); }
diff --git a/sw/source/core/crsr/crsrsh.cxx b/sw/source/core/crsr/crsrsh.cxx
index bd1d749..d3f65cf 100644
--- a/sw/source/core/crsr/crsrsh.cxx
+++ b/sw/source/core/crsr/crsrsh.cxx
@@ -1186,8 +1186,8 @@
         m_pVisCrsr->Hide();
 
     m_bVisPortChgd = sal_True;
-    m_aOldRBPos.X() = VisArea().Right();
-    m_aOldRBPos.Y() = VisArea().Bottom();
+    m_aOldRBPos.setX(VisArea().Right());
+    m_aOldRBPos.setY(VisArea().Bottom());
 
     // For not having problems with the SV cursor, Update() is called for the
     // Window in ViewShell::VisPo...
@@ -1445,14 +1445,14 @@
                 m_pTblCrsr->Show();
 
             // set Cursor-Points to the new Positions
-            m_pTblCrsr->GetPtPos().X() = m_aCharRect.Left();
-            m_pTblCrsr->GetPtPos().Y() = m_aCharRect.Top();
+            m_pTblCrsr->GetPtPos().setX(m_aCharRect.Left());
+            m_pTblCrsr->GetPtPos().setY(m_aCharRect.Top());
 
             if( m_bSVCrsrVis )
             {
-                m_aCrsrHeight.X() = 0;
-                m_aCrsrHeight.Y() = aTmpState.aRealHeight.Y() < 0 ?
-                                  -m_aCharRect.Width() : m_aCharRect.Height();
+                m_aCrsrHeight.setX(0);
+                m_aCrsrHeight.setY(aTmpState.aRealHeight.getY() < 0 ?
+                                  -m_aCharRect.Width() : m_aCharRect.Height());
                 m_pVisCrsr->Show(); // show again
             }
             m_eMvState = MV_NONE;  // state for cursor travelling - GetCrsrOfst
@@ -1653,15 +1653,15 @@
                 m_aCrsrHeight = aTmpState.aRealHeight;
             else
             {
-                m_aCrsrHeight.X() = 0;
-                m_aCrsrHeight.Y() = aTmpState.aRealHeight.Y() < 0 ?
-                                  -m_aCharRect.Width() : m_aCharRect.Height();
+                m_aCrsrHeight.setX(0);
+                m_aCrsrHeight.setY(aTmpState.aRealHeight.getY() < 0 ?
+                                  -m_aCharRect.Width() : m_aCharRect.Height());
             }
         }
         else
         {
-            m_aCrsrHeight.X() = 0;
-            m_aCrsrHeight.Y() = m_aCharRect.Height();
+            m_aCrsrHeight.setX(0);
+            m_aCrsrHeight.setY(m_aCharRect.Height());
         }
 
         if( !bFirst && aOld == m_aCharRect )
@@ -1679,8 +1679,8 @@
         bFirst = false;
 
         // update cursor Points to the new Positions
-        pShellCrsr->GetPtPos().X() = m_aCharRect.Left();
-        pShellCrsr->GetPtPos().Y() = m_aCharRect.Top();
+        pShellCrsr->GetPtPos().setX(m_aCharRect.Left());
+        pShellCrsr->GetPtPos().setY(m_aCharRect.Top());
 
         if( !(eFlags & SwCrsrShell::UPDOWN ))   // delete old Pos. of Up/Down
         {
@@ -1773,9 +1773,9 @@
         if( pFrm )
         {
             if( pFrm->IsVertical() )
-                aPt.Y() = pFrm->Frm().Top() + GetUpDownX();
+                aPt.setY(pFrm->Frm().Top() + GetUpDownX());
             else
-                aPt.X() = pFrm->Frm().Left() + GetUpDownX();
+                aPt.setX(pFrm->Frm().Left() + GetUpDownX());
         }
         aMk = rBlock.GetMkPos();
     }
@@ -2284,8 +2284,8 @@
             m_aCrsrHeight = aTmpState.aRealHeight;
         else
         {
-            m_aCrsrHeight.X() = 0;
-            m_aCrsrHeight.Y() = m_aCharRect.Height();
+            m_aCrsrHeight.setX(0);
+            m_aCrsrHeight.setY(m_aCharRect.Height());
         }
 
         m_pVisCrsr->SetDragCrsr( true );
@@ -2623,12 +2623,12 @@
     {
         SwRect aTmp( m_aCharRect );
         long nDiff = m_aCharRect.Height() - VisArea().Height();
-        if( nDiff < m_aCrsrHeight.X() )
+        if( nDiff < m_aCrsrHeight.getX() )
             aTmp.Top( nDiff + m_aCharRect.Top() );
         else
         {
-            aTmp.Top( m_aCrsrHeight.X() + m_aCharRect.Top() );
-            aTmp.Height( m_aCrsrHeight.Y() );
+            aTmp.Top( m_aCrsrHeight.getX() + m_aCharRect.Top() );
+            aTmp.Height( m_aCrsrHeight.getY() );
         }
         if( !aTmp.HasArea() )
         {
diff --git a/sw/source/core/crsr/crstrvl.cxx b/sw/source/core/crsr/crstrvl.cxx
index 6e9d24d..a206d0c 100644
--- a/sw/source/core/crsr/crstrvl.cxx
+++ b/sw/source/core/crsr/crstrvl.cxx
@@ -87,13 +87,13 @@
             pFrm->Calc();
             if( pFrm->IsVertical() )
             {
-                aPt.X() = m_aCharRect.Center().X();
-                aPt.Y() = pFrm->Frm().Top() + m_nUpDownX;
+                aPt.setX(m_aCharRect.Center().getX());
+                aPt.setY(pFrm->Frm().Top() + m_nUpDownX);
             }
             else
             {
-                aPt.Y() = m_aCharRect.Center().Y();
-                aPt.X() = pFrm->Frm().Left() + m_nUpDownX;
+                aPt.setY(m_aCharRect.Center().getY());
+                aPt.setX(pFrm->Frm().Left() + m_nUpDownX);
             }
             pFrm->GetCrsrOfst( m_pCurCrsr->GetPoint(), aPt );
             bRet = !m_pCurCrsr->IsSelOvr( nsSwCursorSelOverFlags::SELOVER_TOGGLE |
@@ -127,13 +127,13 @@
             pFrm->Calc();
             if( pFrm->IsVertical() )
             {
-                aPt.X() = m_aCharRect.Center().X();
-                aPt.Y() = pFrm->Frm().Top() + m_nUpDownX;
+                aPt.setX(m_aCharRect.Center().getX());
+                aPt.setY(pFrm->Frm().Top() + m_nUpDownX);
             }
             else
             {
-                aPt.Y() = m_aCharRect.Center().Y();
-                aPt.X() = pFrm->Frm().Left() + m_nUpDownX;
+                aPt.setY(m_aCharRect.Center().getY());
+                aPt.setX(pFrm->Frm().Left() + m_nUpDownX);
             }
             pFrm->GetCrsrOfst( m_pCurCrsr->GetPoint(), aPt );
             bRet = !m_pCurCrsr->IsSelOvr( nsSwCursorSelOverFlags::SELOVER_TOGGLE |
diff --git a/sw/source/core/crsr/swcrsr.cxx b/sw/source/core/crsr/swcrsr.cxx
index 54715ed..73a9edd 100644
--- a/sw/source/core/crsr/swcrsr.cxx
+++ b/sw/source/core/crsr/swcrsr.cxx
@@ -1774,8 +1774,8 @@
             aPt = aTmpRect.Pos();
 
             nUpDownX = pFrm->IsVertical() ?
-                aPt.Y() - pFrm->Frm().Top() :
-                aPt.X() - pFrm->Frm().Left();
+                aPt.getY() - pFrm->Frm().Top() :
+                aPt.getX() - pFrm->Frm().Left();
         }
 
         // It is allowed to move footnotes in other footnotes but not sections
@@ -1825,15 +1825,15 @@
                 pFrm->GetCharRect( aTmpRect, *GetPoint(), &eTmpState );
                 if ( pFrm->IsVertical() )
                 {
-                    aPt.X() = aTmpRect.Center().X();
+                    aPt.setX(aTmpRect.Center().getX());
                     pFrm->Calc();
-                    aPt.Y() = pFrm->Frm().Top() + nUpDownX;
+                    aPt.setY(pFrm->Frm().Top() + nUpDownX);
                 }
                 else
                 {
-                    aPt.Y() = aTmpRect.Center().Y();
+                    aPt.setY(aTmpRect.Center().getY());
                     pFrm->Calc();
-                    aPt.X() = pFrm->Frm().Left() + nUpDownX;
+                    aPt.setX(pFrm->Frm().Left() + nUpDownX);
                 }
                 pFrm->GetCrsrOfst( GetPoint(), aPt, &eTmpState );
             }
diff --git a/sw/source/core/crsr/trvlcol.cxx b/sw/source/core/crsr/trvlcol.cxx
index ef78656..d45d593 100644
--- a/sw/source/core/crsr/trvlcol.cxx
+++ b/sw/source/core/crsr/trvlcol.cxx
@@ -98,8 +98,8 @@
                 Point aPt( pCnt->Frm().Pos() + pCnt->Prt().Pos() );
                 if( fnPosCol == GetColumnEnd )
                 {
-                    aPt.X() += pCnt->Prt().Width();
-                    aPt.Y() += pCnt->Prt().Height();
+                    aPt.setX(aPt.getX() + pCnt->Prt().Width());
+                    aPt.setY(aPt.getY() + pCnt->Prt().Height());
                 }
 
                 pCnt->GetCrsrOfst( m_pCurCrsr->GetPoint(), aPt );
diff --git a/sw/source/core/crsr/trvlfnfl.cxx b/sw/source/core/crsr/trvlfnfl.cxx
index 3a88cf6..40a17ed 100644
--- a/sw/source/core/crsr/trvlfnfl.cxx
+++ b/sw/source/core/crsr/trvlfnfl.cxx
@@ -322,9 +322,9 @@
         aTmpRect = pFrm->Frm();
     Point aPt( aTmpRect.Left(), aTmpRect.Top() +
                 ( aTmpRect.Bottom() - aTmpRect.Top() ) / 2 );
-    aPt.X() = aPt.X() > (pFrm->Frm().Left() + (pFrm->Frm().SSize().Width() / 2 ))
+    aPt.setX(aPt.getX() > (pFrm->Frm().Left() + (pFrm->Frm().SSize().Width() / 2 ))
                 ? pFrm->Frm().Right()
-                : pFrm->Frm().Left();
+                : pFrm->Frm().Left());
 
     const SwPageFrm* pPageFrm = pFrm->FindPageFrm();
     const SwCntntFrm* pFndFrm = pPageFrm->GetCntntPos( aPt, sal_False, sal_True );
diff --git a/sw/source/core/crsr/viscrs.cxx b/sw/source/core/crsr/viscrs.cxx
index 271d698..7c2d1de 100644
--- a/sw/source/core/crsr/viscrs.cxx
+++ b/sw/source/core/crsr/viscrs.cxx
@@ -104,23 +104,23 @@
 void SwVisCrsr::_SetPosAndShow()
 {
     SwRect aRect;
-    long nTmpY = pCrsrShell->m_aCrsrHeight.Y();
+    long nTmpY = pCrsrShell->m_aCrsrHeight.getY();
     if( 0 > nTmpY )
     {
         nTmpY = -nTmpY;
         aTxtCrsr.SetOrientation( 900 );
         aRect = SwRect( pCrsrShell->m_aCharRect.Pos(),
            Size( pCrsrShell->m_aCharRect.Height(), nTmpY ) );
-        aRect.Pos().X() += pCrsrShell->m_aCrsrHeight.X();
+        aRect.Pos().setX(aRect.Pos().getX() + pCrsrShell->m_aCrsrHeight.getX());
         if( pCrsrShell->IsOverwriteCrsr() )
-            aRect.Pos().Y() += aRect.Width();
+            aRect.Pos().setY(aRect.Pos().getY() + aRect.Width());
     }
     else
     {
         aTxtCrsr.SetOrientation( 0 );
         aRect = SwRect( pCrsrShell->m_aCharRect.Pos(),
            Size( pCrsrShell->m_aCharRect.Width(), nTmpY ) );
-        aRect.Pos().Y() += pCrsrShell->m_aCrsrHeight.X();
+        aRect.Pos().setY(aRect.Pos().getY() + pCrsrShell->m_aCrsrHeight.getX());
     }
 
     // check if cursor should show the current cursor bidi level
diff --git a/sw/source/core/doc/docdraw.cxx b/sw/source/core/doc/docdraw.cxx
index a147595..7419baf 100644
--- a/sw/source/core/doc/docdraw.cxx
+++ b/sw/source/core/doc/docdraw.cxx
@@ -148,23 +148,23 @@
         {
             if ( bR2L ) {
                   //FRMDIR_VERT_TOP_LEFT
-                  nHoriRelPos = aObjRect.Left() - aAnchorPos.X();
-                  nVertRelPos = aObjRect.Top() - aAnchorPos.Y();
+                  nHoriRelPos = aObjRect.Left() - aAnchorPos.getX();
+                  nVertRelPos = aObjRect.Top() - aAnchorPos.getY();
             } else {
                 //FRMDIR_VERT_TOP_RIGHT
-                nHoriRelPos = aObjRect.Top() - aAnchorPos.Y();
-                nVertRelPos = aAnchorPos.X() - aObjRect.Right();
+                nHoriRelPos = aObjRect.Top() - aAnchorPos.getY();
+                nVertRelPos = aAnchorPos.getX() - aObjRect.Right();
             }
         }
         else if ( bR2L )
         {
-            nHoriRelPos = aAnchorPos.X() - aObjRect.Right();
-            nVertRelPos = aObjRect.Top() - aAnchorPos.Y();
+            nHoriRelPos = aAnchorPos.getX() - aObjRect.Right();
+            nVertRelPos = aObjRect.Top() - aAnchorPos.getY();
         }
         else
         {
-            nHoriRelPos = aObjRect.Left() - aAnchorPos.X();
-            nVertRelPos = aObjRect.Top() - aAnchorPos.Y();
+            nHoriRelPos = aObjRect.Left() - aAnchorPos.getX();
+            nVertRelPos = aObjRect.Top() - aAnchorPos.getY();
         }
         //End of SCMS
     }
@@ -246,7 +246,7 @@
             // grouped.
             Point aAnchorPos( pObj->GetAnchorPos() );
             pObj->NbcSetAnchorPos( Point( 0, 0 ) );
-            pObj->NbcMove( Size( aAnchorPos.X(), aAnchorPos.Y() ) );
+            pObj->NbcMove( Size( aAnchorPos.getX(), aAnchorPos.getY() ) );
         }
 
         pFmt = MakeDrawFrmFmt( OUString("DrawObject"),
diff --git a/sw/source/core/doc/docfly.cxx b/sw/source/core/doc/docfly.cxx
index 7ac644e..132480c 100644
--- a/sw/source/core/doc/docfly.cxx
+++ b/sw/source/core/doc/docfly.cxx
@@ -315,7 +315,7 @@
                 aOldH.GetPos() == ((SwFmtHoriOrient*)pItem)->GetPos() ))
             {
                 SwTwips nPos = (FLY_AS_CHAR == nOld) ? 0 : aOldH.GetPos();
-                nPos += aOldAnchorPos.X() - aNewAnchorPos.X();
+                nPos += aOldAnchorPos.getX() - aNewAnchorPos.getX();
 
                 if( pItem )
                 {
@@ -337,7 +337,7 @@
                 aOldV.GetPos() == ((SwFmtVertOrient*)pItem)->GetPos() ) )
             {
                 SwTwips nPos = (FLY_AS_CHAR == nOld) ? 0 : aOldV.GetPos();
-                nPos += aOldAnchorPos.Y() - aNewAnchorPos.Y();
+                nPos += aOldAnchorPos.getY() - aNewAnchorPos.getY();
                 if( pItem )
                 {
                     SwFmtVertOrient* pV = (SwFmtVertOrient*)pItem;
@@ -692,7 +692,7 @@
                         SwCrsrMoveState aState( MV_SETONLYTEXT );
                         SwPosition aPos( GetNodes() );
                         Point aPoint( aPt );
-                        aPoint.X() -= 1;
+                        aPoint.setX(aPoint.getX() - 1);
                         GetCurrentLayout()->GetCrsrOfst( &aPos, aPoint, &aState );
                         // consider that drawing objects can be in
                         // header/footer. Thus, <GetFrm()> by left-top-corner
@@ -746,7 +746,7 @@
 
                     bUnmark = ( 0 != i );
                     Point aPoint( aPt );
-                    aPoint.X() -= 1;    // Do not load in the DrawObj!
+                    aPoint.setX(aPoint.getX() - 1);    // Do not load in the DrawObj!
                     aNewAnch.SetType( FLY_AS_CHAR );
                     SwPosition aPos( *((SwCntntFrm*)pNewAnchorFrm)->GetNode() );
                     if ( pNewAnchorFrm->Frm().IsInside( aPoint ) )
diff --git a/sw/source/core/doc/notxtfrm.cxx b/sw/source/core/doc/notxtfrm.cxx
index ced0f10..28a7515 100644
--- a/sw/source/core/doc/notxtfrm.cxx
+++ b/sw/source/core/doc/notxtfrm.cxx
@@ -307,13 +307,13 @@
 {
     if( nMirror == RES_MIRROR_GRAPH_VERT || nMirror == RES_MIRROR_GRAPH_BOTH )
     {
-        rPt.X() += rDim.Width() -1;
+        rPt.setX(rPt.getX() + rDim.Width() -1);
         rDim.Width() = -rDim.Width();
     }
 
     if( nMirror == RES_MIRROR_GRAPH_HOR || nMirror == RES_MIRROR_GRAPH_BOTH )
     {
-        rPt.Y() += rDim.Height() -1;
+        rPt.setY(rPt.getY() + rDim.Height() -1);
         rDim.Height() = -rDim.Height();
     }
 }
@@ -403,12 +403,12 @@
     // Set the "visible" rectangle first
     if ( nLeftCrop > 0 )
     {
-        aVisPt.X()  += nLeftCrop;
+        aVisPt.setX(aVisPt.getX() + nLeftCrop);
         aVisSz.Width() -= nLeftCrop;
     }
     if ( nTopCrop > 0 )
     {
-        aVisPt.Y()   += nTopCrop;
+        aVisPt.setY(aVisPt.getY() + nTopCrop);
         aVisSz.Height() -= nTopCrop;
     }
     if ( nRightCrop > 0 )
@@ -423,9 +423,9 @@
     if ( pOrigRect )
     {
         Size aTmpSz( aGrfSz );
-        aGrfPt.X()    += nLeftCrop;
+        aGrfPt.setX(aGrfPt.getX() + nLeftCrop);
         aTmpSz.Width() -= nLeftCrop + nRightCrop;
-        aGrfPt.Y()      += nTopCrop;
+        aGrfPt.setY(aGrfPt.getY() + nTopCrop);
         aTmpSz.Height()-= nTopCrop + nBottomCrop;
 
         if( RES_MIRROR_GRAPH_DONT != nMirror )
@@ -511,8 +511,8 @@
     {
         if ( pCMS->bRealHeight )
         {
-            pCMS->aRealHeight.Y() = rRect.Height();
-            pCMS->aRealHeight.X() = 0;
+            pCMS->aRealHeight.setY(rRect.Height());
+            pCMS->aRealHeight.setX(0);
         }
     }
 
diff --git a/sw/source/core/docnode/ndnotxt.cxx b/sw/source/core/docnode/ndnotxt.cxx
index ccdaf72..916a70f 100644
--- a/sw/source/core/docnode/ndnotxt.cxx
+++ b/sw/source/core/docnode/ndnotxt.cxx
@@ -160,8 +160,8 @@
                         // #i102238#
                         if ( nGrfDPIx != 0 && nGrfDPIy != 0 )
                         {
-                            rPoly[i] = Point( rPoly[i].X() * pOutDev->ImplGetDPIX() / nGrfDPIx,
-                                              rPoly[i].Y() * pOutDev->ImplGetDPIY() / nGrfDPIy );
+                            rPoly[i] = Point( rPoly[i].getX() * pOutDev->ImplGetDPIX() / nGrfDPIx,
+                                              rPoly[i].getY() * pOutDev->ImplGetDPIY() / nGrfDPIy );
                         }
                     }
                     else
diff --git a/sw/source/core/draw/dflyobj.cxx b/sw/source/core/draw/dflyobj.cxx
index 0733af1..18337b2 100644
--- a/sw/source/core/draw/dflyobj.cxx
+++ b/sw/source/core/draw/dflyobj.cxx
@@ -879,8 +879,8 @@
 
         if( aRect.IsInside( rRec.aPos ) )
         {
-            aRect.Pos().X() += rRec.nTol;
-            aRect.Pos().Y() += rRec.nTol;
+            aRect.Pos().setX(aRect.Pos().getX() + rRec.nTol);
+            aRect.Pos().setY(aRect.Pos().getY() + rRec.nTol);
             aRect.SSize().Height()-= 2 * rRec.nTol;
             aRect.SSize().Width() -= 2 * rRec.nTol;
 
diff --git a/sw/source/core/draw/dpage.cxx b/sw/source/core/draw/dpage.cxx
index cee0e0d..ff1fcde 100644
--- a/sw/source/core/draw/dpage.cxx
+++ b/sw/source/core/draw/dpage.cxx
@@ -167,8 +167,8 @@
                     // without MapMode-Offset, without Offset, w ... !!!!!
                     aPt = pWindow->LogicToPixel(
                             aPt, MapMode( MAP_TWIP ) );
-                    ((( sTxt += '?' ) += OUString::number( aPt.X() ))
-                             += ',' ) += OUString::number( aPt.Y() );
+                    ((( sTxt += '?' ) += OUString::number( aPt.getX() ))
+                             += ',' ) += OUString::number( aPt.getY() );
                 }
             }
 
diff --git a/sw/source/core/draw/dview.cxx b/sw/source/core/draw/dview.cxx
index 694f915..6cbd572 100644
--- a/sw/source/core/draw/dview.cxx
+++ b/sw/source/core/draw/dview.cxx
@@ -730,7 +730,7 @@
         }
         else if ( !bFly )
         {
-            const SwRect aRect( aPt.X(), aPt.Y(), 1, 1 );
+            const SwRect aRect( aPt.getX(), aPt.getY(), 1, 1 );
 
             SwDrawContact* pContact = (SwDrawContact*)GetUserCall(pObj);
             if ( pContact->GetAnchorFrm( pObj ) &&
diff --git a/sw/source/core/frmedt/fecopy.cxx b/sw/source/core/frmedt/fecopy.cxx
index 0bedb73..3fa4a73 100644
--- a/sw/source/core/frmedt/fecopy.cxx
+++ b/sw/source/core/frmedt/fecopy.cxx
@@ -396,8 +396,8 @@
                         aPos -= rSttPt - pObj->GetSnapRect().TopLeft();
                         // OD 2004-04-05 #i26791# - change attributes instead of
                         // direct positioning
-                        pFmt->SetFmtAttr( SwFmtHoriOrient( aPos.X(), text::HoriOrientation::NONE, text::RelOrientation::FRAME ) );
-                        pFmt->SetFmtAttr( SwFmtVertOrient( aPos.Y(), text::VertOrientation::NONE, text::RelOrientation::FRAME ) );
+                        pFmt->SetFmtAttr( SwFmtHoriOrient( aPos.getX(), text::HoriOrientation::NONE, text::RelOrientation::FRAME ) );
+                        pFmt->SetFmtAttr( SwFmtVertOrient( aPos.getY(), text::VertOrientation::NONE, text::RelOrientation::FRAME ) );
                         // #i47455# - notify draw frame format
                         // that position attributes are already set.
                         if ( pFmt->ISA(SwDrawFrmFmt) )
@@ -541,8 +541,8 @@
                 Point aPos( rInsPt );
                 aPos -= aNewAnch;
                 aPos -= rSttPt - pFly->Frm().Pos();
-                pFlyFmt->SetFmtAttr( SwFmtHoriOrient( aPos.X(),text::HoriOrientation::NONE, text::RelOrientation::FRAME ) );
-                pFlyFmt->SetFmtAttr( SwFmtVertOrient( aPos.Y(),text::VertOrientation::NONE, text::RelOrientation::FRAME ) );
+                pFlyFmt->SetFmtAttr( SwFmtHoriOrient( aPos.getX(),text::HoriOrientation::NONE, text::RelOrientation::FRAME ) );
+                pFlyFmt->SetFmtAttr( SwFmtVertOrient( aPos.getY(),text::VertOrientation::NONE, text::RelOrientation::FRAME ) );
             }
 
             const Point aPt( pDestShell->GetCrsrDocPos() );
@@ -1401,7 +1401,7 @@
                 pNewObj->NbcResize( aNewRect.TopLeft(), aScaleWidth, aScaleHeight);
 
                 Point aVec = aOldObjRect.TopLeft() - aNewRect.TopLeft();
-                pNewObj->NbcMove(Size(aVec.X(), aVec.Y()));
+                pNewObj->NbcMove(Size(aVec.getX(), aVec.getY()));
 
                 if( pNewObj->ISA( SdrUnoObj ) )
                     pNewObj->SetLayer( GetDoc()->GetControlsId() );
diff --git a/sw/source/core/frmedt/fefly1.cxx b/sw/source/core/frmedt/fefly1.cxx
index ed33b5c..7fc3756 100644
--- a/sw/source/core/frmedt/fefly1.cxx
+++ b/sw/source/core/frmedt/fefly1.cxx
@@ -138,7 +138,7 @@
                 // textnode should be found, as only in those
                 // a content bound frame can be anchored
                 SwCrsrMoveState aState( MV_SETONLYTEXT );
-                aTmpPnt.X() -= 1;                   // do not land in the fly!
+                aTmpPnt.setX(aTmpPnt.getX() - 1);                   // do not land in the fly!
                 if( !pNewAnch->GetCrsrOfst( &aPos, aTmpPnt, &aState ) )
                 {
                     SwCntntNode* pCNd = ((SwCntntFrm*)pNewAnch)->GetNode();
@@ -158,7 +158,7 @@
             // search nearest SwFlyFrm
             SwCrsrMoveState aState( MV_SETONLYTEXT );
             SwPosition aPos( rDoc.GetNodes() );
-            aTmpPnt.X() -= 1;                   // do not land in the fly!
+            aTmpPnt.setX(aTmpPnt.getX() - 1);                   // do not land in the fly!
             rDoc.GetCurrentLayout()->GetCrsrOfst( &aPos, aTmpPnt, &aState );    //swmod 071108//swmod 071225
             pNewAnch = ::FindAnchor(
                 aPos.nNode.GetNode().GetCntntNode()->getLayoutFrm( rFrm.getRootFrm(), 0, 0, sal_False ),
@@ -325,7 +325,7 @@
         SwNodeIndex aSwNodeIndex( GetDoc()->GetNodes() );
         SwPosition aPos( aSwNodeIndex );
         Point aPoint( aTmpPos );
-        aPoint.X() -= 1;                    //do not land in the fly!!
+        aPoint.setX(aPoint.getX() - 1);                    //do not land in the fly!!
         GetLayout()->GetCrsrOfst( &aPos, aPoint, &aState );
         // determine text frame by left-top-corner of object
         pTxtFrm = aPos.nNode.GetNode().GetCntntNode()->getLayoutFrm( GetLayout(), &aTmpPos, 0, sal_False );
@@ -367,11 +367,11 @@
             Point aOrient( pAnch->Frm().Pos() );
 
         if ( pFly->IsFlyInCntFrm() )
-            aOrient.X() = rAbsPos.X();
+            aOrient.setX(rAbsPos.getX());
 
         // calculate RelPos.
-        aOrient.X() = rAbsPos.X() - aOrient.X();
-        aOrient.Y() = rAbsPos.Y() - aOrient.Y();
+        aOrient.setX(rAbsPos.getX() - aOrient.getX());
+        aOrient.setY(rAbsPos.getY() - aOrient.getY());
         pFly->ChgRelPos( aOrient );
     }
     CallChgLnk();       // call the AttrChangeNotify on the UI-side.
@@ -1436,8 +1436,8 @@
     if ( rRect.Top() != LONG_MIN && rRect.Pos() != aPt && !bPosProt )
     {
         aPt = rRect.Pos();
-        aPt.X() -= pFly->Prt().Left();
-        aPt.Y() -= pFly->Prt().Top();
+        aPt.setX(aPt.getX() - pFly->Prt().Left());
+        aPt.setY(aPt.getY() - pFly->Prt().Top());
 
         // in case of paragraph-bound Flys, starting from the new position,
         // a new anchor is to be set. The anchor and the new RelPos are
@@ -1449,8 +1449,8 @@
             const SwFrmFmt *pFmt = pFly->GetFmt();
             const SwFmtVertOrient &rVert = pFmt->GetVertOrient();
             const SwFmtHoriOrient &rHori = pFmt->GetHoriOrient();
-            const long lXDiff = aPt.X() - pFly->Frm().Left();
-            const long lYDiff = aPt.Y() - pFly->Frm().Top();
+            const long lXDiff = aPt.getX() - pFly->Frm().Left();
+            const long lYDiff = aPt.getY() - pFly->Frm().Top();
             const Point aTmp( rHori.GetPos() + lXDiff,
                               rVert.GetPos() + lYDiff );
             pFly->ChgRelPos( aTmp );
@@ -1601,8 +1601,8 @@
                         // without MapMode-Offset, without Offset, o ... !!!!!
                         aPt = GetOut()->LogicToPixel(
                                 aPt, MapMode( MAP_TWIP ) );
-                        ((( *pURL += '?' ) += OUString::number( aPt.X() ))
-                                  += ',' ) += OUString::number(aPt.Y() );
+                        ((( *pURL += '?' ) += OUString::number( aPt.getX() ))
+                                  += ',' ) += OUString::number(aPt.getY() );
                     }
                 }
                 pRet = pFly->GetFmt();
@@ -1865,10 +1865,10 @@
                                 std::max( nHeight, long(MINFLY) )));
 
             if( SFX_ITEM_SET != aFrmSet.GetItemState( RES_HORI_ORIENT ))
-                aFrmSet.Put( SwFmtHoriOrient( aRelPos.X(), text::HoriOrientation::NONE, text::RelOrientation::FRAME ));
+                aFrmSet.Put( SwFmtHoriOrient( aRelPos.getX(), text::HoriOrientation::NONE, text::RelOrientation::FRAME ));
 
             if( SFX_ITEM_SET != aFrmSet.GetItemState( RES_VERT_ORIENT ))
-                aFrmSet.Put( SwFmtVertOrient( aRelPos.Y(), text::VertOrientation::NONE, text::RelOrientation::FRAME ));
+                aFrmSet.Put( SwFmtVertOrient( aRelPos.getY(), text::VertOrientation::NONE, text::RelOrientation::FRAME ));
 
         }
 
diff --git a/sw/source/core/frmedt/feshview.cxx b/sw/source/core/frmedt/feshview.cxx
index 4c97979..f477af7 100644
--- a/sw/source/core/frmedt/feshview.cxx
+++ b/sw/source/core/frmedt/feshview.cxx
@@ -170,7 +170,7 @@
                     // For simplicity we put the cursor next to the upper-left
                     // corner.
                     Point aPt( pOldSelFly->Frm().Pos() );
-                    aPt.X() -= 1;
+                    aPt.setX(aPt.getX() - 1);
                     bool bUnLockView = !IsViewLocked();
                     LockView( sal_True );
                     SetCrsr( aPt, sal_True );
@@ -266,12 +266,12 @@
 |*
 *************************************************************************/
 
-#define LESS_X( aPt1, aPt2, bOld ) ( aPt1.X() < aPt2.X() || \
-        ( aPt1.X() == aPt2.X() && ( aPt1.Y() < aPt2.Y() || \
-        ( aPt1.Y() == aPt2.Y() && bOld ) ) ) )
-#define LESS_Y( aPt1, aPt2, bOld ) ( aPt1.Y() < aPt2.Y() || \
-        ( aPt1.Y() == aPt2.Y() && ( aPt1.X() < aPt2.X() || \
-        ( aPt1.X() == aPt2.X() && bOld ) ) ) )
+#define LESS_X( aPt1, aPt2, bOld ) ( aPt1.getX() < aPt2.getX() || \
+        ( aPt1.getX() == aPt2.getX() && ( aPt1.getY() < aPt2.getY() || \
+        ( aPt1.getY() == aPt2.getY() && bOld ) ) ) )
+#define LESS_Y( aPt1, aPt2, bOld ) ( aPt1.getY() < aPt2.getY() || \
+        ( aPt1.getY() == aPt2.getY() && ( aPt1.getX() < aPt2.getX() || \
+        ( aPt1.getX() == aPt2.getX() && bOld ) ) ) )
 
 sal_Bool SwFEShell::MoveAnchor( sal_uInt16 nDir )
 {
@@ -1390,9 +1390,9 @@
 
             // Special case if another object is on same Y.
             if( aCurPos != aPos &&          // only when it is not me
-                aCurPos.Y() == aPos.Y() &&  // Y positions equal
-                (bNext? (aCurPos.X() > aPos.X()) :  // lies next to me
-                        (aCurPos.X() < aPos.X())) ) // " reverse
+                aCurPos.getY() == aPos.getY() &&  // Y positions equal
+                (bNext? (aCurPos.getX() > aPos.getX()) :  // lies next to me
+                        (aCurPos.getX() < aPos.getX())) ) // " reverse
             {
                 aBestPos = Point( nTmp, nTmp );
                 SdrObjListIter aTmpIter( *pList, bFlat ? IM_FLAT : IM_DEEPNOGROUPS );
@@ -1411,10 +1411,10 @@
                         aCurPos = pTmpObj->GetCurrentBoundRect().TopLeft();
 
                     if( aCurPos != aPos && aCurPos.Y() == aPos.Y() &&
-                        (bNext? (aCurPos.X() > aPos.X()) :  // lies next to me
-                                (aCurPos.X() < aPos.X())) &&    // " reverse
-                        (bNext? (aCurPos.X() < aBestPos.X()) :  // better as best
-                                (aCurPos.X() > aBestPos.X())) ) // " reverse
+                        (bNext? (aCurPos.getX() > aPos.getX()) :  // lies next to me
+                                (aCurPos.getX() < aPos.getX())) &&    // " reverse
+                        (bNext? (aCurPos.getX() < aBestPos.getX()) :  // better as best
+                                (aCurPos.getX() > aBestPos.getX())) ) // " reverse
                     {
                         aBestPos = aCurPos;
                         pBest = pTmpObj;
@@ -1424,32 +1424,32 @@
             }
 
             if( (
-                (bNext? (aPos.Y() < aCurPos.Y()) :          // only below me
-                        (aPos.Y() > aCurPos.Y())) &&        // " reverse
-                (bNext? (aBestPos.Y() > aCurPos.Y()) :      // closer below
-                        (aBestPos.Y() < aCurPos.Y()))
+                (bNext? (aPos.getY() < aCurPos.getY()) :          // only below me
+                        (aPos.getY() > aCurPos.getY())) &&        // " reverse
+                (bNext? (aBestPos.getY() > aCurPos.getY()) :      // closer below
+                        (aBestPos.getY() < aCurPos.getY()))
                     ) ||    // " reverse
-                        (aBestPos.Y() == aCurPos.Y() &&
-                (bNext? (aBestPos.X() > aCurPos.X()) :      // further left
-                        (aBestPos.X() < aCurPos.X()))))     // " reverse
+                        (aBestPos.getY() == aCurPos.getY() &&
+                (bNext? (aBestPos.getX() > aCurPos.getX()) :      // further left
+                        (aBestPos.getX() < aCurPos.getX()))))     // " reverse
 
             {
                 aBestPos = aCurPos;
                 pBest = pObj;
             }
 
-            if( (bNext? (aTopPos.Y() > aCurPos.Y()) :       // higher as best
-                        (aTopPos.Y() < aCurPos.Y())) ||     // " reverse
-                        (aTopPos.Y() == aCurPos.Y() &&
-                (bNext? (aTopPos.X() > aCurPos.X()) :       // further left
-                        (aTopPos.X() < aCurPos.X()))))      // " reverse
+            if( (bNext? (aTopPos.getY() > aCurPos.getY()) :       // higher as best
+                        (aTopPos.getY() < aCurPos.getY())) ||     // " reverse
+                        (aTopPos.getY() == aCurPos.getY() &&
+                (bNext? (aTopPos.getX() > aCurPos.getX()) :       // further left
+                        (aTopPos.getX() < aCurPos.getX()))))      // " reverse
             {
                 aTopPos = aCurPos;
                 pTop = pObj;
             }
         }
         // unfortunately nothing found
-        if( (bNext? (aBestPos.X() == LONG_MAX) : (aBestPos.X() == 0)) )
+        if( (bNext? (aBestPos.getX() == LONG_MAX) : (aBestPos.getX() == 0)) )
             pBest = pTop;
     }
 
@@ -1630,7 +1630,7 @@
     {
         SwPosition aPos( GetDoc()->GetNodes() );
         SwCrsrMoveState aState( MV_SETONLYTEXT );
-        Point aPoint( aPt.X(), aPt.Y() + rBound.GetHeight()/2 );
+        Point aPoint( aPt.getX(), aPt.getY() + rBound.GetHeight()/2 );
         GetLayout()->GetCrsrOfst( &aPos, aPoint, &aState ); //swmod 080317
 
         // characterbinding not allowed in readonly-content
@@ -2864,13 +2864,13 @@
 
                     aInnerPoly.append(basegfx::B2DPoint(aRect.Left(), aRect.Bottom()));
 
-                    const basegfx::B2DPoint aCenterBottom(aRect.Center().X(), aRect.Bottom());
+                    const basegfx::B2DPoint aCenterBottom(aRect.Center().getX(), aRect.Bottom());
                     aInnerPoly.appendBezierSegment(
                         aCenterBottom,
                         aCenterBottom,
-                        basegfx::B2DPoint(aRect.Center().X(), aRect.Center().Y()));
+                        basegfx::B2DPoint(aRect.Center().getX(), aRect.Center().getY()));
 
-                    const basegfx::B2DPoint aCenterTop(aRect.Center().X(), aRect.Top());
+                    const basegfx::B2DPoint aCenterTop(aRect.Center().getX(), aRect.Top());
                     aInnerPoly.appendBezierSegment(
                         aCenterTop,
                         aCenterTop,
@@ -2888,11 +2888,11 @@
 
                     aInnerPoly.appendBezierSegment(
                         basegfx::B2DPoint(aRect.Left(), aRect.Top()),
-                        basegfx::B2DPoint(aRect.Center().X(), aRect.Top()),
-                        basegfx::B2DPoint(aRect.Center().X(), aRect.Center().Y()));
+                        basegfx::B2DPoint(aRect.Center().getX(), aRect.Top()),
+                        basegfx::B2DPoint(aRect.Center().getX(), aRect.Center().getY()));
 
                     aInnerPoly.appendBezierSegment(
-                        basegfx::B2DPoint(aRect.Center().X(), aRect.Bottom()),
+                        basegfx::B2DPoint(aRect.Center().getX(), aRect.Bottom()),
                         basegfx::B2DPoint(aRect.Right(), aRect.Bottom()),
                         basegfx::B2DPoint(aRect.Right(), aRect.Top()));
 
@@ -2919,7 +2919,7 @@
 
                     if(OBJ_PLIN == eSdrObjectKind)
                     {
-                        aInnerPoly.append(basegfx::B2DPoint(aRect.Center().X(), aRect.Bottom()));
+                        aInnerPoly.append(basegfx::B2DPoint(aRect.Center().getX(), aRect.Bottom()));
                     }
                     else
                     {
@@ -2933,8 +2933,8 @@
                 {
                     sal_Int32 nYMiddle((aRect.Top() + aRect.Bottom()) / 2);
                     basegfx::B2DPolygon aTempPoly;
-                    aTempPoly.append(basegfx::B2DPoint(aRect.TopLeft().X(), nYMiddle));
-                    aTempPoly.append(basegfx::B2DPoint(aRect.BottomRight().X(), nYMiddle));
+                    aTempPoly.append(basegfx::B2DPoint(aRect.TopLeft().getX(), nYMiddle));
+                    aTempPoly.append(basegfx::B2DPoint(aRect.BottomRight().getX(), nYMiddle));
                     aPoly.append(aTempPoly);
                 }
                 break;
diff --git a/sw/source/core/frmedt/fetab.cxx b/sw/source/core/frmedt/fetab.cxx
index bbf7516..2442438 100644
--- a/sw/source/core/frmedt/fetab.cxx
+++ b/sw/source/core/frmedt/fetab.cxx
@@ -1807,19 +1807,19 @@
     // project onto closest line:
     if ( bLeft || bRight )
     {
-        aRet.X() = aMin1.X();
-        if ( aRet.Y() > aMin2.Y() )
-            aRet.Y() = aMin2.Y();
-        else if ( aRet.Y() < aMin1.Y() )
-            aRet.Y() = aMin1.Y();
+        aRet.setX(aMin1.getX());
+        if ( aRet.getY() > aMin2.getY() )
+            aRet.setY(aMin2.getY());
+        else if ( aRet.getY() < aMin1.getY() )
+            aRet.setY(aMin1.getY());
     }
     else
     {
-        aRet.Y() = aMin1.Y();
-        if ( aRet.X() > aMin2.X() )
-            aRet.X() = aMin2.X();
-        else if ( aRet.X() < aMin1.X() )
-            aRet.X() = aMin1.X();
+        aRet.setY(aMin1.getY());
+        if ( aRet.getX() > aMin2.getX() )
+            aRet.setX(aMin2.getX());
+        else if ( aRet.getX() < aMin1.getX() )
+            aRet.setX(aMin1.getX());
     }
 
     return aRet;
diff --git a/sw/source/core/frmedt/fews.cxx b/sw/source/core/frmedt/fews.cxx
index cb89b35..4593b69 100644
--- a/sw/source/core/frmedt/fews.cxx
+++ b/sw/source/core/frmedt/fews.cxx
@@ -933,7 +933,7 @@
                         aPos.X() = aVertEnvironRect.Right();
                         if ( _eVertRelOrient == text::RelOrientation::PAGE_PRINT_AREA )
                         {
-                            aPos.X() -= rVertEnvironLayFrm.GetRightMargin();
+                            aPos.setX(aPos.getX() - rVertEnvironLayFrm.GetRightMargin());
                         }
                     }
                     else if ( bVertL2R )
@@ -941,7 +941,7 @@
                         aPos.X() = aVertEnvironRect.Left();
                         if ( _eVertRelOrient == text::RelOrientation::PAGE_PRINT_AREA )
                         {
-                            aPos.X() += rVertEnvironLayFrm.GetLeftMargin();
+                            aPos.setX(aPos.getX() + rVertEnvironLayFrm.GetLeftMargin());
                         }
                     }
                     else
@@ -949,12 +949,12 @@
                         aPos.Y() = aVertEnvironRect.Top();
                         if ( _eVertRelOrient == text::RelOrientation::PAGE_PRINT_AREA )
                         {
-                            aPos.Y() += rVertEnvironLayFrm.GetTopMargin();
+                            aPos.setY(aPos.getY() + rVertEnvironLayFrm.GetTopMargin());
                             // add height of page header
                             const SwFrm* pTmpFrm = rVertEnvironLayFrm.Lower();
                             if ( pTmpFrm->IsHeaderFrm() )
                             {
-                                aPos.Y() += pTmpFrm->Frm().Height();
+                                aPos.setY(aPos.getY() + pTmpFrm->Frm().Height());
                             }
                         }
                     }
@@ -1004,11 +1004,11 @@
                 }
                 if ( bVert || bVertL2R )
                 {
-                    aPos.X() = nTop;
+                    aPos.setX(nTop);
                 }
                 else
                 {
-                    aPos.Y() = nTop;
+                    aPos.setY(nTop);
                 }
             }
 
@@ -1034,11 +1034,11 @@
                 nLeft = (aChRect.*fnRect->fnGetLeft)();
                 if ( bVert || bVertL2R )
                 {
-                    aPos.Y() = nLeft;
+                    aPos.setY(nLeft);
                 }
                 else
                 {
-                    aPos.X() = nLeft;
+                    aPos.setX(nLeft);
                 }
             }
             if ( bVert || bVertL2R )
@@ -1095,35 +1095,35 @@
             {
                 case text::RelOrientation::FRAME_RIGHT:
                 {
-                    aPos.Y() += pFrm->Prt().Height();
+                    aPos.setY(aPos.getY() + pFrm->Prt().Height());
                     aPos += (pFrm->Prt().*fnRect->fnGetPos)();
                     break;
                 }
                 case text::RelOrientation::PRINT_AREA:
                 {
                     aPos += (pFrm->Prt().*fnRect->fnGetPos)();
-                    aPos.Y() += nBaseOfstForFly;
+                    aPos.setY(aPos.getY() + nBaseOfstForFly);
                     break;
                 }
                 case text::RelOrientation::PAGE_RIGHT:
                 {
-                    aPos.Y() = pPage->Frm().Top() + pPage->Prt().Bottom();
+                    aPos.setY(pPage->Frm().Top() + pPage->Prt().Bottom());
                     break;
                 }
                 case text::RelOrientation::PAGE_PRINT_AREA:
                 {
-                    aPos.Y() = pPage->Frm().Top() + pPage->Prt().Top();
+                    aPos.setY(pPage->Frm().Top() + pPage->Prt().Top());
                     break;
                 }
                 case text::RelOrientation::PAGE_LEFT:
                 case text::RelOrientation::PAGE_FRAME:
                 {
-                    aPos.Y() = pPage->Frm().Top();
+                    aPos.setY(pPage->Frm().Top());
                     break;
                 }
                 case text::RelOrientation::FRAME:
                 {
-                    aPos.Y() += nBaseOfstForFly;
+                    aPos.setY(aPos.getY() + nBaseOfstForFly);
                     break;
                 }
                 default: break;
@@ -1133,14 +1133,14 @@
         {
             switch ( _eHoriRelOrient )
             {
-                case text::RelOrientation::FRAME_RIGHT:   aPos.X() += pFrm->Prt().Left(); break;
+                case text::RelOrientation::FRAME_RIGHT:   aPos.setX(aPos.getX() + pFrm->Prt().Left()); break;
                 case text::RelOrientation::FRAME:
-                case text::RelOrientation::FRAME_LEFT: aPos.X() += pFrm->Frm().Width(); break;
-                case text::RelOrientation::PRINT_AREA: aPos.X() += pFrm->Prt().Right(); break;
+                case text::RelOrientation::FRAME_LEFT: aPos.setX(aPos.getX() + pFrm->Frm().Width()); break;
+                case text::RelOrientation::PRINT_AREA: aPos.setX(aPos.getX() + pFrm->Prt().Right()); break;
                 case text::RelOrientation::PAGE_LEFT:
-                case text::RelOrientation::PAGE_FRAME: aPos.X() = pPage->Frm().Right(); break;
-                case text::RelOrientation::PAGE_PRINT_AREA: aPos.X() = pPage->Frm().Left()
-                                              + pPage->Prt().Left(); break;
+                case text::RelOrientation::PAGE_FRAME: aPos.setX(pPage->Frm().Right()); break;
+                case text::RelOrientation::PAGE_PRINT_AREA: aPos.setX(pPage->Frm().Left()
+                                              + pPage->Prt().Left()); break;
                 default: break;
             }
         }
@@ -1149,32 +1149,32 @@
             switch ( _eHoriRelOrient )
             {
                 case text::RelOrientation::FRAME_LEFT:
-                    aPos.X() = pFrm->Frm().Left() +
-                               pFrm->Prt().Left();
+                    aPos.setX(pFrm->Frm().Left() +
+                               pFrm->Prt().Left());
                     break;
 
                 case text::RelOrientation::PRINT_AREA:
-                    aPos.X() = pFrm->Frm().Left() + pFrm->Prt().Left() +
-                               pFrm->Prt().Width();
-                    aPos.X() += nBaseOfstForFly;
+                    aPos.setX(pFrm->Frm().Left() + pFrm->Prt().Left() +
+                               pFrm->Prt().Width());
+                    aPos.setX(aPos.getX() + nBaseOfstForFly);
                     break;
 
                 case text::RelOrientation::PAGE_LEFT:
-                    aPos.X() = pPage->Frm().Left() + pPage->Prt().Left();
+                    aPos.setX(pPage->Frm().Left() + pPage->Prt().Left());
                     break;
 
                 case text::RelOrientation::PAGE_PRINT_AREA:
-                    aPos.X() = pPage->Frm().Left() + pPage->Prt().Left() +
-                               pPage->Prt().Width() ;
+                    aPos.setX(pPage->Frm().Left() + pPage->Prt().Left() +
+                               pPage->Prt().Width());
                     break;
 
                 case text::RelOrientation::PAGE_RIGHT:
                 case text::RelOrientation::PAGE_FRAME:
-                    aPos.X() = pPage->Frm().Right();
+                    aPos.setX(pPage->Frm().Right());
                     break;
 
                 case text::RelOrientation::FRAME:
-                    aPos.X() += nBaseOfstForFly;
+                    aPos.setX(aPos.getX() + nBaseOfstForFly);
                     break;
                 default: break;
             }
@@ -1189,20 +1189,20 @@
                     break;
                 case text::RelOrientation::PRINT_AREA:
                     aPos += pFrm->Prt().Pos();
-                    aPos.X() += nBaseOfstForFly;
+                    aPos.setX(aPos.getX() + nBaseOfstForFly);
                     break;
                 case text::RelOrientation::PAGE_RIGHT:
-                    aPos.X() = pPage->Frm().Left() + pPage->Prt().Right();
+                    aPos.setX(pPage->Frm().Left() + pPage->Prt().Right());
                     break;
                 case text::RelOrientation::PAGE_PRINT_AREA:
-                    aPos.X() = pPage->Frm().Left() + pPage->Prt().Left();
+                    aPos.setX(pPage->Frm().Left() + pPage->Prt().Left());
                     break;
                 case text::RelOrientation::PAGE_LEFT:
                 case text::RelOrientation::PAGE_FRAME:
-                    aPos.X() = pPage->Frm().Left();
+                    aPos.setX(pPage->Frm().Left());
                     break;
                 case text::RelOrientation::FRAME:
-                    aPos.X() += nBaseOfstForFly;
+                    aPos.setX(aPos.getX() + nBaseOfstForFly);
                     break;
                 default: break;
             }
@@ -1212,13 +1212,13 @@
     if( !_opRef )
     {
         if( bVertic && !bVerticalL2R )
-            _orRect.Pos( aPos.X() - _orRect.Width() - _orRect.Left(), _orRect.Top() - aPos.Y() );
+            _orRect.Pos( aPos.getX() - _orRect.Width() - _orRect.Left(), _orRect.Top() - aPos.getY() );
         else if( bVerticalL2R )
-            _orRect.Pos( _orRect.Left() - aPos.X(), _orRect.Top() - aPos.Y() );
+            _orRect.Pos( _orRect.Left() - aPos.getX(), _orRect.Top() - aPos.getY() );
         else if ( bRTL )
-            _orRect.Pos( - ( _orRect.Right() - aPos.X() ), _orRect.Top() - aPos.Y() );
+            _orRect.Pos( - ( _orRect.Right() - aPos.getX() ), _orRect.Top() - aPos.getY() );
         else
-            _orRect.Pos( _orRect.Left() - aPos.X(), _orRect.Top() - aPos.Y() );
+            _orRect.Pos( _orRect.Left() - aPos.getX(), _orRect.Top() - aPos.getY() );
         if( _bMirror )
             _orRect.Pos( -_orRect.Right(), _orRect.Top() );
     }
diff --git a/sw/source/core/frmedt/tblsel.cxx b/sw/source/core/frmedt/tblsel.cxx
index f0687c8..48c2520 100644
--- a/sw/source/core/frmedt/tblsel.cxx
+++ b/sw/source/core/frmedt/tblsel.cxx
@@ -329,33 +329,33 @@
                                 const Point aBottomLeft( pCell->Frm().BottomLeft() );
                                 const Point aBottomRight( pCell->Frm().BottomRight() );
 
-                                if ( aTopLeft.Y() < aCurrentTopLeft.Y() ||
-                                     ( aTopLeft.Y() == aCurrentTopLeft.Y() &&
-                                       aTopLeft.X() <  aCurrentTopLeft.X() ) )
+                                if ( aTopLeft.getY() < aCurrentTopLeft.getY() ||
+                                     ( aTopLeft.getY() == aCurrentTopLeft.getY() &&
+                                       aTopLeft.getX() <  aCurrentTopLeft.getX() ) )
                                 {
                                     aCurrentTopLeft = aTopLeft;
                                     pCurrentTopLeftFrm = static_cast<const SwCellFrm*>( pCell );
                                 }
 
-                                if ( aTopRight.Y() < aCurrentTopRight.Y() ||
-                                     ( aTopRight.Y() == aCurrentTopRight.Y() &&
-                                       aTopRight.X() >  aCurrentTopRight.X() ) )
+                                if ( aTopRight.getY() < aCurrentTopRight.getY() ||
+                                     ( aTopRight.getY() == aCurrentTopRight.getY() &&
+                                       aTopRight.getX() >  aCurrentTopRight.getX() ) )
                                 {
                                     aCurrentTopRight = aTopRight;
                                     pCurrentTopRightFrm = static_cast<const SwCellFrm*>( pCell );
                                 }
 
-                                if ( aBottomLeft.Y() > aCurrentBottomLeft.Y() ||
-                                     ( aBottomLeft.Y() == aCurrentBottomLeft.Y() &&
-                                       aBottomLeft.X() <  aCurrentBottomLeft.X() ) )
+                                if ( aBottomLeft.getY() > aCurrentBottomLeft.getY() ||
+                                     ( aBottomLeft.getY() == aCurrentBottomLeft.getY() &&
+                                       aBottomLeft.getX() <  aCurrentBottomLeft.getX() ) )
                                 {
                                     aCurrentBottomLeft = aBottomLeft;
                                     pCurrentBottomLeftFrm = static_cast<const SwCellFrm*>( pCell );
                                 }
 
-                                if ( aBottomRight.Y() > aCurrentBottomRight.Y() ||
-                                     ( aBottomRight.Y() == aCurrentBottomRight.Y() &&
-                                       aBottomRight.X() >  aCurrentBottomRight.X() ) )
+                                if ( aBottomRight.getY() > aCurrentBottomRight.getY() ||
+                                     ( aBottomRight.getY() == aCurrentBottomRight.getY() &&
+                                       aBottomRight.getX() >  aCurrentBottomRight.getX() ) )
                                 {
                                     aCurrentBottomRight = aBottomRight;
                                     pCurrentBottomRightFrm = static_cast<const SwCellFrm*>( pCell );
diff --git a/sw/source/core/layout/anchoreddrawobject.cxx b/sw/source/core/layout/anchoreddrawobject.cxx
index 574cc7c..62a3dea 100644
--- a/sw/source/core/layout/anchoreddrawobject.cxx
+++ b/sw/source/core/layout/anchoreddrawobject.cxx
@@ -528,8 +528,8 @@
     if ( aNewAnchorPos != aCurrAnchorPos )
     {
         // determine movement to be applied after setting the new anchor position
-        Size aMove( aCurrAnchorPos.X() - aNewAnchorPos.X(),
-                    aCurrAnchorPos.Y() - aNewAnchorPos.Y() );
+        Size aMove( aCurrAnchorPos.getX() - aNewAnchorPos.getX(),
+                    aCurrAnchorPos.getY() - aNewAnchorPos.getY() );
         // set new anchor position
         DrawObj()->SetAnchorPos( aNewAnchorPos );
         // correct object position, caused by setting new anchor position
diff --git a/sw/source/core/layout/calcmove.cxx b/sw/source/core/layout/calcmove.cxx
index 884bfba..a7fc278 100644
--- a/sw/source/core/layout/calcmove.cxx
+++ b/sw/source/core/layout/calcmove.cxx
@@ -547,23 +547,23 @@
 
                 // cells may now leave their uppers
                 if( bVert && FRM_CELL & nMyType && !mbReverse )
-                    maFrm.Pos().X() -= maFrm.Width() -pPrv->Frm().Width();
+                    maFrm.Pos().setX(maFrm.Pos().getX() - maFrm.Width() + pPrv->Frm().Width());
             }
             else if( bVert && FRM_NOTE_VERT & nMyType )
             {
                 if( mbReverse )
-                    maFrm.Pos().X() += pPrv->Frm().Width();
+                    maFrm.Pos().setX(maFrm.Pos().getX() + pPrv->Frm().Width());
                 else
                     //Badaa: 2008-04-18 * Support for Classical Mongolian Script (SCMS) joint with Jiayanmin
                 {
                     if ( bVertL2R )
-                           maFrm.Pos().X() += pPrv->Frm().Width();
+                           maFrm.Pos().setX(maFrm.Pos().getX() + pPrv->Frm().Width());
                     else
-                           maFrm.Pos().X() -= maFrm.Width();
+                           maFrm.Pos().setX(maFrm.Pos().getX() - maFrm.Width());
                   }
             }
             else
-                maFrm.Pos().Y() += pPrv->Frm().Height();
+                maFrm.Pos().setY(maFrm.Pos().getY() + pPrv->Frm().Height());
         }
         else if ( GetUpper() )
         {
@@ -602,17 +602,17 @@
 
                     // cells may now leave their uppers
                     if( bVert && FRM_CELL & nMyType && !mbReverse )
-                        maFrm.Pos().X() -= maFrm.Width() -pPrv->Frm().Width();
+                        maFrm.Pos().setX(maFrm.Pos().getX() - maFrm.Width() + pPrv->Frm().Width());
                 }
                 else if( bVert && FRM_NOTE_VERT & nMyType )
                 {
                     if( mbReverse )
-                        maFrm.Pos().X() += pPrv->Frm().Width();
+                        maFrm.Pos().setX(maFrm.Pos().getX() + pPrv->Frm().Width());
                     else
-                        maFrm.Pos().X() -= maFrm.Width();
+                        maFrm.Pos().setX(maFrm.Pos().getX() - maFrm.Width());
                 }
                 else
-                    maFrm.Pos().Y() += pPrv->Frm().Height();
+                    maFrm.Pos().setY(maFrm.Pos().getY() + pPrv->Frm().Height());
             }
             else
             {
@@ -621,22 +621,25 @@
                 if( FRM_NEIGHBOUR & nMyType && IsRightToLeft() )
                 {
                     if( bVert )
-                        maFrm.Pos().Y() += GetUpper()->Prt().Height()
-                                          - maFrm.Height();
+                        maFrm.Pos().setY(maFrm.Pos().getY() + GetUpper()->Prt().Height()
+                                          - maFrm.Height());
                     else
-                        maFrm.Pos().X() += GetUpper()->Prt().Width()
-                                          - maFrm.Width();
+                        maFrm.Pos().setX(maFrm.Pos().getX() + GetUpper()->Prt().Width()
+                                          - maFrm.Width());
                 }
                 //Badaa: 2008-04-18 * Support for Classical Mongolian Script (SCMS) joint with Jiayanmin
                 else if( bVert && !bVertL2R && FRM_NOTE_VERT & nMyType && !mbReverse )
-                    maFrm.Pos().X() -= maFrm.Width() - GetUpper()->Prt().Width();
+                    maFrm.Pos().setX(maFrm.Pos().getX() - maFrm.Width() + GetUpper()->Prt().Width());
             }
         }
         else
-            maFrm.Pos().X() = maFrm.Pos().Y() = 0;
+        {
+            maFrm.Pos().setX(0);
+            maFrm.Pos().setY(0);
+        }
         //Badaa: 2008-04-18 * Support for Classical Mongolian Script (SCMS) joint with Jiayanmin
         if( IsBodyFrm() && bVert && !bVertL2R && !mbReverse && GetUpper() )
-            maFrm.Pos().X() += GetUpper()->Prt().Width() - maFrm.Width();
+            maFrm.Pos().setX(maFrm.Pos().getX() + GetUpper()->Prt().Width() - maFrm.Width());
         mbValidPos = sal_True;
     }
 }
@@ -966,7 +969,8 @@
 
             if( (Prt().*fnRect->fnGetHeight)() )
                 ((SwTxtFrm*)this)->HideHidden();
-            Prt().Pos().X() = Prt().Pos().Y() = 0;
+            Prt().Pos().setX(0);
+            Prt().Pos().setY(0);
             (Prt().*fnRect->fnSetWidth)( (Frm().*fnRect->fnGetWidth)() );
             (Prt().*fnRect->fnSetHeight)( 0 );
             nUpper = -( (Frm().*fnRect->fnGetHeight)() );
diff --git a/sw/source/core/layout/fly.cxx b/sw/source/core/layout/fly.cxx
index f72a1ee..d921849 100644
--- a/sw/source/core/layout/fly.cxx
+++ b/sw/source/core/layout/fly.cxx
@@ -177,7 +177,8 @@
 
     //Und erstmal in den Wald stellen die Kiste, damit bei neuen Dokument nicht
     //unnoetig viel formatiert wird.
-    Frm().Pos().X() = Frm().Pos().Y() = FAR_AWAY;
+    Frm().Pos().setX(FAR_AWAY);
+    Frm().Pos().setY(FAR_AWAY);
 }
 
 void SwFlyFrm::Chain( SwFrm* _pAnch )
@@ -1324,7 +1325,8 @@
         if ( Frm().Top() == FAR_AWAY && Frm().Left() == FAR_AWAY )
         {
             //Sicherheitsschaltung wegnehmen (siehe SwFrm::CTor)
-            Frm().Pos().X() = Frm().Pos().Y() = 0;
+            Frm().Pos().setX(0);
+            Frm().Pos().setY(0);
             // #i68520#
             InvalidateObjRectWithSpaces();
         }
@@ -2678,7 +2680,7 @@
                             else
                                 aNewPoint = pOutDev->LogicToLogic( rPoly[ i ], aGrfMap, aDispMap  );
 
-                            rPoly[ i ] = Point( FRound( aNewPoint.X() * fScaleX ), FRound( aNewPoint.Y() * fScaleY ) );
+                            rPoly[ i ] = Point( FRound( aNewPoint.getX() * fScaleX ), FRound( aNewPoint.getY() * fScaleY ) );
                         }
                     }
                 }
@@ -2750,17 +2752,17 @@
 // #i68520#
 bool SwFlyFrm::_SetObjTop( const SwTwips _nTop )
 {
-    const bool bChanged( Frm().Pos().Y() != _nTop );
+    const bool bChanged( Frm().Pos().getY() != _nTop );
 
-    Frm().Pos().Y() = _nTop;
+    Frm().Pos().setY(_nTop);
 
     return bChanged;
 }
 bool SwFlyFrm::_SetObjLeft( const SwTwips _nLeft )
 {
-    const bool bChanged( Frm().Pos().X() != _nLeft );
+    const bool bChanged( Frm().Pos().getX() != _nLeft );
 
-    Frm().Pos().X() = _nLeft;
+    Frm().Pos().setX(_nLeft);
 
     return bChanged;
 }
diff --git a/sw/source/core/layout/flycnt.cxx b/sw/source/core/layout/flycnt.cxx
index 41057cd..32c6f89 100644
--- a/sw/source/core/layout/flycnt.cxx
+++ b/sw/source/core/layout/flycnt.cxx
@@ -1039,14 +1039,14 @@
 {
     SwRect aTmp( pFrm->Prt() );
     aTmp += pFrm->Frm().Pos();
-    if ( rPoint.X() < aTmp.Left() )
-        rPoint.X() = aTmp.Left();
-    else if ( rPoint.X() > aTmp.Right() )
-        rPoint.X() = aTmp.Right();
-    if ( rPoint.Y() < aTmp.Top() )
-        rPoint.Y() = aTmp.Top();
-    else if ( rPoint.Y() > aTmp.Bottom() )
-        rPoint.Y() = aTmp.Bottom();
+    if ( rPoint.getX() < aTmp.Left() )
+        rPoint.setX(aTmp.Left());
+    else if ( rPoint.getX() > aTmp.Right() )
+        rPoint.setX(aTmp.Right());
+    if ( rPoint.getY() < aTmp.Top() )
+        rPoint.setY(aTmp.Top());
+    else if ( rPoint.getY() > aTmp.Bottom() )
+        rPoint.setY(aTmp.Bottom());
 
 }
 
@@ -1214,7 +1214,7 @@
     Point aNew( rNew );
     //Badaa: 2008-04-18 * Support for Classical Mongolian Script (SCMS) joint with Jiayanmin
       if( ( GetAnchorFrm()->IsVertical() && !GetAnchorFrm()->IsVertLR() ) || GetAnchorFrm()->IsRightToLeft() )
-        aNew.X() += Frm().Width();
+        aNew.setX(aNew.getX() + Frm().Width());
     SwCntntFrm *pCnt = (SwCntntFrm*)::FindAnchor( GetAnchorFrm(), aNew );
     if( pCnt->IsProtected() )
         pCnt = (SwCntntFrm*)GetAnchorFrm();
@@ -1229,9 +1229,9 @@
         ( !bRTL !=  !GetAnchorFrm()->IsRightToLeft() ) )
     {
         if( bVert || bRTL )
-            aNew.X() += Frm().Width();
+            aNew.setX(aNew.getX() + Frm().Width());
         else
-            aNew.X() -= Frm().Width();
+            aNew.setX(aNew.getX() - Frm().Width());
     }
 
     if ( pCnt->IsInDocBody() )
@@ -1305,7 +1305,7 @@
             //Badaa: 2008-04-18 * Support for Classical Mongolian Script (SCMS) joint with Jiayanmin
             {
                 if ( pUp->IsVertLR()  )
-                    nDiff += pUp->Prt().Width() - pFollow->GetRelPos().X();
+                    nDiff += pUp->Prt().Width() - pFollow->GetRelPos().getX();
                 else
                        nDiff += pFollow->Frm().Left() + pFollow->Frm().Width()
                              - pUp->Frm().Left() - pUp->Prt().Left();
diff --git a/sw/source/core/layout/flyincnt.cxx b/sw/source/core/layout/flyincnt.cxx
index 671c249..a0d9a3f 100644
--- a/sw/source/core/layout/flyincnt.cxx
+++ b/sw/source/core/layout/flyincnt.cxx
@@ -47,9 +47,9 @@
     // OD 2004-05-27 #i26791# - member <aRelPos> moved to <SwAnchoredObject>
     Point aRelPos;
     if( pAnch && pAnch->IsVertical() )
-        aRelPos.X() = pAnch->IsReverse() ? nRel : -nRel;
+        aRelPos.setX(pAnch->IsReverse() ? nRel : -nRel);
     else
-        aRelPos.Y() = nRel;
+        aRelPos.setY(nRel);
     SetCurrRelPos( aRelPos );
 }
 
diff --git a/sw/source/core/layout/frmtool.cxx b/sw/source/core/layout/frmtool.cxx
index ca95552..f9337b1 100644
--- a/sw/source/core/layout/frmtool.cxx
+++ b/sw/source/core/layout/frmtool.cxx
@@ -3435,8 +3435,8 @@
                     // nocht nicht "formatiert" sind
                     if( !bCalcFrm && nFrmType & FRM_FLY &&
                         ((SwFlyFrm*)pTmpFrm)->GetAnchorFrm() &&
-                        FAR_AWAY == pTmpFrm->Frm().Pos().X() &&
-                        FAR_AWAY == pTmpFrm->Frm().Pos().Y() )
+                        FAR_AWAY == pTmpFrm->Frm().Pos().getX() &&
+                        FAR_AWAY == pTmpFrm->Frm().Pos().getY() )
                         aCalcRect = ((SwFlyFrm*)pTmpFrm)->GetAnchorFrm()->Frm();
                     else
                         aCalcRect = pTmpFrm->Frm();
@@ -3450,7 +3450,7 @@
                     // Point not in rectangle. Compare distances:
                     const Point aCalcRectCenter = aCalcRect.Center();
                     const Point aDiff = aCalcRectCenter - *pPoint;
-                    const sal_uInt64 nCurrentDist = aDiff.X() * aDiff.X() + aDiff.Y() * aDiff.Y(); // opt: no sqrt
+                    const sal_uInt64 nCurrentDist = aDiff.getX() * aDiff.getX() + aDiff.getY() * aDiff.getY(); // opt: no sqrt
                     if ( !pMinFrm || nCurrentDist < nMinDist )
                     {
                         pMinFrm = pTmpFrm;
diff --git a/sw/source/core/layout/pagechg.cxx b/sw/source/core/layout/pagechg.cxx
index 5421332..304bbcb 100644
--- a/sw/source/core/layout/pagechg.cxx
+++ b/sw/source/core/layout/pagechg.cxx
@@ -120,7 +120,7 @@
         Frm().Height( nHeight );
         //Badaa: 2008-04-18 * Support for Classical Mongolian Script (SCMS) joint with Jiayanmin
         if( IsVertical() && !IsVertLR() && !IsReverse() && nWidth != Frm().Width() )
-            Frm().Pos().X() += Frm().Width() - nWidth;
+            Frm().Pos().setX(Frm().Pos().getX() + Frm().Width() - nWidth);
         Frm().Width( nWidth );
     }
 
@@ -169,7 +169,8 @@
     }
     if( bNoGrid )
     {
-        Prt().Pos().X() = Prt().Pos().Y() = 0;
+        Prt().Pos().setX(0);
+        Prt().Pos().setY(0);
         Prt().Height( Frm().Height() );
         Prt().Width( Frm().Width() );
     }
@@ -1691,11 +1692,13 @@
 {
     if ( !mbValidPos )
     {   mbValidPos = sal_True;
-        maFrm.Pos().X() = maFrm.Pos().Y() = DOCUMENTBORDER;
+        maFrm.Pos().setX(DOCUMENTBORDER);
+        maFrm.Pos().setY(DOCUMENTBORDER);
     }
     if ( !mbValidPrtArea )
     {   mbValidPrtArea = sal_True;
-        maPrt.Pos().X() = maPrt.Pos().Y() = 0;
+        maPrt.Pos().setX(0);
+        maPrt.Pos().setY(0);
         maPrt.SSize( maFrm.SSize() );
     }
     if ( !mbValidSize )
@@ -2118,7 +2121,7 @@
 
     maPageRects.clear();
 
-    const long nBorder = Frm().Pos().X();
+    const long nBorder = Frm().Pos().getX();
     const long nVisWidth = mnViewWidth - 2 * nBorder;
     const long nGapBetweenPages = GAPBETWEENPAGES;
 
@@ -2288,10 +2291,10 @@
                 // RTL view layout: Calculate mirrored page position
                 if ( bRTL )
                 {
-                    const long nXOffsetInRow = aNewPagePos.X() - nRowStart;
-                    aNewPagePos.X() = nRowEnd - nXOffsetInRow - nCurrentPageWidth;
+                    const long nXOffsetInRow = aNewPagePos.getX() - nRowStart;
+                    aNewPagePos.setX(nRowEnd - nXOffsetInRow - nCurrentPageWidth);
                     aNewPagePosWithLeftOffset = aNewPagePos;
-                    aNewPagePosWithLeftOffset.X() += nLeftPageAddOffset;
+                    aNewPagePosWithLeftOffset.setX(aNewPagePosWithLeftOffset.getX() + nLeftPageAddOffset);
                 }
 
                 if ( aNewPagePosWithLeftOffset != aOldPagePos )
@@ -2307,12 +2310,12 @@
                 const bool bLastColumn = pPageToAdjust->GetNext() == pEndOfRow;
                 const bool bLastRow = !pEndOfRow;
 
-                nMinPageLeft  = std::min( nMinPageLeft, aNewPagePos.X() );
-                nMaxPageRight = std::max( nMaxPageRight, aNewPagePos.X() + nCurrentPageWidth);
+                nMinPageLeft  = std::min( nMinPageLeft, aNewPagePos.getX() );
+                nMaxPageRight = std::max( nMaxPageRight, aNewPagePos.getX() + nCurrentPageWidth);
 
                 // border of nGapBetweenPages around the current page:
-                SwRect aPageRectWithBorders( aNewPagePos.X() - nGapBetweenPages,
-                                             aNewPagePos.Y(),
+                SwRect aPageRectWithBorders( aNewPagePos.getX() - nGapBetweenPages,
+                                             aNewPagePos.getY(),
                                              pPageToAdjust->Frm().SSize().Width() + nGapBetweenPages + nSidebarWidth,
                                              nCurrentRowHeight );
 
diff --git a/sw/source/core/layout/paintfrm.cxx b/sw/source/core/layout/paintfrm.cxx
index 45b0fd4..615d7d8 100644
--- a/sw/source/core/layout/paintfrm.cxx
+++ b/sw/source/core/layout/paintfrm.cxx
@@ -5416,11 +5416,11 @@
                     bool bBorder = bLeft || bRight;
                     while( nY < aInter.Top() )
                     {
-                        aTmp.Pos().Y() = nY;
+                        aTmp.Pos().setY(nY);
                         if( bGrid )
                         {
                             nY += nGrid;
-                            SwTwips nPosY = std::max( aInter.Top(), aTmp.Pos().Y() );
+                            SwTwips nPosY = std::max( aInter.Top(), aTmp.Pos().getY() );
                             SwTwips nHeight = std::min(nBottom, nY ) - nPosY;
                             if( nHeight )
                             {
@@ -5453,7 +5453,7 @@
                             nY += nRuby;
                             if( bBorder )
                             {
-                                SwTwips nPos = std::max(aInter.Top(),aTmp.Pos().Y());
+                                SwTwips nPos = std::max(aInter.Top(),aTmp.Pos().getY());
                                 SwTwips nH = std::min( nBottom, nY ) - nPos;
                                 SwRect aVert( Point( aGrid.Left(), nPos ),
                                             Size( 1, nH ) );
@@ -5463,7 +5463,7 @@
                                         PaintBorderLine(rRect,aVert,this,pCol);
                                     if( bRight )
                                     {
-                                        aVert.Pos().X() = nGridRight;
+                                        aVert.Pos().setX(nGridRight);
                                         PaintBorderLine(rRect,aVert,this,pCol);
                                     }
                                 }
@@ -5473,33 +5473,33 @@
                     }
                     while( nY <= nBottom )
                     {
-                        aTmp.Pos().Y() = nY;
+                        aTmp.Pos().setY(nY);
                         PaintBorderLine( rRect, aTmp, this, pCol);
                         if( bGrid )
                         {
                             nY += nGrid;
-                            SwTwips nHeight = std::min(nBottom, nY) - aTmp.Pos().Y();
+                            SwTwips nHeight = std::min(nBottom, nY) - aTmp.Pos().getY();
                             if( nHeight )
                             {
                                 if( bCell )
                                 {
-                                    SwRect aVert( Point( nX, aTmp.Pos().Y() ),
+                                    SwRect aVert( Point( nX, aTmp.Pos().getY() ),
                                                 Size( 1, nHeight ) );
                                     while( aVert.Left() <= nRight )
                                     {
                                         PaintBorderLine( rRect, aVert, this, pCol);
-                                        aVert.Pos().X() += nGridWidth;  //for textgrid refactor
+                                        aVert.Pos().setX(aVert.Pos().getX() + nGridWidth);  //for textgrid refactor
                                     }
                                 }
                                 else if( bBorder )
                                 {
                                     SwRect aVert( Point( aGrid.Left(),
-                                        aTmp.Pos().Y() ), Size( 1, nHeight ) );
+                                        aTmp.Pos().getY() ), Size( 1, nHeight ) );
                                     if( bLeft )
                                         PaintBorderLine(rRect,aVert,this,pCol);
                                     if( bRight )
                                     {
-                                        aVert.Pos().X() = nGridRight;
+                                        aVert.Pos().setX(nGridRight);
                                         PaintBorderLine(rRect,aVert,this,pCol);
                                     }
                                 }
@@ -5520,7 +5520,7 @@
                                         PaintBorderLine(rRect,aVert,this,pCol);
                                     if( bRight )
                                     {
-                                        aVert.Pos().X() = nGridRight;
+                                        aVert.Pos().setX(nGridRight);
                                         PaintBorderLine(rRect,aVert,this,pCol);
                                     }
                                 }
@@ -6467,7 +6467,7 @@
         basegfx::B2DVector aHorizVector( aXOffDirs[i], 0.0 );
         basegfx::B2DVector aVertVector( 0.0, aYOffDirs[i] );
 
-        basegfx::B2DPoint aBPoint( aPoints[i].X(), aPoints[i].Y() );
+        basegfx::B2DPoint aBPoint( aPoints[i].getX(), aPoints[i].getY() );
 
         basegfx::B2DPolygon aPolygon;
         aPolygon.append( aBPoint + aHorizVector * nLineLength );
@@ -6522,7 +6522,7 @@
         basegfx::B2DVector aHorizVector( aXOffDirs[i], 0.0 );
         basegfx::B2DVector aVertVector( 0.0, aYOffDirs[i] );
 
-        basegfx::B2DPoint aBPoint( aPoints[i].X(), aPoints[i].Y() );
+        basegfx::B2DPoint aBPoint( aPoints[i].getX(), aPoints[i].getY() );
 
         basegfx::B2DPolygon aPolygon;
         aPolygon.append( aBPoint + aHorizVector * nLineLength );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I79fe610e4cc8bbbddbd8415198d6309a93f4ba18
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Miklos Vajna <vmiklos at suse.cz>
Gerrit-Reviewer: Julien Nabet <serval2412 at yahoo.fr>



More information about the LibreOffice mailing list