<div dir="ltr">Thank you for the patch! Note, however, that most clang-tidy reviews are done using Phabricator (see <a href="http://llvm.org/docs/Phabricator.html">llvm.org/docs/Phabricator.html</a>). It's not required, but it makes the reviews much easier (and much easier to keep track of).</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 17, 2016 at 10:47 PM, Mads Ravn 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Cool :) don't the sweat the time. I was just a little excited. Small patch but it's nice to get started somewhere. </p><div class="HOEnZb"><div class="h5">
<p dir="ltr">Best regards, <br>
Mads Ravn </p>
<div style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On May 17, 2016, at 2:59 AM, Mads Ravn <<a href="mailto:madsravn@gmail.com" target="_blank">madsravn@gmail.com</a>> wrote:<br>
><br>
> Hi guys,<br>
><br>
> I just wanted to check up on this patch. I heard I could just reply to this mail and see if I could 'ping' anyone in this regard. Hope it's OK.<br>
<br>
Sorry for the delay! This looks good. Committed as r269786.<br>
<br>
thanks,<br>
vedant<br>
<br>
><br>
> Best regards,<br>
> Mads Ravn<br>
><br>
> On Thu, May 12, 2016 at 6:11 PM Mads Ravn <<a href="mailto:madsravn@gmail.com" target="_blank">madsravn@gmail.com</a>> wrote:<br>
> Hi,<br>
><br>
> I have fixed the things you mentioned now. I have attached the new patch to this email.<br>
><br>
> Best regards,<br>
> Mads Ravn<br>
><br>
> On Wed, May 11, 2016 at 11:54 PM Vedant Kumar <<a href="mailto:vsk@apple.com" target="_blank">vsk@apple.com</a>> wrote:<br>
> Hi,<br>
><br>
> Thanks for the patch!<br>
><br>
> This patch is missing a small, lit-style test case. You can find examples of test cases here:<br>
><br>
>   extra/test/clang-tidy/<br>
><br>
> Apart from that, my only other nit-pick is that llvm uses 2-space indents, and spaces between "if" and "(".<br>
><br>
> If you reply to this list with an updated patch, someone would be happy to commit it for you.<br>
><br>
> best<br>
> vedant<br>
><br>
> > On May 11, 2016, at 10:01 AM, Mads Ravn via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
> ><br>
> > Hi,<br>
> ><br>
> > I would like to submit a patch for <a href="https://llvm.org/bugs/show_bug.cgi?id=27400" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=27400</a> .<br>
> ><br>
> > Beside attaching the patch, is there anything I should be aware of? I have not submitted a patch before.<br>
> ><br>
> > You can find the patch attached to this mail.<br>
> ><br>
> > Kind regards,<br>
> > Mads Ravn<br>
> > <patch_for_bug_27400.patch>_______________________________________________<br>
> > cfe-commits mailing list<br>
> > <a href="mailto:cfe-commits@lists.llvm.org" target="_blank">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/mailman/listinfo/cfe-commits</a><br>
><br>
<br>
</div>
</div></div><br>_______________________________________________<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/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>