<div dir="ltr">But still it is going to be only -6 lines which doesn't seem worth to add this new convenient function. If you can make the function simpler (in terms of semantics), I may be convinced, but the current definition of the function (return the token after the next if the next token is just one character, or otherwise return the next token) looks too arbitrary.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 30, 2016 at 8:27 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">grimar added inline comments.<br>
<br>
<br>
> ruiu wrote in LinkerScript.cpp:1658-1661<br>
<span class="">> You removed this error check, so the real delta of this patch is -4 lines. That's probably too small to introduce a new convenient function.<br>
<br>
</span>I know at least one more case when this function can be usefull.<br>
If we are going to support assigning memory regions one day, it also exist in 2 forms in the wild scripts:<br>
<br>
  *(.text) } >rom<br>
  *(.text) } > rom<br>
<br>
Probably there are other cases we do not know about yet.<br>
Generally caller code looks more clean with it then without I think.<br>
<br>
<a href="https://reviews.llvm.org/D25094" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D25094</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>