<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 10, 2017 at 11:13 AM, Gabor Horvath via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Author: xazax<br>
Date: Thu Aug 10 02:13:26 2017<br>
New Revision: 310559<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=310559&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=310559&view=rev</a><br>
Log:<br>
[clang-tidy] Minor documentation improvement<br>
<br>
Patch by: Lilla Barancsuk<br>
<br>
Modified:<br>
    clang-tools-extra/trunk/docs/<wbr>clang-tidy/checks/readability-<wbr>static-accessed-through-<wbr>instance.rst<br>
    clang-tools-extra/trunk/test/<wbr>clang-tidy/readability-static-<wbr>accessed-through-instance.cpp<br>
<br>
Modified: clang-tools-extra/trunk/docs/<wbr>clang-tidy/checks/readability-<wbr>static-accessed-through-<wbr>instance.rst<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/readability-static-accessed-through-instance.rst?rev=310559&r1=310558&r2=310559&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/clang-tools-extra/<wbr>trunk/docs/clang-tidy/checks/<wbr>readability-static-accessed-<wbr>through-instance.rst?rev=<wbr>310559&r1=310558&r2=310559&<wbr>view=diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- clang-tools-extra/trunk/docs/<wbr>clang-tidy/checks/readability-<wbr>static-accessed-through-<wbr>instance.rst (original)<br>
+++ clang-tools-extra/trunk/docs/<wbr>clang-tidy/checks/readability-<wbr>static-accessed-through-<wbr>instance.rst Thu Aug 10 02:13:26 2017<br>
@@ -25,6 +25,7 @@ is changed to:<br>
<br>
 .. code-block:: c++<br>
<br>
+  C *c1 = new C();<br>
   C::foo();<br>
   C::x;<br>
<br>
<br>
Modified: clang-tools-extra/trunk/test/<wbr>clang-tidy/readability-static-<wbr>accessed-through-instance.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/readability-static-accessed-through-instance.cpp?rev=310559&r1=310558&r2=310559&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/clang-tools-extra/<wbr>trunk/test/clang-tidy/<wbr>readability-static-accessed-<wbr>through-instance.cpp?rev=<wbr>310559&r1=310558&r2=310559&<wbr>view=diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- clang-tools-extra/trunk/test/<wbr>clang-tidy/readability-static-<wbr>accessed-through-instance.cpp (original)<br>
+++ clang-tools-extra/trunk/test/<wbr>clang-tidy/readability-static-<wbr>accessed-through-instance.cpp Thu Aug 10 02:13:26 2017<br>
@@ -116,7 +116,7 @@ using E = D;<br>
<br>
 template <typename T> void f(T t, C c) {<br>
   t.x; // OK, t is a template parameter.<br>
-  c.x; // 1<br>
+  c.x;<br></blockquote><div><br></div><div><span style="color:rgb(0,0,0);font-family:"Segoe UI","Segoe UI Emoji","Segoe UI Symbol",Lato,"Helvetica Neue",Helvetica,Arial,sans-serif;font-size:13px">This comment was there to make the CHECK-FIXES line unique so that it doesn't match wrong line in the test. Any specific reason to remove it?</span></div><div><span style="color:rgb(0,0,0);font-family:"Segoe UI","Segoe UI Emoji","Segoe UI Symbol",Lato,"Helvetica Neue",Helvetica,Arial,sans-serif;font-size:13px"></span> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
   // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: static member<br>
   // CHECK-FIXES: {{^}}  C::x; // 1{{$}}<br>
 }<br>
<br>
<br>
______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div>