hokein added a comment.

In https://reviews.llvm.org/D43847#1021967, @aaron.ballman wrote:

> I need a bit more context because I'm unfamiliar with `absl`. What is this 
> module's intended use?


As `absl` has been open-sourced (https://github.com/abseil/abseil-cpp), I think 
there will be more absl-related checks contributed from google or external 
contributors in the future, so it make sense to create a new module.

@niko, could you please refine the code style of this patch to follow the LLVM 
code standard 
<https://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly>
 (especially the variable name, should be camel case "VariableName")?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D43847



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to