Skip to content

Commit 112c5d6

Browse files
[llvm-remarkutil] filter: Fix curses.h namespace pollution
Remove the filter namespace, because `filter` is used by `curses.h`, causing some external build failures (llvm#159784 (comment)). We don't really need the namespace here anyways, because everything is static. This was just following what some of the other commands in llvm-remarkutil are doing. Pull Request: llvm#160802
1 parent da51a10 commit 112c5d6

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

llvm/tools/llvm-remarkutil/RemarkFilter.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,9 @@ using namespace llvm;
2020
using namespace remarks;
2121
using namespace llvm::remarkutil;
2222

23-
namespace filter {
23+
// Note: Avoid using the identifier "filter" in this file, as it is prone to
24+
// namespace collision with headers that might get included e.g.
25+
// curses.h.
2426

2527
static cl::SubCommand FilterSub("filter",
2628
"Filter remarks based on specified criteria.");
@@ -76,5 +78,3 @@ static Error tryFilter() {
7678
}
7779

7880
static CommandRegistration FilterReg(&FilterSub, tryFilter);
79-
80-
} // namespace filter

0 commit comments

Comments
 (0)