<div dir="ltr">Hi<div><br></div><div>I am a happy user of various LLVM tools, but this is my first attempt to contribute to the project.</div><div><br></div><div>Does anyone know about the progress on BUG-43106.</div><div><a href="https://bugs.llvm.org/show_bug.cgi?id=43106">https://bugs.llvm.org/show_bug.cgi?id=43106</a> <br></div><div>In summery this ask for a clang-tidy check to enforce leading capital letter with snake case for user defined types and concepts.</div><div><br></div><div>I have attached a possible extension to the readability-identifier-naming checker in clang-tidy.</div><div>I have checked the patch in windows building with MSVC by running check_clang_tidy.py on the readability-identifier-naming.cpp file.</div><div><br></div><div>This results in the expected change on the union type definition:</div><div><div>...</div><div>- union __MyUnion_is_wonderful__ {};<br>+ union UMy_union_is_wonderful {};<br></div></div><div>...</div><div>I do not have strong opinions about what to call the option. I followed what was asked for in the ticket, but perhaps we should discuss what to call the option some more. I follow the consistency argument made in the description of
BUG-43106:</div><div>"</div><div><span style="color:rgb(0,0,0);white-space:pre-wrap">Let's call it "Camel_lower_case" for the purposes of this description. This is to match the existing "Camel_Snake_Case" and "lower_case".</span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap">"</span></div><div><font color="#000000"><span style="white-space:pre-wrap">But there are other options like "Upper_snake_case", so I encourage a discussion on this.</span></font></div><div><br></div><div>Could you provide some guidance on how to progress towards getting this checked in to the LLVM project?</div><div><br></div><div>Thank you for your effort and time.</div><div><br></div><div>Best regards,</div><div>Anders Jensen</div></div>