From bf3bd35ef8c0538328444feda4ae0745c40af7ec Mon Sep 17 00:00:00 2001 From: Sundong Ahn Date: Wed, 23 Mar 2022 12:44:55 +0000 Subject: Support multiple whitespace characters When parsing the list input, only whitespace was supported " ". So change the split character logic to support "\t", "\n", etc. Bug: 225984277 Test: atest xsdc-cpp-tests Change-Id: Ie9815e39a10fee198b773d66b6beb64fc399f13e --- src/com/android/xsdc/cpp/CppCodeGenerator.java | 1 + src/com/android/xsdc/cpp/CppSimpleType.java | 13 +++++-------- tests/resources/attr_group_simple.xml | 3 ++- 3 files changed, 8 insertions(+), 9 deletions(-) diff --git a/src/com/android/xsdc/cpp/CppCodeGenerator.java b/src/com/android/xsdc/cpp/CppCodeGenerator.java index 272a296..e470216 100644 --- a/src/com/android/xsdc/cpp/CppCodeGenerator.java +++ b/src/com/android/xsdc/cpp/CppCodeGenerator.java @@ -154,6 +154,7 @@ public class CppCodeGenerator { parserHeaderFile.printf("#include \n"); parserHeaderFile.printf("#include \n"); parserHeaderFile.printf("#include \n"); + parserHeaderFile.printf("#include \n"); if (writer) { parserHeaderFile.printf("#include \n"); } diff --git a/src/com/android/xsdc/cpp/CppSimpleType.java b/src/com/android/xsdc/cpp/CppSimpleType.java index 282855f..d063ebe 100644 --- a/src/com/android/xsdc/cpp/CppSimpleType.java +++ b/src/com/android/xsdc/cpp/CppSimpleType.java @@ -63,15 +63,12 @@ class CppSimpleType implements CppType { if (list) { expression.append( String.format("%s value;\n", getName())); - expression.append(String.format("{\nint base = 0;\n" - + "int found;\n" - + "while(true) {\n" - + "found = raw.find_first_of(\" \", base);\n" - + "value.push_back(%s);\n" - + "if (found == raw.npos) break;\n" - + "base = found + 1;\n" + expression.append(String.format("{\n" + + "std::istringstream stream(raw);\n" + + "for(std::string str; stream >> str; ) {\n" + + " value.push_back(%s);\n" + "}\n", - String.format(rawParsingExpression, "raw.substr(base, found - base)"))); + String.format(rawParsingExpression, "str"))); expression.append("}\n"); } else { expression.append( diff --git a/tests/resources/attr_group_simple.xml b/tests/resources/attr_group_simple.xml index 0a14222..fa400f9 100644 --- a/tests/resources/attr_group_simple.xml +++ b/tests/resources/attr_group_simple.xml @@ -1,4 +1,5 @@ - + Jun -- cgit v1.2.3