[Libreoffice-commits] core.git: 5 commits - lotuswordpro/source sc/source sw/inc sw/source
Stephan Bergmann
sbergman at redhat.com
Tue Jun 10 10:45:01 PDT 2014
lotuswordpro/source/filter/lwplayout.hxx | 1
sc/source/ui/inc/optload.hxx | 59 ---------------------------
sw/inc/IMark.hxx | 2
sw/source/core/crsr/bookmrk.cxx | 14 +++---
sw/source/core/crsr/crossrefbookmark.cxx | 4 -
sw/source/core/doc/docbm.cxx | 34 +++++++--------
sw/source/core/doc/docdde.cxx | 2
sw/source/core/doc/docedt.cxx | 4 -
sw/source/core/doc/swserv.cxx | 2
sw/source/core/docnode/ndcopy.cxx | 4 -
sw/source/core/fields/reffld.cxx | 2
sw/source/core/inc/bookmrk.hxx | 12 ++---
sw/source/core/inc/crossrefbookmark.hxx | 2
sw/source/core/undo/rolbck.cxx | 6 +-
sw/source/core/undo/undobj.cxx | 8 +--
sw/source/core/unocore/unobkm.cxx | 4 -
sw/source/core/unocore/unoobj2.cxx | 2
sw/source/filter/writer/writer.cxx | 8 +--
sw/source/filter/ww8/docxattributeoutput.cxx | 2
sw/source/filter/ww8/wrtww8.cxx | 2
sw/source/uibase/utlui/content.cxx | 10 +---
21 files changed, 60 insertions(+), 124 deletions(-)
New commits:
commit 9a171514a1287adf17cba2a177c0563f81b2b1db
Author: Stephan Bergmann <sbergman at redhat.com>
Date: Tue Jun 10 19:43:40 2014 +0200
Duplicate #include
Change-Id: If83bcfb0e39c0b41ea5323b0ad3832bd65185c6c
diff --git a/lotuswordpro/source/filter/lwplayout.hxx b/lotuswordpro/source/filter/lwplayout.hxx
index 6571b73..38c6673 100644
--- a/lotuswordpro/source/filter/lwplayout.hxx
+++ b/lotuswordpro/source/filter/lwplayout.hxx
@@ -83,7 +83,6 @@
#include "xfilter/xfpagemaster.hxx"
#include "xfilter/xfcolumns.hxx"
#include "xfilter/xfborders.hxx"
-#include "xfilter/xffont.hxx"
#include "xfilter/xfframestyle.hxx"
#include "xfilter/xfframe.hxx"
#include "xfilter/xfbgimage.hxx"
commit dcf0ac048ce745774f401ada72b051757298c559
Author: Stephan Bergmann <sbergman at redhat.com>
Date: Tue Jun 10 19:21:16 2014 +0200
Change sw::mark::IMark::GetOtherMarkPos return type from ref to pointer
...so CrossRefBookmark::GetOtherMarkPos can legitimately return a null pointer
Change-Id: I88774cf0d139d017d67d6727ee01214997a73636
diff --git a/sw/inc/IMark.hxx b/sw/inc/IMark.hxx
index 13c1e00..72cdad1 100644
--- a/sw/inc/IMark.hxx
+++ b/sw/inc/IMark.hxx
@@ -41,7 +41,7 @@ namespace sw { namespace mark
virtual const SwPosition& GetMarkPos() const =0;
// GetOtherMarkPos() is only guaranteed to return a valid
// reference if IsExpanded() returned true
- virtual const SwPosition& GetOtherMarkPos() const =0;
+ virtual const SwPosition* GetOtherMarkPos() const =0;
virtual const SwPosition& GetMarkStart() const =0;
virtual const SwPosition& GetMarkEnd() const =0;
virtual const OUString& GetName() const =0;
diff --git a/sw/source/core/crsr/bookmrk.cxx b/sw/source/core/crsr/bookmrk.cxx
index 5bd6dc9..124aaec 100644
--- a/sw/source/core/crsr/bookmrk.cxx
+++ b/sw/source/core/crsr/bookmrk.cxx
@@ -311,7 +311,7 @@ namespace sw { namespace mark
void Fieldmark::SetMarkStartPos( const SwPosition& rNewStartPos )
{
- if ( GetMarkPos( ) <= GetOtherMarkPos( ) )
+ if ( GetMarkPos( ) <= *GetOtherMarkPos( ) )
return SetMarkPos( rNewStartPos );
else
return SetOtherMarkPos( rNewStartPos );
@@ -319,7 +319,7 @@ namespace sw { namespace mark
void Fieldmark::SetMarkEndPos( const SwPosition& rNewEndPos )
{
- if ( GetMarkPos( ) <= GetOtherMarkPos( ) )
+ if ( GetMarkPos( ) <= *GetOtherMarkPos( ) )
return SetOtherMarkPos( rNewEndPos );
else
return SetMarkPos( rNewEndPos );
@@ -330,15 +330,15 @@ namespace sw { namespace mark
return "Fieldmark: ( Name, Type, [ Nd1, Id1 ], [ Nd2, Id2 ] ): ( " + m_aName + ", "
+ m_aFieldname + ", [ " + OUString::number( GetMarkPos().nNode.GetIndex( ) )
+ ", " + OUString::number( GetMarkPos( ).nContent.GetIndex( ) ) + " ], ["
- + OUString::number( GetOtherMarkPos().nNode.GetIndex( ) ) + ", "
- + OUString::number( GetOtherMarkPos( ).nContent.GetIndex( ) ) + " ] ) ";
+ + OUString::number( GetOtherMarkPos()->nNode.GetIndex( ) ) + ", "
+ + OUString::number( GetOtherMarkPos( )->nContent.GetIndex( ) ) + " ] ) ";
}
void Fieldmark::Invalidate( )
{
// TODO: Does exist a better solution to trigger a format of the
// fieldmark portion? If yes, please use it.
- SwPaM aPaM( this->GetMarkPos(), this->GetOtherMarkPos() );
+ SwPaM aPaM( this->GetMarkPos(), *this->GetOtherMarkPos() );
aPaM.InvalidatePaM();
}
@@ -405,8 +405,8 @@ namespace sw { namespace mark
+ m_aName + ", " + GetFieldname() + ", [ "
+ OUString::number(GetMarkPos().nNode.GetIndex( ) ) + ", "
+ OUString::number( GetMarkPos( ).nContent.GetIndex( ) ) + " ], ["
- + OUString::number( GetOtherMarkPos().nNode.GetIndex( ) ) + ", "
- + OUString::number( GetOtherMarkPos( ).nContent.GetIndex( ) ) + " ] ) ";
+ + OUString::number( GetOtherMarkPos()->nNode.GetIndex( ) ) + ", "
+ + OUString::number( GetOtherMarkPos( )->nContent.GetIndex( ) ) + " ] ) ";
}
}}
diff --git a/sw/source/core/crsr/crossrefbookmark.cxx b/sw/source/core/crsr/crossrefbookmark.cxx
index 22438a8..49a4886 100644
--- a/sw/source/core/crsr/crossrefbookmark.cxx
+++ b/sw/source/core/crsr/crossrefbookmark.cxx
@@ -54,12 +54,12 @@ namespace sw { namespace mark
MarkBase::SetMarkPos(rNewPos);
}
- SwPosition& CrossRefBookmark::GetOtherMarkPos() const
+ SwPosition* CrossRefBookmark::GetOtherMarkPos() const
{
OSL_PRECOND(false,
"<sw::mark::CrossRefBookmark::GetOtherMarkPos(..)>"
" - this should never be called!");
- return *static_cast<SwPosition*>(NULL);
+ return NULL;
}
CrossRefHeadingBookmark::CrossRefHeadingBookmark(const SwPaM& rPaM,
diff --git a/sw/source/core/doc/docbm.cxx b/sw/source/core/doc/docbm.cxx
index 676b669..62a8002 100644
--- a/sw/source/core/doc/docbm.cxx
+++ b/sw/source/core/doc/docbm.cxx
@@ -188,12 +188,12 @@ namespace
if ( ( bChangedPos || bChangedOPos )
&& io_pMark->IsExpanded()
- && io_pMark->GetOtherMarkPos().nNode.GetNode().FindTableBoxStartNode() !=
+ && io_pMark->GetOtherMarkPos()->nNode.GetNode().FindTableBoxStartNode() !=
io_pMark->GetMarkPos().nNode.GetNode().FindTableBoxStartNode() )
{
if ( !bChangedOPos )
{
- io_pMark->SetMarkPos( io_pMark->GetOtherMarkPos() );
+ io_pMark->SetMarkPos( *io_pMark->GetOtherMarkPos() );
}
io_pMark->ClearOtherMarkPos();
DdeBookmark * const pDdeBkmk = dynamic_cast< DdeBookmark*>(io_pMark);
@@ -584,7 +584,7 @@ namespace sw { namespace mark
}
bool bChangedOPos = false;
if (pMark->IsExpanded() &&
- &pMark->GetOtherMarkPos().nNode.GetNode() == pOldNode)
+ &pMark->GetOtherMarkPos()->nNode.GetNode() == pOldNode)
{
pMark->SetMarkPos(aNewPos);
bChangedOPos= true;
@@ -626,10 +626,10 @@ namespace sw { namespace mark
bChangedPos = true;
}
if(pMark->IsExpanded() &&
- &pMark->GetOtherMarkPos().nNode.GetNode() == pOldNode)
+ &pMark->GetOtherMarkPos()->nNode.GetNode() == pOldNode)
{
SwPosition aNewPosRel(aNewPos);
- aNewPosRel.nContent += pMark->GetOtherMarkPos().nContent.GetIndex();
+ aNewPosRel.nContent += pMark->GetOtherMarkPos()->nContent.GetIndex();
pMark->SetOtherMarkPos(aNewPosRel);
bChangedOPos = true;
}
@@ -679,8 +679,8 @@ namespace sw { namespace mark
bool bIsPosInRange = lcl_GreaterThan(pMark->GetMarkPos(), rStt, pSttIdx)
&& lcl_Lower(pMark->GetMarkPos(), rEnd, pEndIdx);
bool bIsOtherPosInRange = pMark->IsExpanded()
- && lcl_GreaterThan(pMark->GetOtherMarkPos(), rStt, pSttIdx)
- && lcl_Lower(pMark->GetOtherMarkPos(), rEnd, pEndIdx);
+ && lcl_GreaterThan(*pMark->GetOtherMarkPos(), rStt, pSttIdx)
+ && lcl_Lower(*pMark->GetOtherMarkPos(), rEnd, pEndIdx);
// special case: completely in range, touching the end?
if ( pEndIdx != NULL
&& ( ( bIsOtherPosInRange
@@ -688,8 +688,8 @@ namespace sw { namespace mark
&& pMark->GetMarkPos().nContent == *pEndIdx )
|| ( bIsPosInRange
&& pMark->IsExpanded()
- && pMark->GetOtherMarkPos().nNode == rEnd
- && pMark->GetOtherMarkPos().nContent == *pEndIdx ) ) )
+ && pMark->GetOtherMarkPos()->nNode == rEnd
+ && pMark->GetOtherMarkPos()->nContent == *pEndIdx ) ) )
{
bIsPosInRange = true, bIsOtherPosInRange = true;
}
@@ -747,7 +747,7 @@ namespace sw { namespace mark
else
{
pNewPos =
- lcl_FindExpelPosition( rStt, rEnd, bIsPosInRange ? pMark->GetOtherMarkPos() : pMark->GetMarkPos() );
+ lcl_FindExpelPosition( rStt, rEnd, bIsPosInRange ? *pMark->GetOtherMarkPos() : pMark->GetMarkPos() );
}
}
@@ -1341,8 +1341,8 @@ SaveBookmark::SaveBookmark(
if(rBkmk.IsExpanded())
{
- m_nNode2 = rBkmk.GetOtherMarkPos().nNode.GetIndex();
- m_nCntnt2 = rBkmk.GetOtherMarkPos().nContent.GetIndex();
+ m_nNode2 = rBkmk.GetOtherMarkPos()->nNode.GetIndex();
+ m_nCntnt2 = rBkmk.GetOtherMarkPos()->nContent.GetIndex();
if(m_bSaveOtherPos)
{
@@ -1533,15 +1533,15 @@ void _SaveCntntIdx(SwDoc* pDoc,
}
if(pBkmk->IsExpanded()
- && pBkmk->GetOtherMarkPos().nNode.GetIndex() == nNode
- && pBkmk->GetOtherMarkPos().nContent.GetIndex() <= nCntnt)
+ && pBkmk->GetOtherMarkPos()->nNode.GetIndex() == nNode
+ && pBkmk->GetOtherMarkPos()->nContent.GetIndex() <= nCntnt)
{
if(bMarkPosEqual)
{ // the other position is before, the (main) position is equal
aSave.SetContent(pBkmk->GetMarkPos().nContent.GetIndex());
aSave.Add(rSaveArr);
}
- aSave.SetContent(pBkmk->GetOtherMarkPos().nContent.GetIndex());
+ aSave.SetContent(pBkmk->GetOtherMarkPos()->nContent.GetIndex());
aSave.IncType();
aSave.Add(rSaveArr);
aSave.DecType();
@@ -1746,7 +1746,7 @@ void _RestoreCntntIdx(SwDoc* pDoc,
case 0x8001:
if (MarkBase* pMark = dynamic_cast<MarkBase*>(pMarkAccess->getAllMarksBegin()[aSave.GetCount()].get()))
{
- SwPosition aNewPos(pMark->GetOtherMarkPos());
+ SwPosition aNewPos(*pMark->GetOtherMarkPos());
aNewPos.nNode = *pCNd;
aNewPos.nContent.Assign(pCNd, aSave.GetContent() + nOffset);
pMark->SetOtherMarkPos(aNewPos);
@@ -1919,7 +1919,7 @@ void _RestoreCntntIdx(std::vector<sal_uLong> &rSaveArr,
MarkBase* pMark = dynamic_cast<MarkBase*>(pMarkAccess->getAllMarksBegin()[aSave.GetCount()].get());
if (pMark)
{
- SwPosition aNewPos(pMark->GetOtherMarkPos());
+ SwPosition aNewPos(*pMark->GetOtherMarkPos());
aNewPos.nNode = rNd;
aNewPos.nContent.Assign(pCNd, std::min(aSave.GetContent(), nLen));
pMark->SetOtherMarkPos(aNewPos);
diff --git a/sw/source/core/doc/docdde.cxx b/sw/source/core/doc/docdde.cxx
index f321b05..1ac67e6 100644
--- a/sw/source/core/doc/docdde.cxx
+++ b/sw/source/core/doc/docdde.cxx
@@ -386,7 +386,7 @@ bool SwDoc::SelectServerObj( const OUString& rStr, SwPaM*& rpPam,
if(pBkmk->IsExpanded())
rpPam = new SwPaM(
pBkmk->GetMarkPos(),
- pBkmk->GetOtherMarkPos());
+ *pBkmk->GetOtherMarkPos());
return static_cast<bool>(rpPam);
}
diff --git a/sw/source/core/doc/docedt.cxx b/sw/source/core/doc/docedt.cxx
index 2ce11ac..a363cc3 100644
--- a/sw/source/core/doc/docedt.cxx
+++ b/sw/source/core/doc/docedt.cxx
@@ -2263,7 +2263,7 @@ bool SwDoc::ReplaceRangeImpl( SwPaM& rPam, const OUString& rStr,
*aDelPam.GetPoint() = pBkmk->GetMarkPos();
if(pBkmk->IsExpanded())
- *aDelPam.GetMark() = pBkmk->GetOtherMarkPos();
+ *aDelPam.GetMark() = *pBkmk->GetOtherMarkPos();
getIDocumentMarkAccess()->deleteMark(pBkmk);
pStt = aDelPam.Start();
pTxtNd = pStt->nNode.GetNode().GetTxtNode();
@@ -2360,7 +2360,7 @@ SetRedlineMode( eOld );
*rPam.GetPoint() = pBkmk->GetMarkPos();
if(pBkmk->IsExpanded())
- *rPam.GetMark() = pBkmk->GetOtherMarkPos();
+ *rPam.GetMark() = *pBkmk->GetOtherMarkPos();
getIDocumentMarkAccess()->deleteMark(pBkmk);
}
bJoinTxt = false;
diff --git a/sw/source/core/doc/swserv.cxx b/sw/source/core/doc/swserv.cxx
index d5e9924..a7d6d90 100644
--- a/sw/source/core/doc/swserv.cxx
+++ b/sw/source/core/doc/swserv.cxx
@@ -64,7 +64,7 @@ bool SwServerObject::GetData( uno::Any & rData,
{
// Span area
pPam = new SwPaM( CNTNT_TYPE.pBkmk->GetMarkPos(),
- CNTNT_TYPE.pBkmk->GetOtherMarkPos() );
+ *CNTNT_TYPE.pBkmk->GetOtherMarkPos() );
}
break;
diff --git a/sw/source/core/docnode/ndcopy.cxx b/sw/source/core/docnode/ndcopy.cxx
index 270e2c9..3c9aa78 100644
--- a/sw/source/core/docnode/ndcopy.cxx
+++ b/sw/source/core/docnode/ndcopy.cxx
@@ -173,8 +173,8 @@ namespace
if(pMark->IsExpanded())
{
aTmpPam.SetMark();
- lcl_NonCopyCount(rPam, aCorrIdx, pMark->GetOtherMarkPos().nNode.GetIndex(), nDelCount);
- lcl_SetCpyPos(pMark->GetOtherMarkPos(), rStt, *pCpyStt, *aTmpPam.GetMark(), nDelCount);
+ lcl_NonCopyCount(rPam, aCorrIdx, pMark->GetOtherMarkPos()->nNode.GetIndex(), nDelCount);
+ lcl_SetCpyPos(*pMark->GetOtherMarkPos(), rStt, *pCpyStt, *aTmpPam.GetMark(), nDelCount);
}
::sw::mark::IMark* const pNewMark = pDestDoc->getIDocumentMarkAccess()->makeMark(
diff --git a/sw/source/core/fields/reffld.cxx b/sw/source/core/fields/reffld.cxx
index f3205df..efdecbd 100644
--- a/sw/source/core/fields/reffld.cxx
+++ b/sw/source/core/fields/reffld.cxx
@@ -880,7 +880,7 @@ SwTxtNode* SwGetRefFieldType::FindAnchor( SwDoc* pDoc, const OUString& rRefMark,
*pEnd = pTxtNd->Len();
}
}
- else if(pBkmk->GetOtherMarkPos().nNode == pBkmk->GetMarkPos().nNode)
+ else if(pBkmk->GetOtherMarkPos()->nNode == pBkmk->GetMarkPos().nNode)
*pEnd = pBkmk->GetMarkEnd().nContent.GetIndex();
else
*pEnd = -1;
diff --git a/sw/source/core/inc/bookmrk.hxx b/sw/source/core/inc/bookmrk.hxx
index dc5cb5d..95ae338 100644
--- a/sw/source/core/inc/bookmrk.hxx
+++ b/sw/source/core/inc/bookmrk.hxx
@@ -53,26 +53,26 @@ namespace sw {
{ return *m_pPos1; }
virtual const OUString& GetName() const SAL_OVERRIDE
{ return m_aName; }
- virtual SwPosition& GetOtherMarkPos() const SAL_OVERRIDE
+ virtual SwPosition* GetOtherMarkPos() const SAL_OVERRIDE
{
OSL_PRECOND(IsExpanded(), "<SwPosition::GetOtherMarkPos(..)> - I have no other Pos set." );
- return *m_pPos2;
+ return m_pPos2.get();
}
virtual SwPosition& GetMarkStart() const SAL_OVERRIDE
{
if( !IsExpanded() ) return GetMarkPos( );
- if ( GetMarkPos( ) < GetOtherMarkPos( ) )
+ if ( GetMarkPos( ) < *GetOtherMarkPos( ) )
return GetMarkPos();
else
- return GetOtherMarkPos( );
+ return *GetOtherMarkPos( );
}
virtual SwPosition& GetMarkEnd() const SAL_OVERRIDE
{
if( !IsExpanded() ) return GetMarkPos();
- if ( GetMarkPos( ) >= GetOtherMarkPos( ) )
+ if ( GetMarkPos( ) >= *GetOtherMarkPos( ) )
return GetMarkPos( );
else
- return GetOtherMarkPos( );
+ return *GetOtherMarkPos( );
}
virtual bool IsCoveringPosition(const SwPosition& rPos) const SAL_OVERRIDE;
diff --git a/sw/source/core/inc/crossrefbookmark.hxx b/sw/source/core/inc/crossrefbookmark.hxx
index b43775d..22c4835 100644
--- a/sw/source/core/inc/crossrefbookmark.hxx
+++ b/sw/source/core/inc/crossrefbookmark.hxx
@@ -37,7 +37,7 @@ namespace sw {
const OUString& rPrefix);
// getters
- virtual SwPosition& GetOtherMarkPos() const SAL_OVERRIDE;
+ virtual SwPosition* GetOtherMarkPos() const SAL_OVERRIDE;
virtual SwPosition& GetMarkStart() const SAL_OVERRIDE
{ return *m_pPos1; }
virtual SwPosition& GetMarkEnd() const SAL_OVERRIDE
diff --git a/sw/source/core/undo/rolbck.cxx b/sw/source/core/undo/rolbck.cxx
index 20fa77b..fbaf20a 100644
--- a/sw/source/core/undo/rolbck.cxx
+++ b/sw/source/core/undo/rolbck.cxx
@@ -569,11 +569,11 @@ SwHistoryBookmark::SwHistoryBookmark(
, m_nNode(bSavePos ?
rBkmk.GetMarkPos().nNode.GetIndex() : 0)
, m_nOtherNode(bSaveOtherPos ?
- rBkmk.GetOtherMarkPos().nNode.GetIndex() : 0)
+ rBkmk.GetOtherMarkPos()->nNode.GetIndex() : 0)
, m_nCntnt(bSavePos ?
rBkmk.GetMarkPos().nContent.GetIndex() : 0)
, m_nOtherCntnt(bSaveOtherPos ?
- rBkmk.GetOtherMarkPos().nContent.GetIndex() :0)
+ rBkmk.GetOtherMarkPos()->nContent.GetIndex() :0)
, m_bSavePos(bSavePos)
, m_bSaveOtherPos(bSaveOtherPos)
, m_bHadOtherPos(rBkmk.IsExpanded())
@@ -642,7 +642,7 @@ void SwHistoryBookmark::SetInDoc( SwDoc* pDoc, bool )
"<SwHistoryBookmark::SetInDoc(..)>"
" - missing pos on old mark");
pPam->SetMark();
- *pPam->GetMark() = pMark->GetOtherMarkPos();
+ *pPam->GetMark() = *pMark->GetOtherMarkPos();
}
if(pPam.get())
diff --git a/sw/source/core/undo/undobj.cxx b/sw/source/core/undo/undobj.cxx
index 78697dc..becb4e3 100644
--- a/sw/source/core/undo/undobj.cxx
+++ b/sw/source/core/undo/undobj.cxx
@@ -710,8 +710,8 @@ void SwUndoSaveCntnt::DelCntntIndex( const SwPosition& rMark,
bSavePos = true;
}
if ( pBkmk->IsExpanded()
- && pStt->nNode <= pBkmk->GetOtherMarkPos().nNode
- && pBkmk->GetOtherMarkPos().nNode < pEnd->nNode )
+ && pStt->nNode <= pBkmk->GetOtherMarkPos()->nNode
+ && pBkmk->GetOtherMarkPos()->nNode < pEnd->nNode )
{
bSaveOtherPos = true;
}
@@ -737,10 +737,10 @@ void SwUndoSaveCntnt::DelCntntIndex( const SwPosition& rMark,
bSavePos = true;
}
if( pBkmk->IsExpanded() &&
- *pStt <= pBkmk->GetOtherMarkPos() && pBkmk->GetOtherMarkPos() <= *pEnd )
+ *pStt <= *pBkmk->GetOtherMarkPos() && *pBkmk->GetOtherMarkPos() <= *pEnd )
{
if ( bSavePos || bSaveOtherPos
- || ( pBkmk->GetOtherMarkPos() < *pEnd && pBkmk->GetOtherMarkPos() > *pStt ) )
+ || ( *pBkmk->GetOtherMarkPos() < *pEnd && *pBkmk->GetOtherMarkPos() > *pStt ) )
{
if( bMaybe )
bSavePos = true;
diff --git a/sw/source/core/unocore/unobkm.cxx b/sw/source/core/unocore/unobkm.cxx
index 9b08e0e..8af253e 100644
--- a/sw/source/core/unocore/unobkm.cxx
+++ b/sw/source/core/unocore/unobkm.cxx
@@ -288,7 +288,7 @@ throw (uno::RuntimeException, std::exception)
*m_pImpl->m_pDoc,
m_pImpl->m_pRegisteredBookmark->GetMarkPos(),
(m_pImpl->m_pRegisteredBookmark->IsExpanded())
- ? &m_pImpl->m_pRegisteredBookmark->GetOtherMarkPos() : NULL);
+ ? m_pImpl->m_pRegisteredBookmark->GetOtherMarkPos() : NULL);
}
void SAL_CALL SwXBookmark::dispose()
@@ -351,7 +351,7 @@ throw (uno::RuntimeException, std::exception)
if (m_pImpl->m_pRegisteredBookmark->IsExpanded())
{
aPam.SetMark();
- *aPam.GetMark() = m_pImpl->m_pRegisteredBookmark->GetOtherMarkPos();
+ *aPam.GetMark() = *m_pImpl->m_pRegisteredBookmark->GetOtherMarkPos();
}
SwRewriter aRewriter;
diff --git a/sw/source/core/unocore/unoobj2.cxx b/sw/source/core/unocore/unoobj2.cxx
index 1328b22..59c71d9 100644
--- a/sw/source/core/unocore/unoobj2.cxx
+++ b/sw/source/core/unocore/unoobj2.cxx
@@ -1021,7 +1021,7 @@ bool SwXTextRange::GetPositions(SwPaM& rToFill) const
if(pBkmk->IsExpanded())
{
rToFill.SetMark();
- *rToFill.GetMark() = pBkmk->GetOtherMarkPos();
+ *rToFill.GetMark() = *pBkmk->GetOtherMarkPos();
}
else
{
diff --git a/sw/source/filter/writer/writer.cxx b/sw/source/filter/writer/writer.cxx
index be1cfe6..5b3df67 100644
--- a/sw/source/filter/writer/writer.cxx
+++ b/sw/source/filter/writer/writer.cxx
@@ -96,9 +96,9 @@ void Writer_Impl::InsertBkmk(const ::sw::mark::IMark& rBkmk)
aBkmkNodePos.insert( SwBookmarkNodeTable::value_type( nNd, &rBkmk ) );
- if(rBkmk.IsExpanded() && rBkmk.GetOtherMarkPos().nNode != nNd)
+ if(rBkmk.IsExpanded() && rBkmk.GetOtherMarkPos()->nNode != nNd)
{
- nNd = rBkmk.GetOtherMarkPos().nNode.GetIndex();
+ nNd = rBkmk.GetOtherMarkPos()->nNode.GetIndex();
aBkmkNodePos.insert( SwBookmarkNodeTable::value_type( nNd, &rBkmk ));
}
}
@@ -417,8 +417,8 @@ sal_uInt16 Writer::GetBookmarks(const SwCntntNode& rNd, sal_Int32 nStt,
rArr.push_back( &rBkmk );
}
else if( rBkmk.IsExpanded() && nNd ==
- rBkmk.GetOtherMarkPos().nNode.GetIndex() && (nCntnt =
- rBkmk.GetOtherMarkPos().nContent.GetIndex() ) >= nStt &&
+ rBkmk.GetOtherMarkPos()->nNode.GetIndex() && (nCntnt =
+ rBkmk.GetOtherMarkPos()->nContent.GetIndex() ) >= nStt &&
nCntnt < nEnd )
{
rArr.push_back( &rBkmk );
diff --git a/sw/source/filter/ww8/wrtww8.cxx b/sw/source/filter/ww8/wrtww8.cxx
index d110c90..27b40fe 100644
--- a/sw/source/filter/ww8/wrtww8.cxx
+++ b/sw/source/filter/ww8/wrtww8.cxx
@@ -1341,7 +1341,7 @@ void WW8Export::AppendBookmarks( const SwTxtNode& rNd, sal_Int32 nAktPos, sal_In
const SwPosition* pPos = &rBkmk.GetMarkPos();
const SwPosition* pOPos = 0;
if(rBkmk.IsExpanded())
- pOPos = &rBkmk.GetOtherMarkPos();
+ pOPos = rBkmk.GetOtherMarkPos();
if( pOPos && pOPos->nNode == pPos->nNode &&
pOPos->nContent < pPos->nContent )
{
commit 607218e88f8bfe95d0ee13fb7dcc6fa3c94b31e8
Author: Stephan Bergmann <sbergman at redhat.com>
Date: Tue Jun 10 19:19:01 2014 +0200
Dead code
Change-Id: Ib585af0755bdc6f7aeebc89d5d109d3bb6084e22
diff --git a/sc/source/ui/inc/optload.hxx b/sc/source/ui/inc/optload.hxx
deleted file mode 100644
index ef85e37..0000000
--- a/sc/source/ui/inc/optload.hxx
+++ /dev/null
@@ -1,59 +0,0 @@
-/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
-/*
- * This file is part of the LibreOffice project.
- *
- * This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/.
- *
- * This file incorporates work covered by the following license notice:
- *
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed
- * with this work for additional information regarding copyright
- * ownership. The ASF licenses this file to you under the Apache
- * License, Version 2.0 (the "License"); you may not use this file
- * except in compliance with the License. You may obtain a copy of
- * the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-#ifndef INCLUDED_SC_SOURCE_UI_INC_OPTLOAD_HXX
-#define INCLUDED_SC_SOURCE_UI_INC_OPTLOAD_HXX
-
-#include <sfx2/tabdlg.hxx>
-
-#include <vcl/group.hxx>
-#include <vcl/fixed.hxx>
-
-class ScDocument;
-
-class ScLoadOptPage : public SfxTabPage
-{
-private:
- FixedText aLinkFT;
- RadioButton aAlwaysRB;
- RadioButton aRequestRB;
- RadioButton aNeverRB;
- CheckBox aDocOnlyCB;
- GroupBox aLinkGB;
- ScDocument *pDoc;
-
- DECL_LINK( UpdateHdl, CheckBox* );
-
-public:
- ScLoadOptPage( Window* pParent,
- const SfxItemSet& rSet );
- ~ScLoadOptPage();
-
- static SfxTabPage* Create( Window* pParent,
- const SfxItemSet& rAttrSet);
-
- void SetDocument(ScDocument*);
-
- virtual bool FillItemSet( SfxItemSet& rSet );
- virtual void Reset( const SfxItemSet& rSet );
-};
-
-#endif
-
-
-/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
commit bd6ab68d19591ee0be2b830affcad37efa3185cf
Author: Stephan Bergmann <sbergman at redhat.com>
Date: Tue Jun 10 19:09:22 2014 +0200
-Werror,-Wundefined-bool-conversion
Change-Id: I7b1252656a6ada4ab86151462629aaa952011b70
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx b/sw/source/filter/ww8/docxattributeoutput.cxx
index 34b0c06..645688f 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -5457,7 +5457,7 @@ void DocxAttributeOutput::CharEscapement( const SvxEscapementItem& rEscapement )
FSNS( XML_w, XML_val ), sIss.getStr(), FSEND );
const SvxFontHeightItem& rItem = static_cast<const SvxFontHeightItem&>(m_rExport.GetItem(RES_CHRATR_FONTSIZE));
- if (&rItem != NULL && (sIss.isEmpty() || sIss.match(OString("baseline"))))
+ if (sIss.isEmpty() || sIss.match(OString("baseline")))
{
long nHeight = rItem.GetHeight();
OString sPos = OString::number( ( nHeight * nEsc + 500 ) / 1000 );
commit bf86a7a44900a52f8cbfed71d5cf8b32a4852d20
Author: Stephan Bergmann <sbergman at redhat.com>
Date: Tue Jun 10 19:06:58 2014 +0200
-Werror,-Wundefined-bool-conversion
Change-Id: I52d9ae4daf8e4f904f18b150a4adaad9d2921e61
diff --git a/sw/source/uibase/utlui/content.cxx b/sw/source/uibase/utlui/content.cxx
index d55b498..0a838e8 100644
--- a/sw/source/uibase/utlui/content.cxx
+++ b/sw/source/uibase/utlui/content.cxx
@@ -2737,13 +2737,9 @@ void SwContentTree::KeyInput(const KeyEvent& rEvent)
{
SwEditWin& pEditWindow =
pActiveShell->GetView().GetEditWin();
- if( &pEditWindow )
- {
- KeyCode tempKeycode( KEY_ESCAPE );
- KeyEvent rKEvt( 0 , tempKeycode );
- ((Window*)&pEditWindow)->KeyInput( rKEvt );
-
- }
+ KeyCode tempKeycode( KEY_ESCAPE );
+ KeyEvent rKEvt( 0 , tempKeycode );
+ ((Window*)&pEditWindow)->KeyInput( rKEvt );
//rView.GetEditWin().GrabFocus();
}
}
More information about the Libreoffice-commits
mailing list