diff options
author | Pavel Labath <labath@google.com> | 2018-04-10 10:07:22 +0000 |
---|---|---|
committer | Pavel Labath <labath@google.com> | 2018-04-10 10:07:22 +0000 |
commit | aec4593ad93d0dc8b61d1091ee074960cbc89834 (patch) | |
tree | 9b65c1ca4db1de2d58f12e2c6be5a1b87e870176 /lldb/source | |
parent | f974e255fe9f3192603cf53c6d2108e7d81dbfe4 (diff) | |
download | bcm5719-llvm-aec4593ad93d0dc8b61d1091ee074960cbc89834.tar.gz bcm5719-llvm-aec4593ad93d0dc8b61d1091ee074960cbc89834.zip |
Args: replace isprint8 usage with isprint
It looks like we introduced isprint8 way back in r169417 to be used on
getopt's short_options, which we sometimes set to values which are out
of range for normal chars to indicate options with no short form.
However, this is not how the function is used in the Args class, where
we explicitly process a string character by character.
This removes the last external dependency from the Args class.
llvm-svn: 329682
Diffstat (limited to 'lldb/source')
-rw-r--r-- | lldb/source/Interpreter/Args.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lldb/source/Interpreter/Args.cpp b/lldb/source/Interpreter/Args.cpp index 99d62970d1c..1fa7c9e41ab 100644 --- a/lldb/source/Interpreter/Args.cpp +++ b/lldb/source/Interpreter/Args.cpp @@ -13,7 +13,6 @@ // Other libraries and framework includes // Project includes #include "lldb/Interpreter/Args.h" -#include "lldb/Interpreter/Options.h" #include "lldb/Utility/ConstString.h" #include "lldb/Utility/FileSpec.h" #include "lldb/Utility/Stream.h" @@ -620,7 +619,7 @@ void Args::ExpandEscapedCharacters(const char *src, std::string &dst) { dst.clear(); if (src) { for (const char *p = src; *p != '\0'; ++p) { - if (isprint8(*p)) + if (isprint(*p)) dst.append(1, *p); else { switch (*p) { |