<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 24, 2014 at 12:34 PM, Dinesh Dwivedi <span dir="ltr"><<a href="mailto:dinesh.d@samsung.com" target="_blank">dinesh.d@samsung.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">*** updated comment for typos<br>
<div class=""><br>
Hi Klimek,<br>
<br>
Thanks for review.<br>
<br>
With this change, I am trying to figure out whether there is a "gap" in the line.<br>
</div>If this gap is just white spaces, that is not an issue and we can still merge<br>
function. But if there are other constructs such as preprocessor directives, we<br>
should not merge this function to single line.<br></blockquote><div><br></div><div>With "gap" I mean that the UnwrappedLine has a sequence of tokens that are not also subsequent in the original token stream. The correct fix is to figure out whether that has happened.</div>
<div>Architecturally, we put the #ifdef directives into different UnwrappedLines, and the UnwrappedLine with the code then has non-continuous tokens in it.</div><div><br></div><div>Now that I think about it - those tokens also need to have MustBreakBefore set - can we just check that we never merge lines when one of the tokens has MustBreakBefore set?</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
During debugging, I have observed that if there are other constructs, block before<br>
and after the construct (inclusive) are handled separately [Basically, I am trying<br>
to say is I found TheLine->MustBeDeclaration to be false for 'after' block in such<br>
cases]. I assumed, just checking for this case is good enough, as asserted from<br>
test cases too.<br>
<br>
If you think this is not correct, I will look in to code and try to find better fix.<br>
<br>
Regards<br>
<br>
<a href="http://reviews.llvm.org/D3439" target="_blank">http://reviews.llvm.org/D3439</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>