[llvm] [llvm-strings] Style updates primarily for --radix (PR #222454)
James Henderson via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 10 00:28:52 PDT 2026
================
@@ -71,8 +70,9 @@ static cl::list<std::string> InputFileNames(cl::Positional,
static int MinLength = 4;
static bool PrintFileName;
-enum radix { none, octal, hexadecimal, decimal };
-static radix Radix;
+enum class Radix { None, Octal, Hexadecimal, Decimal };
+static Radix Radix;
+} // namespace
----------------
jh7370 wrote:
static variables don't need to be in an anonymous namespace. Why is clang-tidy complaining? Are you referring to the enum itself in the PR description?
https://github.com/llvm/llvm-project/pull/222454
More information about the llvm-commits
mailing list