>Just going to strcmp by tiself is likely to be a problem, since there
>are old .abw documents that use uppercase for attributes.  However, we
>ought to be able to take advantage of the short-circut or and do
>something like this:
>
>if (strcmp(propName, footer) || strcmp(propName, FOOTER))
>
>There should be no mixed case attributes in the wild, so this
>shouldn't break any non-handcrafted documents. And it would be lots
>faster.

Supporting uppercase attributes should be abandoned and my next patch will 
abandon it forever. It is *bad* XML.

<p props=""> and <p PROPS=""> should not both validate against any DTD. 
Supporting uppercase should be considered a bug. The file format is not set 
in stone and not yet at 1.0. Documents with these settings tend to be very 
old ones or hand-crafter anyway.

Dom


_________________________________________________________________
Get your FREE download of MSN Explorer at http://explorer.msn.com


Reply via email to