On Sat, 21 Jan 2012 10:53:20 -0800, Ian Romanick <i...@freedesktop.org> wrote:
> The fix seems correct, but reading that regex made my head hurt.

Thanks. And, yes, it was pretty bad before, and the copying made it much
worse.

> Is there a way to refactor the character classes to be more readable?
> OTHER is basically
> [{PUNCTUATION}{IDENTIFIER_FIRST_CHAR}[:space:]][{PUNCTUATION}[:space:]]*,
> right?

Right, (other than that those classes should be negated). I'll see how
to make something clear before pushing the change.

Thanks for the feedback.

-Carl

Attachment: pgp2GXwUIX0nW.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to