[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