2012/1/30 Rafael Espíndola <[email protected]>

> 2012/1/30 Rafael Espíndola <[email protected]>:
> >> Is it possible to error on the #prama, even when diagnosing the case
> about
> >> the namespace end? That would be ideal...
> >
> > Not sure what you mean. We do error on:
>
> I think I figured it out. You want the error to be in the pragma line
> and the note to point to the end of the namespace, right?
>
> The attached patch implements this. Let me know what do you think.
>

I think we could be more explicit that the problem is caused by the
namespace having a visibility attribute. Possible wording for the three
cases:

error: #pragma visibility push with no matching #pragma visibility pop
note: surrounding namespace with visibility attribute ends here

error: #pragma visibility pop with no matching #pragma visibility push
note: surrounding namespace with visibility attribute starts here

error: #pragma visibility pop with no matching #pragma visibility push

- Richard
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to