]> git.saurik.com Git - android/aapt.git/commitdiff
Use "ldrtl" and "ldltr" config qualifiers for RTL / LTR resources
authorFabrice Di Meglio <fdimeglio@google.com>
Wed, 5 Sep 2012 20:12:02 +0000 (13:12 -0700)
committerFabrice Di Meglio <fdimeglio@google.com>
Thu, 6 Sep 2012 18:13:24 +0000 (11:13 -0700)
- we cannot use "rtl" / "ltr" qualifiers as they can conflict with ISO-639 Alpha-3
codespace which uses 3 letters for identifying a language code (and could use either
"rtl" or "ltr" strings for defining a language in the future).

- we are using instead "ldrtl" for RTL and "ldltr" for LTR resources. Those qualifiers
are defined by more than 3 chars and outside of what is defined into ISO-639. They
are also more understandable as "ld" prefix is for "layoutdirection"

Change-Id: Id43e948103707e09bef63ebd54ac1779dde58e72

AaptAssets.cpp

index 438a670452e4bc8bc18ceddc6f5677c13832bc88..5b71adc81cbe09d89abb9223bd4906abf2e79476 100644 (file)
@@ -995,12 +995,12 @@ bool AaptGroupEntry::getLayoutDirectionName(const char* name, ResTable_config* o
                 (out->screenLayout&~ResTable_config::MASK_LAYOUTDIR)
                 | ResTable_config::LAYOUTDIR_ANY;
         return true;
-    } else if (strcmp(name, "ltr") == 0) {
+    } else if (strcmp(name, "ldltr") == 0) {
         if (out) out->screenLayout =
                 (out->screenLayout&~ResTable_config::MASK_LAYOUTDIR)
                 | ResTable_config::LAYOUTDIR_LTR;
         return true;
-    } else if (strcmp(name, "rtl") == 0) {
+    } else if (strcmp(name, "ldrtl") == 0) {
         if (out) out->screenLayout =
                 (out->screenLayout&~ResTable_config::MASK_LAYOUTDIR)
                 | ResTable_config::LAYOUTDIR_RTL;