Stefan Behnel <stefan...@behnel.de> writes:
> >         # Keep a charbuffer to handle the escapeFlag
> >         if self.contentModelFlag in\
> >           (contentModelFlags["CDATA"], contentModelFlags["RCDATA"]):
> Is the tuple
>       (contentModelFlags["CDATA"], contentModelFlags["RCDATA"])
> constant? If that is the case, I'd cut it out into a class member ...

I think the main issue for that function comes after that if statement.
There is a multi-way switch on a bunch of different possible character
values.  I do agree with you that the first "if" can also be optimized.
-- 
http://mail.python.org/mailman/listinfo/python-list

Reply via email to