[PATCH] can we stick a with our naming conventions, just a little bi...

Eike Rathke (via Code Review) gerrit at gerrit.libreoffice.org
Mon Apr 29 06:17:09 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3671

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/71/3671/1

can we stick a with our naming conventions, just a little bit?

Change-Id: I1457c64162c2aeb083e98a28737991ae6d0eee2e
---
M i18npool/source/ordinalsuffix/ordinalsuffix.cxx
1 file changed, 4 insertions(+), 4 deletions(-)



diff --git a/i18npool/source/ordinalsuffix/ordinalsuffix.cxx b/i18npool/source/ordinalsuffix/ordinalsuffix.cxx
index 953c886..f9491b2 100644
--- a/i18npool/source/ordinalsuffix/ordinalsuffix.cxx
+++ b/i18npool/source/ordinalsuffix/ordinalsuffix.cxx
@@ -66,19 +66,19 @@
  * than english ones, ICU 4.2+ has to be used.
  */
 uno::Sequence< OUString > SAL_CALL OrdinalSuffix::getOrdinalSuffix( sal_Int32 nNumber,
-        const lang::Locale &aLocale ) throw( RuntimeException )
+        const lang::Locale &rLocale ) throw( RuntimeException )
 {
     uno::Sequence< OUString > retValue;
 
     // Get the value from ICU
     UErrorCode nCode = U_ZERO_ERROR;
-    const icu::Locale rIcuLocale( LanguageTagIcu::getIcuLocale( LanguageTag( aLocale)));
+    const icu::Locale aIcuLocale( LanguageTagIcu::getIcuLocale( LanguageTag( rLocale)));
 
-    icu::RuleBasedNumberFormat formatter(icu::URBNF_ORDINAL, rIcuLocale, nCode);
+    icu::RuleBasedNumberFormat formatter(icu::URBNF_ORDINAL, aIcuLocale, nCode);
     if (!U_SUCCESS(nCode))
         return retValue;
 
-    boost::scoped_ptr<NumberFormat> xNumberFormat(icu::NumberFormat::createInstance(rIcuLocale, nCode));
+    boost::scoped_ptr<NumberFormat> xNumberFormat(icu::NumberFormat::createInstance(aIcuLocale, nCode));
     if (!U_SUCCESS(nCode))
         return retValue;
 

-- 
To view, visit https://gerrit.libreoffice.org/3671
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1457c64162c2aeb083e98a28737991ae6d0eee2e
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Eike Rathke <erack at redhat.com>



More information about the LibreOffice mailing list