Kaydet (Commit) 9fdde645 authored tarafından Eike Rathke's avatar Eike Rathke

use sensible variable naming, tdf#85432 follow-up

Change-Id: Ibb468ff3b4c3210c4c5df60fbd42d74884bbfdb3
üst f318e305
...@@ -76,7 +76,7 @@ AboutDialog::AboutDialog(vcl::Window* pParent) ...@@ -76,7 +76,7 @@ AboutDialog::AboutDialog(vcl::Window* pParent)
m_aVersionTextStr = m_pVersion->GetText(); m_aVersionTextStr = m_pVersion->GetText();
m_aBasedTextStr = get<FixedText>("libreoffice")->GetText(); m_aBasedTextStr = get<FixedText>("libreoffice")->GetText();
m_aBasedDerivedTextStr = get<FixedText>("derived")->GetText(); m_aBasedDerivedTextStr = get<FixedText>("derived")->GetText();
m_pLocaleStr = get<FixedText>("locale")->GetText();; m_aLocaleStr = get<FixedText>("locale")->GetText();;
m_pVersion->SetText(GetVersionString()); m_pVersion->SetText(GetVersionString());
...@@ -234,7 +234,7 @@ OUString AboutDialog::GetBuildId() ...@@ -234,7 +234,7 @@ OUString AboutDialog::GetBuildId()
OUString AboutDialog::GetLocaleString() OUString AboutDialog::GetLocaleString()
{ {
OUString pLocaleStr; OUString aLocaleStr;
rtl_Locale * pLocale; rtl_Locale * pLocale;
osl_getProcessLocale( &pLocale ); osl_getProcessLocale( &pLocale );
...@@ -242,14 +242,14 @@ OUString AboutDialog::GetLocaleString() ...@@ -242,14 +242,14 @@ OUString AboutDialog::GetLocaleString()
if ( pLocale && pLocale->Language ) if ( pLocale && pLocale->Language )
{ {
if (pLocale->Country && rtl_uString_getLength( pLocale->Country) > 0) if (pLocale->Country && rtl_uString_getLength( pLocale->Country) > 0)
pLocaleStr = OUString(pLocale->Language) + "_" + OUString(pLocale->Country); aLocaleStr = OUString(pLocale->Language) + "_" + OUString(pLocale->Country);
else else
pLocaleStr = OUString(pLocale->Language); aLocaleStr = OUString(pLocale->Language);
if (pLocale->Variant && rtl_uString_getLength( pLocale->Variant) > 0) if (pLocale->Variant && rtl_uString_getLength( pLocale->Variant) > 0)
pLocaleStr += OUString(pLocale->Variant); aLocaleStr += OUString(pLocale->Variant);
} }
return pLocaleStr; return aLocaleStr;
} }
OUString AboutDialog::GetVersionString() OUString AboutDialog::GetVersionString()
...@@ -264,7 +264,7 @@ OUString AboutDialog::GetVersionString() ...@@ -264,7 +264,7 @@ OUString AboutDialog::GetVersionString()
OUString sBuildId = GetBuildId(); OUString sBuildId = GetBuildId();
OUString pLocaleStr = GetLocaleString(); OUString aLocaleStr = GetLocaleString();
if (!sBuildId.trim().isEmpty()) if (!sBuildId.trim().isEmpty())
{ {
...@@ -282,15 +282,15 @@ OUString AboutDialog::GetVersionString() ...@@ -282,15 +282,15 @@ OUString AboutDialog::GetVersionString()
sVersion += "\n" EXTRA_BUILDID; sVersion += "\n" EXTRA_BUILDID;
} }
if (!pLocaleStr.trim().isEmpty()) if (!aLocaleStr.trim().isEmpty())
{ {
sVersion += "\n"; sVersion += "\n";
if (m_pLocaleStr.indexOf("$LOCALE") == -1) if (m_aLocaleStr.indexOf("$LOCALE") == -1)
{ {
SAL_WARN( "cui.dialogs", "translated locale string in translations doesn't contain $LOCALE placeholder" ); SAL_WARN( "cui.dialogs", "translated locale string in translations doesn't contain $LOCALE placeholder" );
m_pLocaleStr += " $LOCALE"; m_aLocaleStr += " $LOCALE";
} }
sVersion += m_pLocaleStr.replaceAll("$LOCALE", pLocaleStr); sVersion += m_aLocaleStr.replaceAll("$LOCALE", aLocaleStr);
} }
return sVersion; return sVersion;
......
...@@ -49,7 +49,7 @@ private: ...@@ -49,7 +49,7 @@ private:
OUString m_aBasedDerivedTextStr; OUString m_aBasedDerivedTextStr;
OUString m_aCreditsLinkStr; OUString m_aCreditsLinkStr;
OUString m_sBuildStr; OUString m_sBuildStr;
OUString m_pLocaleStr; OUString m_aLocaleStr;
void StyleControls(); void StyleControls();
void SetLogo(); void SetLogo();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment