Fix the unintelligible IsValidClassName API by breaking it into three.
There are only three of the four possibilities you actually want, so we may as
well just call them by name.
Change-Id: I481d91d98d6d47430f1972891fa81a61c57ad331
diff --git a/src/utils.cc b/src/utils.cc
index 4c65036..c37343c 100644
--- a/src/utils.cc
+++ b/src/utils.cc
@@ -274,8 +274,6 @@
return long_name;
}
-namespace {
-
// Helper for IsValidMemberNameUtf8(), a bit vector indicating valid low ascii.
uint32_t DEX_MEMBER_VALID_LOW_ASCII[4] = {
0x00000000, // 00..1f low control characters; nothing valid
@@ -355,11 +353,8 @@
return IsValidMemberNameUtf8Slow(pUtf8Ptr);
}
-} // namespace
-
-bool IsValidClassName(const char* s, bool isClassName, bool dot_or_slash) {
- char separator = (dot_or_slash ? '.' : '/');
-
+enum ClassNameType { kName, kDescriptor };
+bool IsValidClassName(const char* s, ClassNameType type, char separator) {
int arrayCount = 0;
while (*s == '[') {
arrayCount++;
@@ -378,10 +373,10 @@
* format looks the same as a type descriptor in that case, so
* treat it as such.
*/
- isClassName = false;
+ type = kDescriptor;
}
- if (!isClassName) {
+ if (type == kDescriptor) {
/*
* We are looking for a descriptor. Either validate it as a
* single-character primitive type, or continue on to check the
@@ -427,7 +422,7 @@
* empty component (including the degenerate case of
* the empty string "").
*/
- return isClassName && !sepOrFirst;
+ return (type == kName) && !sepOrFirst;
case ';':
/*
* Invalid character for a class name, but the
@@ -436,7 +431,7 @@
* and that it doesn't end with an empty component
* (including the degenerate case of "L;").
*/
- return !isClassName && !sepOrFirst && (s[1] == '\0');
+ return (type == kDescriptor) && !sepOrFirst && (s[1] == '\0');
case '/':
case '.':
if (c != separator) {
@@ -460,6 +455,18 @@
}
}
+bool IsValidBinaryClassName(const char* s) {
+ return IsValidClassName(s, kName, '.');
+}
+
+bool IsValidJniClassName(const char* s) {
+ return IsValidClassName(s, kName, '/');
+}
+
+bool IsValidDescriptor(const char* s) {
+ return IsValidClassName(s, kDescriptor, '/');
+}
+
void Split(const std::string& s, char delim, std::vector<std::string>& result) {
const char* p = s.data();
const char* end = p + s.size();