<div dir="ltr">On Fri, Sep 6, 2013 at 2:50 AM, Enea Zaffanella <span dir="ltr"><<a href="mailto:zaffanella@cs.unipr.it" target="_blank">zaffanella@cs.unipr.it</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 09/05/2013 10:12 PM, Eli Friedman wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
On Thu, Sep 5, 2013 at 12:34 AM, Enea Zaffanella <<a href="mailto:zaffanella@cs.unipr.it" target="_blank">zaffanella@cs.unipr.it</a><br></div>
<mailto:<a href="mailto:zaffanella@cs.unipr.it" target="_blank">zaffanella@cs.unipr.it</a><u></u>>> wrote:<br>
</blockquote>
[...]<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
    To summarize, I see two options:<br>
<br>
    (a) rename getParenRange() as getParenOrBraceRange();<br>
<br>
    (b) have two different methods, getParenRange() and getBraceRange(),<br>
    to be used based on the value of a boolean flag.<br>
    Question: is it the case that<br></div>
       bool CXXConstructExpr::__<u></u>isListInitialization()<div class="im"><br>
    is such an appropriate flag to be used here?<br>
<br>
<br>
Yes.<br>
<br>
    I would probably opt for (b).<br>
    Do you see a third option which is better?<br>
<br>
<br>
I don't really see any other options; I guess I would lean towards<br>
option A, and let the users branch if they care about the difference,<br>
but either way works.<br>
</div></blockquote>
<br>
Please find attached a revised patch implementing option A.<br>
OK to commit?<br><span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>LGTM.</div><div><br></div><div>-Eli </div></div><br></div></div>