include/xmloff/ProgressBarHelper.hxx | 32 ++++++------- include/xmloff/XMLBase64ImportContext.hxx | 2 include/xmloff/i18nmap.hxx | 14 ++--- include/xmloff/xmlerror.hxx | 2 xmloff/source/core/ProgressBarHelper.cxx | 64 +++++++++++++------------- xmloff/source/core/XMLBase64ImportContext.cxx | 6 +- xmloff/source/core/xmlerror.cxx | 6 +- 7 files changed, 63 insertions(+), 63 deletions(-)
New commits: commit d9a276bc6aaf66241e3a0b5d53b1e8d2dd238d7e Author: Miklos Vajna <vmik...@collabora.com> AuthorDate: Mon May 8 06:01:13 2023 +0200 Commit: Miklos Vajna <vmik...@collabora.com> CommitDate: Mon May 8 08:53:18 2023 +0200 sw: prefix members of ProgressBarHelper, SvI18NMapEntry_Key, ... ... XMLBase64ImportContext and XMLErrors See tdf#94879 for motivation. Change-Id: Ic8d1b3515727981c50d8f3a64fe6bf030f0521fa Reviewed-on: https://gerrit.libreoffice.org/c/core/+/151491 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmik...@collabora.com> diff --git a/include/xmloff/ProgressBarHelper.hxx b/include/xmloff/ProgressBarHelper.hxx index 770cf7986604..1fc42c96db60 100644 --- a/include/xmloff/ProgressBarHelper.hxx +++ b/include/xmloff/ProgressBarHelper.hxx @@ -31,37 +31,37 @@ inline constexpr OUStringLiteral XML_PROGRESSREPEAT = u"ProgressRepeat"; class XMLOFF_DLLPUBLIC ProgressBarHelper { - css::uno::Reference < css::task::XStatusIndicator > xStatusIndicator; - sal_Int32 nRange; - sal_Int32 nReference; - sal_Int32 nValue; - double fOldPercent; - bool bStrict; + css::uno::Reference < css::task::XStatusIndicator > m_xStatusIndicator; + sal_Int32 m_nRange; + sal_Int32 m_nReference; + sal_Int32 m_nValue; + double m_fOldPercent; + bool m_bStrict; // #96469#; if the value goes over the Range the progressbar starts again - bool bRepeat; + bool m_bRepeat; #ifdef DBG_UTIL - bool bFailure; + bool m_bFailure; #endif public: ProgressBarHelper(css::uno::Reference < css::task::XStatusIndicator> xStatusIndicator, const bool bStrict); ~ProgressBarHelper(); - void SetRange(sal_Int32 nVal) { nRange = nVal; } - void SetReference(sal_Int32 nVal) { nReference = nVal; } + void SetRange(sal_Int32 nVal) { m_nRange = nVal; } + void SetReference(sal_Int32 nVal) { m_nReference = nVal; } void SetValue(sal_Int32 nValue); - void SetRepeat(bool bValue) { bRepeat = bValue; } - void Increment(sal_Int32 nInc = 1) { SetValue( nValue+nInc ); } - void End() { if (xStatusIndicator.is()) xStatusIndicator->end(); } + void SetRepeat(bool bValue) { m_bRepeat = bValue; } + void Increment(sal_Int32 nInc = 1) { SetValue( m_nValue+nInc ); } + void End() { if (m_xStatusIndicator.is()) m_xStatusIndicator->end(); } // set the new reference and returns the new value which gives the // Progress Bar the same position as before void ChangeReference(sal_Int32 nNewReference); - sal_Int32 GetReference() const { return nReference; } - sal_Int32 GetValue() const { return nValue; } - bool GetRepeat() const { return bRepeat; } + sal_Int32 GetReference() const { return m_nReference; } + sal_Int32 GetValue() const { return m_nValue; } + bool GetRepeat() const { return m_bRepeat; } }; #endif diff --git a/include/xmloff/XMLBase64ImportContext.hxx b/include/xmloff/XMLBase64ImportContext.hxx index e2b62abea1a4..053ec8cec56c 100644 --- a/include/xmloff/XMLBase64ImportContext.hxx +++ b/include/xmloff/XMLBase64ImportContext.hxx @@ -30,7 +30,7 @@ namespace com::sun::star::io { class XOutputStream; } class XMLOFF_DLLPUBLIC XMLBase64ImportContext final : public SvXMLImportContext { - css::uno::Reference< css::io::XOutputStream > xOut; + css::uno::Reference< css::io::XOutputStream > m_xOut; OUStringBuffer maCharBuffer; public: diff --git a/include/xmloff/i18nmap.hxx b/include/xmloff/i18nmap.hxx index 91aed83eddf5..31dd07692a87 100644 --- a/include/xmloff/i18nmap.hxx +++ b/include/xmloff/i18nmap.hxx @@ -28,20 +28,20 @@ class SvI18NMapEntry_Key { - sal_uInt16 nKind; - OUString aName; + sal_uInt16 m_nKind; + OUString m_aName; public: SvI18NMapEntry_Key( sal_uInt16 nKnd, OUString sName ) : - nKind( nKnd ), - aName(std::move( sName )) + m_nKind( nKnd ), + m_aName(std::move( sName )) { } bool operator<( const SvI18NMapEntry_Key& r ) const { - return nKind < r.nKind || - ( nKind == r.nKind && - aName < r.aName); + return m_nKind < r.m_nKind || + ( m_nKind == r.m_nKind && + m_aName < r.m_aName); } }; diff --git a/include/xmloff/xmlerror.hxx b/include/xmloff/xmlerror.hxx index 2b4c8717d50c..027639e6ab53 100644 --- a/include/xmloff/xmlerror.hxx +++ b/include/xmloff/xmlerror.hxx @@ -104,7 +104,7 @@ class XMLErrors /// definition of type for error list typedef ::std::vector<ErrorRecord> ErrorList; - ErrorList aErrors; /// list of error records + ErrorList m_aErrors; /// list of error records public: diff --git a/xmloff/source/core/ProgressBarHelper.cxx b/xmloff/source/core/ProgressBarHelper.cxx index 61248c06e8df..3ff70fb0434d 100644 --- a/xmloff/source/core/ProgressBarHelper.cxx +++ b/xmloff/source/core/ProgressBarHelper.cxx @@ -29,15 +29,15 @@ const float fProgressStep = 0.5; ProgressBarHelper::ProgressBarHelper(css::uno::Reference < css::task::XStatusIndicator> xTempStatusIndicator, const bool bTempStrict) -: xStatusIndicator(std::move(xTempStatusIndicator)) -, nRange(nDefaultProgressBarRange) -, nReference(100) -, nValue(0) -, fOldPercent(0.0) -, bStrict(bTempStrict) -, bRepeat(true) +: m_xStatusIndicator(std::move(xTempStatusIndicator)) +, m_nRange(nDefaultProgressBarRange) +, m_nReference(100) +, m_nValue(0) +, m_fOldPercent(0.0) +, m_bStrict(bTempStrict) +, m_bRepeat(true) #ifdef DBG_UTIL -, bFailure(false) +, m_bFailure(false) #endif { } @@ -48,59 +48,59 @@ ProgressBarHelper::~ProgressBarHelper() void ProgressBarHelper::ChangeReference(sal_Int32 nNewReference) { - if((nNewReference <= 0) || (nNewReference == nReference)) + if((nNewReference <= 0) || (nNewReference == m_nReference)) return; - if (nReference) + if (m_nReference) { - double fPercent(static_cast<double>(nNewReference) / nReference); - double fValue(nValue * fPercent); - nValue = static_cast<sal_Int32>(fValue); - nReference = nNewReference; + double fPercent(static_cast<double>(nNewReference) / m_nReference); + double fValue(m_nValue * fPercent); + m_nValue = static_cast<sal_Int32>(fValue); + m_nReference = nNewReference; } else { - nReference = nNewReference; - nValue = 0; + m_nReference = nNewReference; + m_nValue = 0; } } void ProgressBarHelper::SetValue(sal_Int32 nTempValue) { - if (!xStatusIndicator.is() || (nReference <= 0)) + if (!m_xStatusIndicator.is() || (m_nReference <= 0)) return; - if ((nTempValue >= nValue) && (!bStrict || (nTempValue <= nReference))) + if ((nTempValue >= m_nValue) && (!m_bStrict || (nTempValue <= m_nReference))) { // #91317# no progress bar with values > 100% - if (nTempValue > nReference) + if (nTempValue > m_nReference) { - if (!bRepeat) - nValue = nReference; + if (!m_bRepeat) + m_nValue = m_nReference; else { - xStatusIndicator->reset(); - nValue = 0; + m_xStatusIndicator->reset(); + m_nValue = 0; } } else - nValue = nTempValue; + m_nValue = nTempValue; - double fValue(nValue); - double fNewValue ((fValue * nRange) / nReference); + double fValue(m_nValue); + double fNewValue ((fValue * m_nRange) / m_nReference); - double fPercent((fNewValue * 100) / nRange); - if (fPercent >= (fOldPercent + fProgressStep) || fPercent < fOldPercent) + double fPercent((fNewValue * 100) / m_nRange); + if (fPercent >= (m_fOldPercent + fProgressStep) || fPercent < m_fOldPercent) { - xStatusIndicator->setValue(static_cast<sal_Int32>(fNewValue)); - fOldPercent = fPercent; + m_xStatusIndicator->setValue(static_cast<sal_Int32>(fNewValue)); + m_fOldPercent = fPercent; } } #ifdef DBG_UTIL - else if (!bFailure) + else if (!m_bFailure) { OSL_FAIL("tried to set a wrong value on the progressbar"); - bFailure = true; + m_bFailure = true; } #endif } diff --git a/xmloff/source/core/XMLBase64ImportContext.cxx b/xmloff/source/core/XMLBase64ImportContext.cxx index 81d2498bbabd..1476733642a3 100644 --- a/xmloff/source/core/XMLBase64ImportContext.cxx +++ b/xmloff/source/core/XMLBase64ImportContext.cxx @@ -34,7 +34,7 @@ XMLBase64ImportContext::XMLBase64ImportContext( SvXMLImport& rImport, const Reference< XOutputStream >& rOut ) : SvXMLImportContext( rImport ), - xOut( rOut ) + m_xOut( rOut ) { } @@ -49,10 +49,10 @@ void XMLBase64ImportContext::endFastElement(sal_Int32 ) { Sequence< sal_Int8 > aBuffer( (sChars.size() / 4) * 3 ); ::comphelper::Base64::decodeSomeChars( aBuffer, sChars ); - xOut->writeBytes( aBuffer ); + m_xOut->writeBytes( aBuffer ); } maCharBuffer.setLength(0); - xOut->closeOutput(); + m_xOut->closeOutput(); } void XMLBase64ImportContext::characters( const OUString& rChars ) diff --git a/xmloff/source/core/xmlerror.cxx b/xmloff/source/core/xmlerror.cxx index 9cda2297e0af..df99c4a281fb 100644 --- a/xmloff/source/core/xmlerror.cxx +++ b/xmloff/source/core/xmlerror.cxx @@ -96,7 +96,7 @@ void XMLErrors::AddRecord( const OUString& rPublicId, const OUString& rSystemId ) { - aErrors.emplace_back( nId, rParams, rExceptionMessage, + m_aErrors.emplace_back( nId, rParams, rExceptionMessage, nRow, nColumn, rPublicId, rSystemId ); #ifdef DBG_UTIL @@ -186,12 +186,12 @@ void XMLErrors::AddRecord( void XMLErrors::ThrowErrorAsSAXException(sal_Int32 nIdMask) { // search first error/warning that matches the nIdMask - for( const auto& rError : aErrors ) + for( const auto& rError : m_aErrors ) { if ( (rError.nId & nIdMask) != 0 ) { // we throw the error - ErrorRecord& rErr = aErrors[0]; + ErrorRecord& rErr = m_aErrors[0]; throw SAXParseException( rErr.sExceptionMessage, nullptr, Any(rErr.aParams), rErr.sPublicId, rErr.sSystemId, rErr.nRow, rErr.nColumn );