<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 11, 2014 at 6:51 PM, Carl Worth <span dir="ltr"><<a href="mailto:cworth@cworth.org" target="_blank">cworth@cworth.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> writes:<br>
>> #if 0<br>
>> #else garbage<br>
>> #endif<br>
>><br>
>> The Khronos test contains two tokens after #else. I'm not sure if that<br>
>> is the difference or not.<br>
> Khronos is testing many different cases. Failing case tests:<br>
><br>
> #if 1<br>
> #else garbage<br>
> #endif<br>
<br>
</div>It is the "#if 1" that causes the existing code to catch the garbage<br>
after #else. So yes, the existing test case does need to be expanded.<br>
<br>
In the meantime, the proposed fix (to the lexer) is not correct as it<br>
will cause an error in the following case which should be accepted:<br></blockquote><div>Right. I'll drop this patch.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
#if 1<br>
#else /* comments are whitespace, not garbage */<br>
#endif<br>
<br>
I'll poke around at the existing code to see why the garbage after #else<br>
is caught with #if 0, but not with #if 1.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Carl<br>
<br>
--<br>
<a href="mailto:carl.d.worth@intel.com">carl.d.worth@intel.com</a><br>
</font></span></blockquote></div><br></div></div>