[llvm] [llvm] fix mustache template whitespace (PR #153724)

Paul Kirth via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 26 10:48:04 PDT 2025


================
@@ -681,10 +685,59 @@ void ASTNode::renderChild(const json::Value &Contexts, llvm::raw_ostream &OS) {
     Child->render(Contexts, OS);
 }
 
+void ASTNode::indentTextNode(std::string &Body, size_t Indentation,
+                             bool FinalNode) {
+  std::string spaces(Indentation, ' ');
----------------
ilovepi wrote:

```suggestion
  std::string Spaces(Indentation, ' ');
```
The style guide prefers that we capitalize variables, and that they should be nouns. https://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly

https://github.com/llvm/llvm-project/pull/153724


More information about the llvm-commits mailing list