<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2015-05-06 15:00 GMT-07:00 Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On 2015 May 6, at 17:57, Alex L <<a href="mailto:arphaman@gmail.com">arphaman@gmail.com</a>> wrote:<br>
><br>
><br>
> > +  Current = skip_while(&Scanner::skip_s_white, Current);<br>
> > +  skipComment();<br>
><br>
> These two lines are both part of `scanToNextToken()`.  I wonder if you<br>
> picked the right thing to extract?<br>
><br>
> I'm not sure what you mean by this. 'skipComment' is used here as well<br>
> as in 'scanToNextToken', but the rest is different.<br>
<br>
</span>This:<br>
<span class=""><br>
    Current = skip_while(&Scanner::skip_s_white, Current)<br>
<br>
</span>looks equivalent to the code just prior to `skipComment()` in<br>
`scanToNextToken()`.<br></blockquote><div><br></div><div>It is basically equivalent, but I don't see the issue here. I think</div><div>that the 'skipComment' method should just skip the comment</div><div>if it's present at the current position and if the callers want</div><div>to skip whitespace before it they can. I think the code</div><div>benefits from such a distinction.</div></div><br></div></div>