<div dir="ltr">Hi,<div><br></div><div>I have fixed the things you mentioned now. I have attached the new patch to this email.</div><div><br></div><div>Best regards,</div><div>Mads Ravn</div><div dir="ltr"><br><div class="gmail_quote"><div dir="ltr">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></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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>
</blockquote></div></div></div>