[Libreoffice-commits] core.git: 5 commits - sw/source
Caolán McNamara
caolanm at redhat.com
Thu Aug 4 08:16:35 UTC 2016
sw/source/core/text/itrcrsr.cxx | 1 +
sw/source/core/text/porfly.cxx | 5 +++++
sw/source/core/text/porfly.hxx | 1 +
3 files changed, 7 insertions(+)
New commits:
commit 2b3112b30a8dea8a009c11ddb209e12697f72e32
Author: Caolán McNamara <caolanm at redhat.com>
Date: Thu Aug 4 09:15:15 2016 +0100
the sole caller is ok with rPoint being modified, so change to non-const arg
Change-Id: I40a24502e2e8ad312e13e0c47ee1fde3118d2251
diff --git a/sw/source/core/text/porfly.cxx b/sw/source/core/text/porfly.cxx
index 633c7b5..e38e99d 100644
--- a/sw/source/core/text/porfly.cxx
+++ b/sw/source/core/text/porfly.cxx
@@ -414,14 +414,9 @@ void SwFlyCntPortion::SetBase( const SwTextFrame& rFrame, const Point &rBase,
}
}
-void SwFlyCntPortion::GetFlyCursorOfst(const Point &rPoint, SwPosition &rPos, SwCursorMoveState* pCMS) const
+void SwFlyCntPortion::GetFlyCursorOfst(Point &rPoint, SwPosition &rPos, SwCursorMoveState* pCMS) const
{
- // As the FlyCnt are not attached to the side, their GetCursorOfst() will
- // not be called.
- // In order to reduce management overhead for the layout page, the paragraph
- // calls the FlyFrame's GetCursorOfst() only when needed
- Point aPoint(rPoint);
- GetFlyFrame()->GetCursorOfst(&rPos, aPoint, pCMS);
+ GetFlyFrame()->GetCursorOfst(&rPos, rPoint, pCMS);
}
sal_Int32 SwFlyCntPortion::GetCursorOfst( const sal_uInt16 nOfst ) const
diff --git a/sw/source/core/text/porfly.hxx b/sw/source/core/text/porfly.hxx
index 9204edb..129bf39 100644
--- a/sw/source/core/text/porfly.hxx
+++ b/sw/source/core/text/porfly.hxx
@@ -79,7 +79,7 @@ public:
long nLnAscent, long nLnDescent,
long nFlyAscent, long nFlyDescent,
AsCharFlags nFlags );
- void GetFlyCursorOfst(const Point &rPoint, SwPosition& rPos, SwCursorMoveState* pCMS) const;
+ void GetFlyCursorOfst(Point &rPoint, SwPosition& rPos, SwCursorMoveState* pCMS) const;
virtual bool Format( SwTextFormatInfo &rInf ) override;
virtual void Paint( const SwTextPaintInfo &rInf ) const override;
OUTPUT_OPERATOR_OVERRIDE
commit cc7a55450658696009fe01e94632e80bb3785d5b
Author: Caolán McNamara <caolanm at redhat.com>
Date: Thu Aug 4 09:07:03 2016 +0100
only called if !bDraw
Change-Id: I7310b5f8cbbc4176e76d92ce2bcd49168470843c
diff --git a/sw/source/core/text/porfly.cxx b/sw/source/core/text/porfly.cxx
index 40663a2..633c7b5 100644
--- a/sw/source/core/text/porfly.cxx
+++ b/sw/source/core/text/porfly.cxx
@@ -420,8 +420,6 @@ void SwFlyCntPortion::GetFlyCursorOfst(const Point &rPoint, SwPosition &rPos, Sw
// not be called.
// In order to reduce management overhead for the layout page, the paragraph
// calls the FlyFrame's GetCursorOfst() only when needed
- if (bDraw)
- return;
Point aPoint(rPoint);
GetFlyFrame()->GetCursorOfst(&rPos, aPoint, pCMS);
}
commit 35f4248206e844cabd83757ec57ddde1c2ef66b4
Author: Caolán McNamara <caolanm at redhat.com>
Date: Thu Aug 4 08:58:59 2016 +0100
SwLinePortion::GetCursorOfst is const and its ret is ignored here
so nOfst is unnecessary
Change-Id: I27f73057b55ebdf30eb69a19ee1b0647ba25583e
diff --git a/sw/source/core/text/itrcrsr.cxx b/sw/source/core/text/itrcrsr.cxx
index e087d77..3e871f1 100644
--- a/sw/source/core/text/itrcrsr.cxx
+++ b/sw/source/core/text/itrcrsr.cxx
@@ -1713,8 +1713,7 @@ sal_Int32 SwTextCursor::GetCursorOfst( SwPosition *pPos, const Point &rPoint,
if( bChgNodeInner && pTmp->Frame().IsInside( aTmpPoint ) &&
!( pTmp->IsProtected() ) )
{
- static_cast<SwFlyCntPortion*>(pPor)->
- GetFlyCursorOfst( nX, aTmpPoint, *pPos, pCMS );
+ static_cast<SwFlyCntPortion*>(pPor)->GetFlyCursorOfst(aTmpPoint, *pPos, pCMS);
// After a change of the frame, our font must be still
// available for/in the OutputDevice.
// For comparison: Paint and new SwFlyCntPortion !
diff --git a/sw/source/core/text/porfly.cxx b/sw/source/core/text/porfly.cxx
index 079d9af..40663a2 100644
--- a/sw/source/core/text/porfly.cxx
+++ b/sw/source/core/text/porfly.cxx
@@ -414,16 +414,16 @@ void SwFlyCntPortion::SetBase( const SwTextFrame& rFrame, const Point &rBase,
}
}
-void SwFlyCntPortion::GetFlyCursorOfst( const sal_uInt16 nOfst,
- const Point &rPoint, SwPosition &rPos, SwCursorMoveState* pCMS ) const
+void SwFlyCntPortion::GetFlyCursorOfst(const Point &rPoint, SwPosition &rPos, SwCursorMoveState* pCMS) const
{
// As the FlyCnt are not attached to the side, their GetCursorOfst() will
// not be called.
// In order to reduce management overhead for the layout page, the paragraph
// calls the FlyFrame's GetCursorOfst() only when needed
- Point aPoint( rPoint );
- if( bDraw || !( GetFlyFrame()->GetCursorOfst(&rPos, aPoint, pCMS) ) )
- SwLinePortion::GetCursorOfst(nOfst);
+ if (bDraw)
+ return;
+ Point aPoint(rPoint);
+ GetFlyFrame()->GetCursorOfst(&rPos, aPoint, pCMS);
}
sal_Int32 SwFlyCntPortion::GetCursorOfst( const sal_uInt16 nOfst ) const
diff --git a/sw/source/core/text/porfly.hxx b/sw/source/core/text/porfly.hxx
index 3aaf6ad..9204edb 100644
--- a/sw/source/core/text/porfly.hxx
+++ b/sw/source/core/text/porfly.hxx
@@ -79,8 +79,7 @@ public:
long nLnAscent, long nLnDescent,
long nFlyAscent, long nFlyDescent,
AsCharFlags nFlags );
- void GetFlyCursorOfst( const sal_uInt16 nOfst, const Point &rPoint,
- SwPosition& rPos, SwCursorMoveState* pCMS ) const;
+ void GetFlyCursorOfst(const Point &rPoint, SwPosition& rPos, SwCursorMoveState* pCMS) const;
virtual bool Format( SwTextFormatInfo &rInf ) override;
virtual void Paint( const SwTextPaintInfo &rInf ) const override;
OUTPUT_OPERATOR_OVERRIDE
commit 2702796806cae2b28ae78e78d29c3559bf97607e
Author: Caolán McNamara <caolanm at redhat.com>
Date: Thu Aug 4 08:57:20 2016 +0100
pPos can never be null
Change-Id: Ib7b57d628fbd80a1d2214917cd439e09d4803d91
diff --git a/sw/source/core/text/itrcrsr.cxx b/sw/source/core/text/itrcrsr.cxx
index c903af9..e087d77 100644
--- a/sw/source/core/text/itrcrsr.cxx
+++ b/sw/source/core/text/itrcrsr.cxx
@@ -1713,8 +1713,8 @@ sal_Int32 SwTextCursor::GetCursorOfst( SwPosition *pPos, const Point &rPoint,
if( bChgNodeInner && pTmp->Frame().IsInside( aTmpPoint ) &&
!( pTmp->IsProtected() ) )
{
- nLength = static_cast<SwFlyCntPortion*>(pPor)->
- GetFlyCursorOfst( nX, aTmpPoint, pPos, pCMS );
+ static_cast<SwFlyCntPortion*>(pPor)->
+ GetFlyCursorOfst( nX, aTmpPoint, *pPos, pCMS );
// After a change of the frame, our font must be still
// available for/in the OutputDevice.
// For comparison: Paint and new SwFlyCntPortion !
diff --git a/sw/source/core/text/porfly.cxx b/sw/source/core/text/porfly.cxx
index ca37415..079d9af 100644
--- a/sw/source/core/text/porfly.cxx
+++ b/sw/source/core/text/porfly.cxx
@@ -414,18 +414,16 @@ void SwFlyCntPortion::SetBase( const SwTextFrame& rFrame, const Point &rBase,
}
}
-sal_Int32 SwFlyCntPortion::GetFlyCursorOfst( const sal_uInt16 nOfst,
- const Point &rPoint, SwPosition *pPos, SwCursorMoveState* pCMS ) const
+void SwFlyCntPortion::GetFlyCursorOfst( const sal_uInt16 nOfst,
+ const Point &rPoint, SwPosition &rPos, SwCursorMoveState* pCMS ) const
{
// As the FlyCnt are not attached to the side, their GetCursorOfst() will
// not be called.
// In order to reduce management overhead for the layout page, the paragraph
// calls the FlyFrame's GetCursorOfst() only when needed
Point aPoint( rPoint );
- if( !pPos || bDraw || !( GetFlyFrame()->GetCursorOfst( pPos, aPoint, pCMS ) ) )
- return SwLinePortion::GetCursorOfst( nOfst );
- else
- return 0;
+ if( bDraw || !( GetFlyFrame()->GetCursorOfst(&rPos, aPoint, pCMS) ) )
+ SwLinePortion::GetCursorOfst(nOfst);
}
sal_Int32 SwFlyCntPortion::GetCursorOfst( const sal_uInt16 nOfst ) const
diff --git a/sw/source/core/text/porfly.hxx b/sw/source/core/text/porfly.hxx
index b51efd9..3aaf6ad 100644
--- a/sw/source/core/text/porfly.hxx
+++ b/sw/source/core/text/porfly.hxx
@@ -79,8 +79,8 @@ public:
long nLnAscent, long nLnDescent,
long nFlyAscent, long nFlyDescent,
AsCharFlags nFlags );
- sal_Int32 GetFlyCursorOfst( const sal_uInt16 nOfst, const Point &rPoint,
- SwPosition *pPos, SwCursorMoveState* pCMS ) const;
+ void GetFlyCursorOfst( const sal_uInt16 nOfst, const Point &rPoint,
+ SwPosition& rPos, SwCursorMoveState* pCMS ) const;
virtual bool Format( SwTextFormatInfo &rInf ) override;
virtual void Paint( const SwTextPaintInfo &rInf ) const override;
OUTPUT_OPERATOR_OVERRIDE
commit 24c009de4d6d27233027e9bcd86ea181d230be8f
Author: Caolán McNamara <caolanm at redhat.com>
Date: Thu Aug 4 08:50:57 2016 +0100
Resolves: tdf#101269 cannot click/edit text inside a frame
regression from...
commit b0c0a074a2318a231f3f635784da494f1b713c53
Date: Mon Apr 18 11:03:43 2016 +0200
clang-tidy clang-analyzer-deadcode.DeadStores
suspect that some of these may reveal latent bugs....
and restore method then removed by
commit 47f62540bd2c2f107313bb0c6f141cd4460b6379
Date: Thu May 19 10:31:47 2016 +0200
loplugin:unusedmethods in sw
Change-Id: I14826eecadbc1a74e408f8907eea8746debe4db4
diff --git a/sw/source/core/text/itrcrsr.cxx b/sw/source/core/text/itrcrsr.cxx
index dbb67b8..c903af9 100644
--- a/sw/source/core/text/itrcrsr.cxx
+++ b/sw/source/core/text/itrcrsr.cxx
@@ -1713,6 +1713,8 @@ sal_Int32 SwTextCursor::GetCursorOfst( SwPosition *pPos, const Point &rPoint,
if( bChgNodeInner && pTmp->Frame().IsInside( aTmpPoint ) &&
!( pTmp->IsProtected() ) )
{
+ nLength = static_cast<SwFlyCntPortion*>(pPor)->
+ GetFlyCursorOfst( nX, aTmpPoint, pPos, pCMS );
// After a change of the frame, our font must be still
// available for/in the OutputDevice.
// For comparison: Paint and new SwFlyCntPortion !
diff --git a/sw/source/core/text/porfly.cxx b/sw/source/core/text/porfly.cxx
index 0eca9ca..ca37415 100644
--- a/sw/source/core/text/porfly.cxx
+++ b/sw/source/core/text/porfly.cxx
@@ -414,6 +414,20 @@ void SwFlyCntPortion::SetBase( const SwTextFrame& rFrame, const Point &rBase,
}
}
+sal_Int32 SwFlyCntPortion::GetFlyCursorOfst( const sal_uInt16 nOfst,
+ const Point &rPoint, SwPosition *pPos, SwCursorMoveState* pCMS ) const
+{
+ // As the FlyCnt are not attached to the side, their GetCursorOfst() will
+ // not be called.
+ // In order to reduce management overhead for the layout page, the paragraph
+ // calls the FlyFrame's GetCursorOfst() only when needed
+ Point aPoint( rPoint );
+ if( !pPos || bDraw || !( GetFlyFrame()->GetCursorOfst( pPos, aPoint, pCMS ) ) )
+ return SwLinePortion::GetCursorOfst( nOfst );
+ else
+ return 0;
+}
+
sal_Int32 SwFlyCntPortion::GetCursorOfst( const sal_uInt16 nOfst ) const
{
// OSL_FAIL("SwFlyCntPortion::GetCursorOfst: use GetFlyCursorOfst()");
diff --git a/sw/source/core/text/porfly.hxx b/sw/source/core/text/porfly.hxx
index 417c347..b51efd9 100644
--- a/sw/source/core/text/porfly.hxx
+++ b/sw/source/core/text/porfly.hxx
@@ -79,6 +79,8 @@ public:
long nLnAscent, long nLnDescent,
long nFlyAscent, long nFlyDescent,
AsCharFlags nFlags );
+ sal_Int32 GetFlyCursorOfst( const sal_uInt16 nOfst, const Point &rPoint,
+ SwPosition *pPos, SwCursorMoveState* pCMS ) const;
virtual bool Format( SwTextFormatInfo &rInf ) override;
virtual void Paint( const SwTextPaintInfo &rInf ) const override;
OUTPUT_OPERATOR_OVERRIDE
More information about the Libreoffice-commits
mailing list