Hi,<div><br></div><div>I saw the <span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse">LLVMConventionsChecker.cpp come up in conversation when others were discussing C++ static analysis.  In any case there were three functions that were checking if you were in the std, clang, or llvm namespace.  I made one function (InNamepace), with an extra argument of a string, then just called that function where needed.  Also, there was a function isClangAttr, I changed it to IsClangAttr to follow the naming conventions of the other functions.</span></div>



<div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse"><br></span></div><div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse">If you wanted to keep the functions In[Std,Clang,LLVM]Namespace for the sake of readability, let me know and I can put it back.  I could also put them back and call the new function, or you can just ignore me :) whichever you prefer as these changes are all trivial. I did not notice them being called outside of this file, so I did not think it would be an issue to remove them.</span></div>



<div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse"><br></span></div><div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse">Like I said all of these changes are trivial, but I thought I would shoot them out because I noticed them.</span></div>



<div><font face="arial, sans-serif"><span style="border-collapse:collapse"><br></span></font></div><div><font face="arial, sans-serif"><span style="border-collapse:collapse">This was all done agains r104634, and it compiled (Release) and tested fine btw...</span></font></div>

<div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse"><br></span></div><div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse">Thanks,</span></div>
<div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse">Jon</span></div>