diff options
author | Eric Liu <ioeric@google.com> | 2017-02-24 12:56:51 +0000 |
---|---|---|
committer | Eric Liu <ioeric@google.com> | 2017-02-24 12:56:51 +0000 |
commit | ec09f6a83b08f46fef85ffcbacb267f569308687 (patch) | |
tree | 63730a92995bf18fa436cf3780ab34611b459768 /clang-tools-extra/change-namespace | |
parent | 8a8772144e6d9e82cf610d00cf971c7ffbd7f44f (diff) |
[change-namespace] fix asan failure in r296110.
Diffstat (limited to 'clang-tools-extra/change-namespace')
-rw-r--r-- | clang-tools-extra/change-namespace/tool/ClangChangeNamespace.cpp | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/clang-tools-extra/change-namespace/tool/ClangChangeNamespace.cpp b/clang-tools-extra/change-namespace/tool/ClangChangeNamespace.cpp index 02a97839f15..6a17f192776 100644 --- a/clang-tools-extra/change-namespace/tool/ClangChangeNamespace.cpp +++ b/clang-tools-extra/change-namespace/tool/ClangChangeNamespace.cpp @@ -80,15 +80,16 @@ cl::opt<std::string> WhiteListFile( cl::init(""), cl::cat(ChangeNamespaceCategory)); llvm::ErrorOr<std::vector<std::string>> GetWhiteListedSymbolPatterns() { + if (WhiteListFile.empty()) + return std::vector<std::string>(); + llvm::SmallVector<StringRef, 8> Lines; - if (!WhiteListFile.empty()) { - llvm::ErrorOr<std::unique_ptr<llvm::MemoryBuffer>> File = - llvm::MemoryBuffer::getFile(WhiteListFile); - if (!File) - return File.getError(); - llvm::StringRef Content = File.get()->getBuffer(); - Content.split(Lines, '\n', /*MaxSplit=*/-1, /*KeepEmpty=*/false); - } + llvm::ErrorOr<std::unique_ptr<llvm::MemoryBuffer>> File = + llvm::MemoryBuffer::getFile(WhiteListFile); + if (!File) + return File.getError(); + llvm::StringRef Content = File.get()->getBuffer(); + Content.split(Lines, '\n', /*MaxSplit=*/-1, /*KeepEmpty=*/false); return std::vector<std::string>(Lines.begin(), Lines.end()); } |