sw/source/filter/xml/XMLRedlineImportHelper.cxx |   24 ++++++++++++------------
 sw/source/filter/xml/xmlfmt.cxx                 |   24 ++++++++++++------------
 sw/source/filter/xml/xmlfmte.cxx                |   18 +++++++++---------
 sw/source/filter/xml/xmlitem.cxx                |   24 ++++++++++++------------
 sw/source/filter/xml/xmlitem.hxx                |    8 ++++----
 5 files changed, 49 insertions(+), 49 deletions(-)

New commits:
commit 10e36bac8dcf25372467ef97a28ad859922e6a3c
Author:     Miklos Vajna <vmik...@collabora.com>
AuthorDate: Mon Jul 12 09:07:29 2021 +0200
Commit:     Miklos Vajna <vmik...@collabora.com>
CommitDate: Mon Jul 12 17:37:00 2021 +0200

    sw: prefix members of SwXMLAutoStylePoolP, SwXMLConditionContext_Impl, ...
    
    ... SwXMLItemSetContext and XTextRangeOrNodeIndexPosition
    
    See tdf#94879 for motivation.
    
    Change-Id: Ied25f44dd03fa7760613d89e11003028ce60982a
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118764
    Reviewed-by: Miklos Vajna <vmik...@collabora.com>
    Tested-by: Jenkins

