<div dir="ltr">Hi David,<div><br></div><div>Thanks for the tips. I probably should have guessed that the formatting changes would be an impediment. </div><div><br></div><div>I’ve split up the patch into 4 pieces and resubmitted for review.</div><div><br></div><div>Yours,<br></div><div>Russ Magee %-)</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 21, 2016 at 12:17 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Generally once you send code for review, if no review is forthcoming the next step(s) are to find a specific reviewer (looking at the blame lists for the files you're changing can give you some hints there) to CC on the review thread and/or to 'ping' (send a reply-all to the thread with the word "ping" usually) the thread on a no-more-than-weekly basis, in case anyone missed it/was too busy the first time it was sent.<br><br>One thing I will suggest, from a cursory glance, is that formatting changes along with semantic changes in a single patch are generally frowned upon - it's probably best just to implement the semantic change alone for now & ask if the formatting change is desirable (either just ask the reviewer directly, or ask in the form of a separate patch submission (either separate email thread or as a separately attached patch to the existing thread in a "oh, and I have this too if you think it's a good idea"))</div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Wed, Jan 20, 2016 at 5:54 AM, Russell Keith-Magee via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr"><span style="font-size:13px">Hi all,</span><div style="font-size:13px"><br></div><div style="font-size:13px">I’ve recently submitted a patch to cfe-patch updating the Python bindings for clang to be compatible with Python 3:</div><div style="font-size:13px"><br></div><div style="font-size:13px"><a href="http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20160111/146926.html" target="_blank">http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20160111/146926.html</a><br></div><div style="font-size:13px"><br></div><div style="font-size:13px">However, it’s not clear what the next step in the process is. Can anyone advise what the next steps would be in getting this patch committed?</div><div style="font-size:13px"><br></div><div style="font-size:13px"><div>Yours,</div><div>Russ Magee %-)</div></div></div>
<br></div></div>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>