[Libreoffice-commits] core.git: sw/inc sw/source
Miklos Vajna (via logerrit)
logerrit at kemper.freedesktop.org
Mon Sep 21 07:46:50 UTC 2020
sw/inc/docufld.hxx | 2 -
sw/inc/edglbldc.hxx | 14 +++----
sw/source/core/edit/edglbldc.cxx | 18 ++++-----
sw/source/core/fields/ddefld.cxx | 41 ++++++++++-----------
sw/source/core/fields/docufld.cxx | 4 +-
sw/source/core/fields/textapi.cxx | 74 +++++++++++++++++++-------------------
sw/source/core/inc/textapi.hxx | 2 -
7 files changed, 78 insertions(+), 77 deletions(-)
New commits:
commit df86ec5d465875a5c59017c24578ca9153440d0b
Author: Miklos Vajna <vmiklos at collabora.com>
AuthorDate: Mon Sep 21 08:04:00 2020 +0200
Commit: Miklos Vajna <vmiklos at collabora.com>
CommitDate: Mon Sep 21 09:46:12 2020 +0200
sw: prefix members of SwGlblDocContent, SwIntrnlRefLink, SwPostItField ...
... and SwTextAPIEditSource
See tdf#94879 for motivation.
Change-Id: If9af8e5fb5d750547a1be9229f69bb79ae4bb0e2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103080
Reviewed-by: Miklos Vajna <vmiklos at collabora.com>
Tested-by: Jenkins
diff --git a/sw/inc/docufld.hxx b/sw/inc/docufld.hxx
index 5c8187dafa13..bc7ef9219093 100644
--- a/sw/inc/docufld.hxx
+++ b/sw/inc/docufld.hxx
@@ -451,7 +451,7 @@ class SW_DLLPUBLIC SwPostItField final : public SwField
sal_uInt32 m_nPostItId;
public:
- static sal_uInt32 m_nLastPostItId;
+ static sal_uInt32 s_nLastPostItId;
SwPostItField( SwPostItFieldType*,
const OUString& rAuthor,
diff --git a/sw/inc/edglbldc.hxx b/sw/inc/edglbldc.hxx
index 6cac147f56e5..9ab0bf4e4b6b 100644
--- a/sw/inc/edglbldc.hxx
+++ b/sw/inc/edglbldc.hxx
@@ -34,12 +34,12 @@ enum GlobalDocContentType {
class SwGlblDocContent
{
- GlobalDocContentType eType;
- sal_uLong nDocPos;
+ GlobalDocContentType m_eType;
+ sal_uLong m_nDocPos;
union {
const SwTOXBase* pTOX;
const SwSection* pSect;
- } PTR;
+ } m_PTR;
public:
SwGlblDocContent( sal_uLong nPos );
@@ -47,12 +47,12 @@ public:
SwGlblDocContent( const SwSection* pSect );
/// Query contents.
- GlobalDocContentType GetType() const { return eType; }
+ GlobalDocContentType GetType() const { return m_eType; }
const SwSection* GetSection() const
- { return GLBLDOC_SECTION == eType ? PTR.pSect : nullptr; }
+ { return GLBLDOC_SECTION == m_eType ? m_PTR.pSect : nullptr; }
const SwTOXBase* GetTOX() const
- { return GLBLDOC_TOXBASE == eType ? PTR.pTOX : nullptr; }
- sal_uLong GetDocPos() const { return nDocPos; }
+ { return GLBLDOC_TOXBASE == m_eType ? m_PTR.pTOX : nullptr; }
+ sal_uLong GetDocPos() const { return m_nDocPos; }
/// For sorting.
bool operator==( const SwGlblDocContent& rCmp ) const
diff --git a/sw/source/core/edit/edglbldc.cxx b/sw/source/core/edit/edglbldc.cxx
index 1849f6990c92..7c30b156db0c 100644
--- a/sw/source/core/edit/edglbldc.cxx
+++ b/sw/source/core/edit/edglbldc.cxx
@@ -358,27 +358,27 @@ void SwEditShell::GotoGlobalDocContent( const SwGlblDocContent& rPos )
SwGlblDocContent::SwGlblDocContent( sal_uLong nPos )
{
- eType = GLBLDOC_UNKNOWN;
- PTR.pTOX = nullptr;
- nDocPos = nPos;
+ m_eType = GLBLDOC_UNKNOWN;
+ m_PTR.pTOX = nullptr;
+ m_nDocPos = nPos;
}
SwGlblDocContent::SwGlblDocContent( const SwTOXBaseSection* pTOX )
{
- eType = GLBLDOC_TOXBASE;
- PTR.pTOX = pTOX;
+ m_eType = GLBLDOC_TOXBASE;
+ m_PTR.pTOX = pTOX;
const SwSectionNode* pSectNd = pTOX->GetFormat()->GetSectionNode();
- nDocPos = pSectNd ? pSectNd->GetIndex() : 0;
+ m_nDocPos = pSectNd ? pSectNd->GetIndex() : 0;
}
SwGlblDocContent::SwGlblDocContent( const SwSection* pSect )
{
- eType = GLBLDOC_SECTION;
- PTR.pSect = pSect;
+ m_eType = GLBLDOC_SECTION;
+ m_PTR.pSect = pSect;
const SwSectionNode* pSectNd = pSect->GetFormat()->GetSectionNode();
- nDocPos = pSectNd ? pSectNd->GetIndex() : 0;
+ m_nDocPos = pSectNd ? pSectNd->GetIndex() : 0;
}
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sw/source/core/fields/ddefld.cxx b/sw/source/core/fields/ddefld.cxx
index 0b80f7feb2ae..9c4ba9c762cb 100644
--- a/sw/source/core/fields/ddefld.cxx
+++ b/sw/source/core/fields/ddefld.cxx
@@ -45,11 +45,12 @@ namespace {
class SwIntrnlRefLink : public SwBaseLink
{
- SwDDEFieldType& rFieldType;
+ SwDDEFieldType& m_rFieldType;
+
public:
- SwIntrnlRefLink( SwDDEFieldType& rType, SfxLinkUpdateMode nUpdateType )
- : SwBaseLink( nUpdateType, SotClipboardFormatId::STRING ),
- rFieldType( rType )
+ SwIntrnlRefLink(SwDDEFieldType& rType, SfxLinkUpdateMode nUpdateType)
+ : SwBaseLink(nUpdateType, SotClipboardFormatId::STRING)
+ , m_rFieldType(rType)
{}
virtual void Closed() override;
@@ -87,9 +88,9 @@ public:
if( bDel )
sStr = sStr.copy( 0, n );
- rFieldType.SetExpansion( sStr );
+ m_rFieldType.SetExpansion(sStr);
// set Expansion first! (otherwise this flag will be deleted)
- rFieldType.SetCRLFDelFlag( bDel );
+ m_rFieldType.SetCRLFDelFlag(bDel);
}
break;
@@ -98,20 +99,20 @@ public:
return SUCCESS;
}
- OSL_ENSURE( rFieldType.GetDoc(), "no pDoc" );
+ OSL_ENSURE(m_rFieldType.GetDoc(), "no pDoc");
// no dependencies left?
- if( rFieldType.HasWriterListeners() && !rFieldType.IsModifyLocked() && !ChkNoDataFlag() )
+ if (m_rFieldType.HasWriterListeners() && !m_rFieldType.IsModifyLocked() && !ChkNoDataFlag())
{
- SwViewShell* pSh = rFieldType.GetDoc()->getIDocumentLayoutAccess().GetCurrentViewShell();
- SwEditShell* pESh = rFieldType.GetDoc()->GetEditShell();
+ SwViewShell* pSh = m_rFieldType.GetDoc()->getIDocumentLayoutAccess().GetCurrentViewShell();
+ SwEditShell* pESh = m_rFieldType.GetDoc()->GetEditShell();
// Search for fields. If no valid found, disconnect.
SwMsgPoolItem aUpdateDDE( RES_UPDATEDDETBL );
- rFieldType.LockModify();
+ m_rFieldType.LockModify();
std::vector<SwFormatField*> vFields;
- rFieldType.GatherFields(vFields, false);
+ m_rFieldType.GatherFields(vFields, false);
if(vFields.size())
{
if(pESh)
@@ -127,7 +128,7 @@ public:
pFormatField->UpdateTextNode( nullptr, &aUpdateDDE );
}
- rFieldType.UnlockModify();
+ m_rFieldType.UnlockModify();
if(vFields.size())
{
@@ -146,15 +147,15 @@ public:
void SwIntrnlRefLink::Closed()
{
- if( rFieldType.GetDoc() && !rFieldType.GetDoc()->IsInDtor() )
+ if (m_rFieldType.GetDoc() && !m_rFieldType.GetDoc()->IsInDtor())
{
// advise goes, convert all fields into text?
- SwViewShell* pSh = rFieldType.GetDoc()->getIDocumentLayoutAccess().GetCurrentViewShell();
- SwEditShell* pESh = rFieldType.GetDoc()->GetEditShell();
+ SwViewShell* pSh = m_rFieldType.GetDoc()->getIDocumentLayoutAccess().GetCurrentViewShell();
+ SwEditShell* pESh = m_rFieldType.GetDoc()->GetEditShell();
if( pESh )
{
pESh->StartAllAction();
- pESh->FieldToText( &rFieldType );
+ pESh->FieldToText(&m_rFieldType);
pESh->EndAllAction();
}
else
@@ -173,15 +174,15 @@ const SwNode* SwIntrnlRefLink::GetAnchor() const
{
// here, any anchor of the normal NodesArray should be sufficient
const SwNode* pNd = nullptr;
- rFieldType.CallSwClientNotify(sw::LinkAnchorSearchHint(rFieldType.GetDoc()->GetNodes(), pNd));
+ m_rFieldType.CallSwClientNotify(
+ sw::LinkAnchorSearchHint(m_rFieldType.GetDoc()->GetNodes(), pNd));
return pNd;
}
bool SwIntrnlRefLink::IsInRange( sal_uLong nSttNd, sal_uLong nEndNd ) const
{
bool bInRange = false;
- rFieldType.CallSwClientNotify(sw::InRangeSearchHint(
- nSttNd, nEndNd, bInRange));
+ m_rFieldType.CallSwClientNotify(sw::InRangeSearchHint(nSttNd, nEndNd, bInRange));
return bInRange;
}
diff --git a/sw/source/core/fields/docufld.cxx b/sw/source/core/fields/docufld.cxx
index 8529b27a05ad..35bf08b5a96e 100644
--- a/sw/source/core/fields/docufld.cxx
+++ b/sw/source/core/fields/docufld.cxx
@@ -1720,7 +1720,7 @@ std::unique_ptr<SwFieldType> SwPostItFieldType::Copy() const
// PostIt field
-sal_uInt32 SwPostItField::m_nLastPostItId = 1;
+sal_uInt32 SwPostItField::s_nLastPostItId = 1;
SwPostItField::SwPostItField( SwPostItFieldType* pT,
const OUString& rAuthor,
@@ -1739,7 +1739,7 @@ SwPostItField::SwPostItField( SwPostItFieldType* pT,
, m_aDateTime( rDateTime )
, m_bResolved( bResolved )
{
- m_nPostItId = nPostItId == 0 ? m_nLastPostItId++ : nPostItId;
+ m_nPostItId = nPostItId == 0 ? s_nLastPostItId++ : nPostItId;
}
SwPostItField::~SwPostItField()
diff --git a/sw/source/core/fields/textapi.cxx b/sw/source/core/fields/textapi.cxx
index 070e1041f449..e0eb014d303a 100644
--- a/sw/source/core/fields/textapi.cxx
+++ b/sw/source/core/fields/textapi.cxx
@@ -84,8 +84,8 @@ SwTextAPIEditSource::SwTextAPIEditSource( const SwTextAPIEditSource& rSource )
: SvxEditSource( *this )
{
// shallow copy; uses internal refcounting
- pImpl = rSource.pImpl;
- pImpl->mnRef++;
+ m_pImpl = rSource.m_pImpl;
+ m_pImpl->mnRef++;
}
std::unique_ptr<SvxEditSource> SwTextAPIEditSource::Clone() const
@@ -99,93 +99,93 @@ void SwTextAPIEditSource::UpdateData()
}
SwTextAPIEditSource::SwTextAPIEditSource(SwDoc* pDoc)
-: pImpl(new SwTextAPIEditSource_Impl)
+: m_pImpl(new SwTextAPIEditSource_Impl)
{
- pImpl->mpPool = &pDoc->GetDocShell()->GetPool();
- pImpl->mpDoc = pDoc;
- pImpl->mnRef = 1;
+ m_pImpl->mpPool = &pDoc->GetDocShell()->GetPool();
+ m_pImpl->mpDoc = pDoc;
+ m_pImpl->mnRef = 1;
}
SwTextAPIEditSource::~SwTextAPIEditSource()
{
- if (!--pImpl->mnRef)
- delete pImpl;
+ if (!--m_pImpl->mnRef)
+ delete m_pImpl;
}
void SwTextAPIEditSource::Dispose()
{
- pImpl->mpPool=nullptr;
- pImpl->mpDoc=nullptr;
- pImpl->mpTextForwarder.reset();
- pImpl->mpOutliner.reset();
+ m_pImpl->mpPool=nullptr;
+ m_pImpl->mpDoc=nullptr;
+ m_pImpl->mpTextForwarder.reset();
+ m_pImpl->mpOutliner.reset();
}
SvxTextForwarder* SwTextAPIEditSource::GetTextForwarder()
{
- if( !pImpl->mpPool )
+ if( !m_pImpl->mpPool )
return nullptr; // mpPool == 0 can be used to flag this as disposed
- if( !pImpl->mpOutliner )
+ if( !m_pImpl->mpOutliner )
{
//init draw model first
- pImpl->mpDoc->getIDocumentDrawModelAccess().GetOrCreateDrawModel();
- pImpl->mpOutliner.reset(new Outliner(pImpl->mpPool, OutlinerMode::TextObject));
- pImpl->mpDoc->SetCalcFieldValueHdl(pImpl->mpOutliner.get());
+ m_pImpl->mpDoc->getIDocumentDrawModelAccess().GetOrCreateDrawModel();
+ m_pImpl->mpOutliner.reset(new Outliner(m_pImpl->mpPool, OutlinerMode::TextObject));
+ m_pImpl->mpDoc->SetCalcFieldValueHdl(m_pImpl->mpOutliner.get());
}
- if( !pImpl->mpTextForwarder )
+ if( !m_pImpl->mpTextForwarder )
{
- pImpl->mpTextForwarder.reset(new SvxOutlinerForwarder(*pImpl->mpOutliner, false));
+ m_pImpl->mpTextForwarder.reset(new SvxOutlinerForwarder(*m_pImpl->mpOutliner, false));
}
- return pImpl->mpTextForwarder.get();
+ return m_pImpl->mpTextForwarder.get();
}
void SwTextAPIEditSource::SetText( OutlinerParaObject const & rText )
{
- if ( pImpl->mpPool )
+ if ( m_pImpl->mpPool )
{
- if( !pImpl->mpOutliner )
+ if( !m_pImpl->mpOutliner )
{
//init draw model first
- pImpl->mpDoc->getIDocumentDrawModelAccess().GetOrCreateDrawModel();
- pImpl->mpOutliner.reset(new Outliner(pImpl->mpPool, OutlinerMode::TextObject));
- pImpl->mpDoc->SetCalcFieldValueHdl(pImpl->mpOutliner.get());
+ m_pImpl->mpDoc->getIDocumentDrawModelAccess().GetOrCreateDrawModel();
+ m_pImpl->mpOutliner.reset(new Outliner(m_pImpl->mpPool, OutlinerMode::TextObject));
+ m_pImpl->mpDoc->SetCalcFieldValueHdl(m_pImpl->mpOutliner.get());
}
- pImpl->mpOutliner->SetText( rText );
+ m_pImpl->mpOutliner->SetText( rText );
}
}
void SwTextAPIEditSource::SetString( const OUString& rText )
{
- if ( !pImpl->mpPool )
+ if ( !m_pImpl->mpPool )
return;
- if( !pImpl->mpOutliner )
+ if( !m_pImpl->mpOutliner )
{
//init draw model first
- pImpl->mpDoc->getIDocumentDrawModelAccess().GetOrCreateDrawModel();
- pImpl->mpOutliner.reset(new Outliner(pImpl->mpPool, OutlinerMode::TextObject));
- pImpl->mpDoc->SetCalcFieldValueHdl(pImpl->mpOutliner.get());
+ m_pImpl->mpDoc->getIDocumentDrawModelAccess().GetOrCreateDrawModel();
+ m_pImpl->mpOutliner.reset(new Outliner(m_pImpl->mpPool, OutlinerMode::TextObject));
+ m_pImpl->mpDoc->SetCalcFieldValueHdl(m_pImpl->mpOutliner.get());
}
else
- pImpl->mpOutliner->Clear();
- pImpl->mpOutliner->Insert( rText );
+ m_pImpl->mpOutliner->Clear();
+ m_pImpl->mpOutliner->Insert( rText );
}
std::unique_ptr<OutlinerParaObject> SwTextAPIEditSource::CreateText()
{
- if ( pImpl->mpPool && pImpl->mpOutliner )
- return pImpl->mpOutliner->CreateParaObject();
+ if ( m_pImpl->mpPool && m_pImpl->mpOutliner )
+ return m_pImpl->mpOutliner->CreateParaObject();
else
return nullptr;
}
OUString SwTextAPIEditSource::GetText() const
{
- if ( pImpl->mpPool && pImpl->mpOutliner )
- return pImpl->mpOutliner->GetEditEngine().GetText();
+ if ( m_pImpl->mpPool && m_pImpl->mpOutliner )
+ return m_pImpl->mpOutliner->GetEditEngine().GetText();
else
return OUString();
}
diff --git a/sw/source/core/inc/textapi.hxx b/sw/source/core/inc/textapi.hxx
index 3362621d7b03..cf24742aa0f8 100644
--- a/sw/source/core/inc/textapi.hxx
+++ b/sw/source/core/inc/textapi.hxx
@@ -29,7 +29,7 @@ class SwDoc;
struct SwTextAPIEditSource_Impl;
class SwTextAPIEditSource : public SvxEditSource
{
- SwTextAPIEditSource_Impl* pImpl;
+ SwTextAPIEditSource_Impl* m_pImpl;
virtual std::unique_ptr<SvxEditSource> Clone() const override;
virtual SvxTextForwarder* GetTextForwarder() override;
More information about the Libreoffice-commits
mailing list