diff options
| author | Noel Grandin <noelgrandin@gmail.com> | 2024-06-30 21:27:23 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2024-07-01 08:56:44 +0200 |
| commit | 4637bad647b15892cb954731ad8fd8ff6533a4eb (patch) | |
| tree | ad8f821fd3df1ca770b03639dac3b155e1e921b6 /include | |
| parent | 26ae64ae9f1feafac0219de75d6c23b54ceefaae (diff) | |
call fillDecimal() in SvxTabStop constructor
otherwise we get the weird behaviour where two SvxTabStop instances
which are constructed the same, return false from operator==
because one instance has had getDecimal() called on it, and the
other not.
Change-Id: I1f708c1b1e03efcd9fc554eaf2777c4c3e8a80b8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169774
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
Diffstat (limited to 'include')
| -rw-r--r-- | include/editeng/tstpitem.hxx | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/include/editeng/tstpitem.hxx b/include/editeng/tstpitem.hxx index 037e76636641..f0f99ce9cc86 100644 --- a/include/editeng/tstpitem.hxx +++ b/include/editeng/tstpitem.hxx @@ -37,11 +37,10 @@ class EDITENG_DLLPUBLIC SvxTabStop private: sal_Int32 nTabPos; SvxTabAdjust eAdjustment; - mutable sal_Unicode - m_cDecimal; + sal_Unicode m_cDecimal; sal_Unicode cFill; - void fillDecimal() const; + void fillDecimal(); public: SvxTabStop(); @@ -56,8 +55,8 @@ public: SvxTabAdjust& GetAdjustment() { return eAdjustment; } SvxTabAdjust GetAdjustment() const { return eAdjustment; } - sal_Unicode& GetDecimal() { fillDecimal(); return m_cDecimal; } - sal_Unicode GetDecimal() const { fillDecimal(); return m_cDecimal; } + sal_Unicode& GetDecimal() { return m_cDecimal; } + sal_Unicode GetDecimal() const { return m_cDecimal; } sal_Unicode& GetFill() { return cFill; } sal_Unicode GetFill() const { return cFill; } |