include/svx/e3ditem.hxx      |    4 ++--
 svx/source/items/e3ditem.cxx |   28 ++++++++++++++--------------
 2 files changed, 16 insertions(+), 16 deletions(-)

New commits:
commit b0eceb839a8ddb70412d8cc5737eb3bd2b90b2c8
Author:     Miklos Vajna <vmik...@collabora.com>
AuthorDate: Tue Nov 19 14:26:51 2024 +0100
Commit:     Miklos Vajna <vmik...@collabora.com>
CommitDate: Tue Nov 19 15:32:51 2024 +0100

    svx: prefix members of SvxB3DVectorItem
    
    See tdf#94879 for motivation.
    
    Change-Id: I74c53028d9966ca4059c7a145b3fb0d914d8ae77
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/176770
    Reviewed-by: Miklos Vajna <vmik...@collabora.com>
    Tested-by: Jenkins

diff --git a/include/svx/e3ditem.hxx b/include/svx/e3ditem.hxx
index bad0e95c89f5..7302c93d97c6 100644
--- a/include/svx/e3ditem.hxx
+++ b/include/svx/e3ditem.hxx
@@ -27,7 +27,7 @@
 
 class SAL_WARN_UNUSED UNLESS_MERGELIBS(SVXCORE_DLLPUBLIC) SvxB3DVectorItem 
final : public SfxPoolItem
 {
-    basegfx::B3DVector  aVal;
+    basegfx::B3DVector  m_aVal;
 
 public:
                             SvxB3DVectorItem( TypedWhichId<SvxB3DVectorItem> 
nWhich, const basegfx::B3DVector& rVal );
@@ -40,7 +40,7 @@ public:
     virtual bool            QueryValue( css::uno::Any& rVal, sal_uInt8 
nMemberId = 0 ) const override;
     virtual bool            PutValue( const css::uno::Any& rVal, sal_uInt8 
nMemberId ) override;
 
-    const basegfx::B3DVector&           GetValue() const { return aVal; }
+    const basegfx::B3DVector&           GetValue() const { return m_aVal; }
 
     void dumpAsXml(xmlTextWriterPtr pWriter) const override;
 };
diff --git a/svx/source/items/e3ditem.cxx b/svx/source/items/e3ditem.cxx
index 05a2ff8bcb2c..736b3ab76e7a 100644
--- a/svx/source/items/e3ditem.cxx
+++ b/svx/source/items/e3ditem.cxx
@@ -32,21 +32,21 @@ SvxB3DVectorItem::~SvxB3DVectorItem()
 
 SvxB3DVectorItem::SvxB3DVectorItem( TypedWhichId<SvxB3DVectorItem> _nWhich, 
const basegfx::B3DVector& rVal ) :
     SfxPoolItem( _nWhich, SfxItemType::SvxB3DVectorItemType ),
-    aVal( rVal )
+    m_aVal( rVal )
 {
 }
 
 
 SvxB3DVectorItem::SvxB3DVectorItem( const SvxB3DVectorItem& rItem ) :
     SfxPoolItem( rItem ),
-    aVal( rItem.aVal )
+    m_aVal( rItem.m_aVal )
 {
 }
 
 bool SvxB3DVectorItem::operator==( const SfxPoolItem &rItem ) const
 {
     assert(SfxPoolItem::operator==(rItem));
-    return static_cast<const SvxB3DVectorItem&>(rItem).aVal == aVal;
+    return static_cast<const SvxB3DVectorItem&>(rItem).m_aVal == m_aVal;
 }
 
 SvxB3DVectorItem* SvxB3DVectorItem::Clone( SfxItemPool* /*pPool*/ ) const
@@ -56,14 +56,14 @@ SvxB3DVectorItem* SvxB3DVectorItem::Clone( SfxItemPool* 
/*pPool*/ ) const
 
 bool SvxB3DVectorItem::QueryValue( uno::Any& rVal, sal_uInt8 /*nMemberId*/ ) 
const
 {
-    assert(!std::isnan(aVal.getX()) && !std::isnan(aVal.getY()) && 
!std::isnan(aVal.getZ()));
+    assert(!std::isnan(m_aVal.getX()) && !std::isnan(m_aVal.getY()) && 
!std::isnan(m_aVal.getZ()));
 
     drawing::Direction3D aDirection;
 
     // enter values
-    aDirection.DirectionX = aVal.getX();
-    aDirection.DirectionY = aVal.getY();
-    aDirection.DirectionZ = aVal.getZ();
+    aDirection.DirectionX = m_aVal.getX();
+    aDirection.DirectionY = m_aVal.getY();
+    aDirection.DirectionZ = m_aVal.getZ();
 
     rVal <<= aDirection;
     return true;
@@ -77,11 +77,11 @@ bool SvxB3DVectorItem::PutValue( const uno::Any& rVal, 
sal_uInt8 /*nMemberId*/ )
     if(!(rVal >>= aDirection))
         return false;
 
-    aVal.setX(aDirection.DirectionX);
-    aVal.setY(aDirection.DirectionY);
-    aVal.setZ(aDirection.DirectionZ);
+    m_aVal.setX(aDirection.DirectionX);
+    m_aVal.setY(aDirection.DirectionY);
+    m_aVal.setZ(aDirection.DirectionZ);
 
-    assert(!std::isnan(aVal.getX()) && !std::isnan(aVal.getY()) && 
!std::isnan(aVal.getZ()));
+    assert(!std::isnan(m_aVal.getX()) && !std::isnan(m_aVal.getY()) && 
!std::isnan(m_aVal.getZ()));
 
     return true;
 }
@@ -91,9 +91,9 @@ void SvxB3DVectorItem::dumpAsXml(xmlTextWriterPtr pWriter) 
const
 {
     (void)xmlTextWriterStartElement(pWriter, BAD_CAST("SvxB3DVectorItem"));
     (void)xmlTextWriterWriteAttribute(pWriter, BAD_CAST("whichId"), 
BAD_CAST(OString::number(Which()).getStr()));
-    (void)xmlTextWriterWriteAttribute(pWriter, BAD_CAST("x"), 
BAD_CAST(OString::number(aVal.getX()).getStr()));
-    (void)xmlTextWriterWriteAttribute(pWriter, BAD_CAST("y"), 
BAD_CAST(OString::number(aVal.getY()).getStr()));
-    (void)xmlTextWriterWriteAttribute(pWriter, BAD_CAST("z"), 
BAD_CAST(OString::number(aVal.getZ()).getStr()));
+    (void)xmlTextWriterWriteAttribute(pWriter, BAD_CAST("x"), 
BAD_CAST(OString::number(m_aVal.getX()).getStr()));
+    (void)xmlTextWriterWriteAttribute(pWriter, BAD_CAST("y"), 
BAD_CAST(OString::number(m_aVal.getY()).getStr()));
+    (void)xmlTextWriterWriteAttribute(pWriter, BAD_CAST("z"), 
BAD_CAST(OString::number(m_aVal.getZ()).getStr()));
     (void)xmlTextWriterEndElement(pWriter);
 }
 

Reply via email to