On Aug 30, 2005, at 6:01 AM, Hamish Lawson wrote:

Neil Schemenauer wrote:


if charset is not None or not content_type.startswith ('text/'):
            self.charset = charset


This code is deciding two things: whether the user asked for charset
to be changed, and whether they are allowed this change. I think it
would be better if it just did the first.

I agree that this would be clearer:  the downside being
that publisher.set_content_type('image/png') would not
change the charset (as Neil wants).

*If* Quixote is to adopt a
policy of outputting a charset parameter only for text/* types,

This change to set_content_type doesn't create that policy.
You can still
publisher.set_content_type('image/svg', 'utf-8')




_______________________________________________
Quixote-users mailing list
[email protected]
http://mail.mems-exchange.org/mailman/listinfo/quixote-users

Reply via email to