> Martin Mathieson wrote:
>
> > name           [A-Za-z][-a-z0-9_]*[-a-zA-Z0-9_]*
>
> Wouldn't
>
> [A-Za-z][-a-zA-Z0-9_]*
>
> suffice?  ([...]* matches zero or more occurrences, and [-a-zA-Z0-9_] is
> a superset of [a-z0-9_].)

That would have been the obvious fix to make in the first place, I was
lazily assuming it hadn't worked (and there was some grumbling about crashes
with some versions of lex/flex...).

Your simpler version works for me.

I don't have the original DTD file with the names beginning with caps, bug I
just nobbled one or 2 fields of an existing file.  The parsing doesn't fail,
but the capitalisation doesn't show in the Filter Expression editor with
either version.  I don't know if it would be able to match the filter with a
field that showed the capitalisation...

Anyway, I think your version should be applied.
Martin

_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@wireshark.org
http://www.wireshark.org/mailman/listinfo/wireshark-dev

Reply via email to