ICU-21960 Add missing "const" to kAttributeKey
authoragrieve <agrieve@chromium.org>
Mon, 2 May 2022 17:13:41 +0000 (13:13 -0400)
committerMarkus Scherer <markus.icu@gmail.com>
Wed, 8 Jun 2022 16:56:24 +0000 (16:56 +0000)
This variable was flagged by a chromium check that looks for variables named like constants that end up in the `.data` ELF section (rather than in `.rodata`).

icu4c/source/common/localebuilder.cpp

index a5f201e8475b5ce5f7f98bc8a63f1f8faf776007..cdeb555758190b43893930e18c25e7063db4db13 100644 (file)
@@ -15,7 +15,7 @@ U_NAMESPACE_BEGIN
 #define UPRV_ISDIGIT(c) (((c) >= '0') && ((c) <= '9'))
 #define UPRV_ISALPHANUM(c) (uprv_isASCIILetter(c) || UPRV_ISDIGIT(c) )
 
-const char* kAttributeKey = "attribute";
+constexpr const char* kAttributeKey = "attribute";
 
 static bool _isExtensionSubtags(char key, const char* s, int32_t len) {
     switch (uprv_tolower(key)) {