| commit | 9e916e5e0e05f293b85cd97c03a7e38d28bb74d2 | [log] [tgz] |
|---|---|---|
| author | Eugene Zemtsov <ezemtsov@google.com> | Tue Feb 06 19:04:12 2018 +0000 |
| committer | Eugene Zemtsov <ezemtsov@google.com> | Tue Feb 06 19:04:12 2018 +0000 |
| tree | 3dd08aaa4477c6f9ff78b0c1e79c0be6b178d7c7 | |
| parent | f951fc399fa5219682b8fd7fe537b43a64f18c1a [diff] [blame] |
More correct handling of error cases C++ name parser Now incorrect type argument that looks like T<A><B> doesn't cause an assert, but just a parsing error. Bug: 36224 Differential Revision: https://reviews.llvm.org/D42939 llvm-svn: 324380
diff --git a/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusNameParser.cpp b/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusNameParser.cpp index aebea6a..a992b72 100644 --- a/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusNameParser.cpp +++ b/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusNameParser.cpp
@@ -242,8 +242,7 @@ } } - assert(template_counter >= 0); - if (template_counter > 0) { + if (template_counter != 0) { return false; } start_position.Remove();