<p dir="ltr"><br>
On Apr 11, 2013 1:43 PM, "Harald van Dijk" <<a href="mailto:harald@gigawatt.nl">harald@gigawatt.nl</a>> wrote:<br>
><br>
> Hi all,<br>
><br>
> As mentioned in <a href="http://llvm.org/bugs/show_bug.cgi?id=12767">http://llvm.org/bugs/show_bug.cgi?id=12767</a>, and what I again reported as <a href="http://llvm.org/bugs/show_bug.cgi?id=15661">http://llvm.org/bugs/show_bug.cgi?id=15661</a>, when concatenating x ## y, argument expansion for y should be suppressed even when x is empty. Does this approach look okay? All preprocessor tests pass, a few other tests were already failing for me and continue to fail the same way, there are no new failures.<br>

><br>
> This is my first contribution to clang, I am not yet familiar with the process. I should not need to do more than send this patch to the list, correct?</p>
<p dir="ltr">That's the general idea, yes. Assuming your patch includes test cases for the fix/added functionality.</p>
<p dir="ltr">You may find that code review doesn't happen in a timely manner because people are busy. It may be necessary to 'ping' (simply by replying to the thread on the mailing list) to get attention if it seems to have been forgotten. Any more frequently than weekly would be considered poor form.</p>

<p dir="ltr">><br>
> Cheers,<br>
> Harald<br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</p>