<div><br></div>Ah yes, I forgot about emitting a reasonable diagnostic.  How's this?  I wasn't sure what the best behavior was after the error was detected, so I just allowed it to continue parsing.  Let me know if that's incorrect.<div>
<br></div><div><br></div><div>-- Anis Ahmad<br><br><div class="gmail_quote">On Mon, Jun 15, 2009 at 9:07 AM, Douglas Gregor <span dir="ltr"><<a href="mailto:dgregor@apple.com">dgregor@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im"><br>
On Jun 11, 2009, at 8:33 AM, Anis Ahmad wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
While familiarizing myself with the code, I noticed a comment to fix the handling of namespace aliases by ensuring no attributes were specified for the namespace (as per the spec).  Unless I'm deeply mistaken, it seemed like it could be handled by a simple refactor.<br>

</blockquote>
<br></div>
While this patch does fix the problem, the error message we get ("expected '{'") isn't very good. I would prefer if we parse both the attribute and then the "=", but emit a diagnostic that states explicitly that attributes are not permitted on namespace aliases. That will give the user a much better understanding of what went wrong.<br>

<br>
        - Doug<br>
</blockquote></div><br></div>