diff --git a/sw/source/filter/xml/XMLRedlineImportHelper.cxx 
b/sw/source/filter/xml/XMLRedlineImportHelper.cxx
index 2a4d939b2d0b..803cc78a04bb 100644
--- a/sw/source/filter/xml/XMLRedlineImportHelper.cxx
+++ b/sw/source/filter/xml/XMLRedlineImportHelper.cxx
@@ -87,8 +87,8 @@ namespace {
 
 class XTextRangeOrNodeIndexPosition
 {
-    Reference<XTextRange> xRange;
-    std::unique_ptr<SwNodeIndex> pIndex;    // pIndex will point to the 
*previous* node
+    Reference<XTextRange> m_xRange;
+    std::unique_ptr<SwNodeIndex> m_pIndex;    // pIndex will point to the 
*previous* node
 
 public:
     XTextRangeOrNodeIndexPosition();
@@ -111,15 +111,15 @@ 
XTextRangeOrNodeIndexPosition::XTextRangeOrNodeIndexPosition()
 
 void XTextRangeOrNodeIndexPosition::Set( Reference<XTextRange> const & rRange )
 {
-    xRange = rRange->getStart();    // set bookmark
-    pIndex.reset();
+    m_xRange = rRange->getStart();    // set bookmark
+    m_pIndex.reset();
 }
 
 void XTextRangeOrNodeIndexPosition::Set( SwNodeIndex const & rIndex )
 {
-    pIndex.reset( new SwNodeIndex(rIndex) );
-    (*pIndex)-- ;   // previous node!!!
-    xRange = nullptr;
+    m_pIndex.reset( new SwNodeIndex(rIndex) );
+    (*m_pIndex)-- ;   // previous node!!!
+    m_xRange = nullptr;
 }
 
 void XTextRangeOrNodeIndexPosition::SetAsNodeIndex(
@@ -149,17 +149,17 @@ 
XTextRangeOrNodeIndexPosition::CopyPositionInto(SwPosition& rPos, SwDoc & rDoc)
     OSL_ENSURE(IsValid(), "Can't get Position");
 
     // create PAM from start cursor (if no node index is present)
-    if (nullptr == pIndex)
+    if (nullptr == m_pIndex)
     {
         SwUnoInternalPaM aUnoPaM(rDoc);
-        bool bSuccess = ::sw::XTextRangeToSwPaM(aUnoPaM, xRange);
+        bool bSuccess = ::sw::XTextRangeToSwPaM(aUnoPaM, m_xRange);
         OSL_ENSURE(bSuccess, "illegal range");
 
         rPos = *aUnoPaM.GetPoint();
     }
     else
     {
-        rPos.nNode = *pIndex;
+        rPos.nNode = *m_pIndex;
         rPos.nNode++;           // pIndex points to previous index !!!
         rPos.nContent.Assign( rPos.nNode.GetNode().GetContentNode(), 0 );
     }
@@ -169,12 +169,12 @@ SwDoc* XTextRangeOrNodeIndexPosition::GetDoc()
 {
     OSL_ENSURE(IsValid(), "Can't get Doc");
 
-    return (nullptr != pIndex) ? &pIndex->GetNodes().GetDoc() : 
lcl_GetDocViaTunnel(xRange);
+    return (nullptr != m_pIndex) ? &m_pIndex->GetNodes().GetDoc() : 
lcl_GetDocViaTunnel(m_xRange);
 }
 
 bool XTextRangeOrNodeIndexPosition::IsValid() const
 {
-    return ( xRange.is() || (pIndex != nullptr) );
+    return ( m_xRange.is() || (m_pIndex != nullptr) );
 }
 
 // RedlineInfo: temporary storage for redline data
diff --git a/sw/source/filter/xml/xmlfmt.cxx b/sw/source/filter/xml/xmlfmt.cxx
index b1d5c7f8a019..7472cd50b068 100644
--- a/sw/source/filter/xml/xmlfmt.cxx
+++ b/sw/source/filter/xml/xmlfmt.cxx
@@ -199,10 +199,10 @@ namespace {
 
 class SwXMLConditionContext_Impl : public SvXMLImportContext
 {
-    Master_CollCondition nCondition;
-    sal_uInt32 nSubCondition;
+    Master_CollCondition m_nCondition;
+    sal_uInt32 m_nSubCondition;
 
-    OUString sApplyStyle;
+    OUString m_sApplyStyle;
 
 public:
 
@@ -210,11 +210,11 @@ public:
             SvXMLImport& rImport, sal_Int32 nElement,
             const uno::Reference< xml::sax::XFastAttributeList > & xAttrList );
 
-    bool IsValid() const { return Master_CollCondition::NONE != nCondition; }
+    bool IsValid() const { return Master_CollCondition::NONE != m_nCondition; }
 
-    Master_CollCondition getCondition() const { return nCondition; }
-    sal_uInt32 getSubCondition() const { return nSubCondition; }
-    OUString const &getApplyStyle() const { return sApplyStyle; }
+    Master_CollCondition getCondition() const { return m_nCondition; }
+    sal_uInt32 getSubCondition() const { return m_nSubCondition; }
+    OUString const &getApplyStyle() const { return m_sApplyStyle; }
 };
 
 }
@@ -223,8 +223,8 @@ SwXMLConditionContext_Impl::SwXMLConditionContext_Impl(
             SvXMLImport& rImport, sal_Int32 /*nElement*/,
             const uno::Reference< xml::sax::XFastAttributeList > & xAttrList ) 
:
     SvXMLImportContext( rImport ),
-    nCondition( Master_CollCondition::NONE ),
-    nSubCondition( 0 )
+    m_nCondition( Master_CollCondition::NONE ),
+    m_nSubCondition( 0 )
 {
     for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList ))
     {
@@ -236,13 +236,13 @@ SwXMLConditionContext_Impl::SwXMLConditionContext_Impl(
                 SwXMLConditionParser_Impl aCondParser( sValue );
                 if( aCondParser.IsValid() )
                 {
-                    nCondition = aCondParser.GetCondition();
-                    nSubCondition = aCondParser.GetSubCondition();
+                    m_nCondition = aCondParser.GetCondition();
+                    m_nSubCondition = aCondParser.GetSubCondition();
                 }
                 break;
             }
             case XML_ELEMENT(STYLE, XML_APPLY_STYLE_NAME):
-                sApplyStyle = sValue;
+                m_sApplyStyle = sValue;
                 break;
             default:
                 XMLOFF_WARN_UNKNOWN("sw", aIter);
diff --git a/sw/source/filter/xml/xmlfmte.cxx b/sw/source/filter/xml/xmlfmte.cxx
index fc582dd39eda..f7866c5290f8 100644
--- a/sw/source/filter/xml/xmlfmte.cxx
+++ b/sw/source/filter/xml/xmlfmte.cxx
@@ -263,9 +263,9 @@ namespace {
 
 class SwXMLAutoStylePoolP : public SvXMLAutoStylePoolP
 {
-    SvXMLExport& rExport;
-    const OUString sListStyleName;
-    const OUString sMasterPageName;
+    SvXMLExport& m_rExport;
+    const OUString m_sListStyleName;
+    const OUString m_sMasterPageName;
 
 protected:
 
@@ -312,12 +312,12 @@ void SwXMLAutoStylePoolP::exportStyleAttributes(
                     // #i70748# - export also empty list styles
                     if( !sStyleName.isEmpty() )
                     {
-                        OUString sTmp = 
rExport.GetTextParagraphExport()->GetListAutoStylePool().Find( sStyleName );
+                        OUString sTmp = 
m_rExport.GetTextParagraphExport()->GetListAutoStylePool().Find( sStyleName );
                         if( !sTmp.isEmpty() )
                             sStyleName = sTmp;
                     }
                     GetExport().AddAttribute( XML_NAMESPACE_STYLE,
-                          sListStyleName,
+                          m_sListStyleName,
                           GetExport().EncodeStyleName( sStyleName ) );
                 }
                 break;
@@ -326,7 +326,7 @@ void SwXMLAutoStylePoolP::exportStyleAttributes(
                     OUString sStyleName;
                     rProperty.maValue >>= sStyleName;
                     GetExport().AddAttribute( XML_NAMESPACE_STYLE,
-                                  sMasterPageName,
+                                  m_sMasterPageName,
                                   GetExport().EncodeStyleName( sStyleName ) );
                 }
                 break;
@@ -337,9 +337,9 @@ void SwXMLAutoStylePoolP::exportStyleAttributes(
 
 SwXMLAutoStylePoolP::SwXMLAutoStylePoolP(SvXMLExport& rExp ) :
     SvXMLAutoStylePoolP( rExp ),
-    rExport( rExp ),
-    sListStyleName( GetXMLToken( XML_LIST_STYLE_NAME ) ),
-    sMasterPageName( GetXMLToken( XML_MASTER_PAGE_NAME ) )
+    m_rExport( rExp ),
+    m_sListStyleName( GetXMLToken( XML_LIST_STYLE_NAME ) ),
+    m_sMasterPageName( GetXMLToken( XML_MASTER_PAGE_NAME ) )
 {
 }
 
diff --git a/sw/source/filter/xml/xmlitem.cxx b/sw/source/filter/xml/xmlitem.cxx
index e97eb5f6922d..3bd1bad6b234 100644
--- a/sw/source/filter/xml/xmlitem.cxx
+++ b/sw/source/filter/xml/xmlitem.cxx
@@ -33,21 +33,21 @@ SwXMLItemSetContext::SwXMLItemSetContext( SvXMLImport& 
rImp, sal_Int32 /*nElemen
                                           SvXMLImportItemMapper& rIMap,
                                           const SvXMLUnitConverter& 
rUnitConverter ):
     SvXMLImportContext( rImp ),
-    rItemSet( rISet ),
-    rIMapper( rIMap ),
-    rUnitConv( rUnitConverter )
+    m_rItemSet( rISet ),
+    m_rIMapper( rIMap ),
+    m_rUnitConv( rUnitConverter )
 {
-    rIMap.importXML( rItemSet, xAttrList, rUnitConv,
+    rIMap.importXML( m_rItemSet, xAttrList, m_rUnitConv,
                            GetImport().GetNamespaceMap() );
 }
 
 SwXMLItemSetContext::~SwXMLItemSetContext()
 {
-    if( xBackground.is() )
+    if( m_xBackground.is() )
     {
         const SvxBrushItem& rItem =
-            
static_cast<SwXMLBrushItemImportContext*>(xBackground.get())->GetItem();
-        rItemSet.Put( rItem );
+            
static_cast<SwXMLBrushItemImportContext*>(m_xBackground.get())->GetItem();
+        m_rItemSet.Put( rItem );
     }
 }
 
@@ -55,7 +55,7 @@ css::uno::Reference< css::xml::sax::XFastContextHandler > 
SwXMLItemSetContext::c
     sal_Int32 nElement,
     const uno::Reference< xml::sax::XFastAttributeList >& xAttrList )
 {
-    SvXMLItemMapEntriesRef xMapEntries = rIMapper.getMapEntries();
+    SvXMLItemMapEntriesRef xMapEntries = m_rIMapper.getMapEntries();
     SvXMLItemMapEntry const * pEntry = xMapEntries->getByName( nElement );
 
     if( pEntry && 0 != (pEntry->nMemberId & MID_SW_FLAG_ELEMENT_ITEM_IMPORT) )
@@ -82,20 +82,20 @@ SvXMLImportContextRef 
SwXMLItemSetContext::createFastChildContext( sal_Int32 nEl
     case RES_BACKGROUND:
         {
             const SfxPoolItem *pItem;
-            if( SfxItemState::SET == rItemSet.GetItemState( RES_BACKGROUND,
+            if( SfxItemState::SET == m_rItemSet.GetItemState( RES_BACKGROUND,
                                                        false, &pItem ) )
             {
                 xContext = new SwXMLBrushItemImportContext(
                                 GetImport(), nElement, xAttrList,
-                                rUnitConv, *static_cast<const SvxBrushItem 
*>(pItem) );
+                                m_rUnitConv, *static_cast<const SvxBrushItem 
*>(pItem) );
             }
             else
             {
                 xContext = new SwXMLBrushItemImportContext(
                                 GetImport(), nElement, xAttrList,
-                                rUnitConv, RES_BACKGROUND );
+                                m_rUnitConv, RES_BACKGROUND );
             }
-            xBackground = xContext;
+            m_xBackground = xContext;
         }
         break;
     }
diff --git a/sw/source/filter/xml/xmlitem.hxx b/sw/source/filter/xml/xmlitem.hxx
index 2993ab2d0b7e..4cff6aaafb79 100644
--- a/sw/source/filter/xml/xmlitem.hxx
+++ b/sw/source/filter/xml/xmlitem.hxx
@@ -31,10 +31,10 @@ struct SvXMLItemMapEntry;
 
 class SwXMLItemSetContext final : public SvXMLImportContext
 {
-    SfxItemSet                  &rItemSet;
-    const SvXMLImportItemMapper &rIMapper;
-    const SvXMLUnitConverter    &rUnitConv;
-    SvXMLImportContextRef xBackground;
+    SfxItemSet                  &m_rItemSet;
+    const SvXMLImportItemMapper &m_rIMapper;
+    const SvXMLUnitConverter    &m_rUnitConv;
+    SvXMLImportContextRef m_xBackground;
 
 public:
 
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to