[Checkins] SVN: z3c.form/trunk/src/z3c/form/browser/widget.py Revert change 103644. The "complicated" algorithm preserves class order, so doctests don't fail.

Dan Korostelev nadako at gmail.com
Fri Sep 11 08:39:54 EDT 2009


2009/9/11 Vincent Fretin <vincent.fretin at gmail.com>

>
> I'm sorry I broke the tests.
> But instead of reverting, maybe we can sort the list and fix the doctests?
> self.klass = u' '.join(sorted(frozenset(parts)))
>
> It's just an idea, if you don't like it, I'm fine with the current code.
> :-)
>
> What's the general idea behind changing that? The original algorithm is not
so complicated and it has a somewhat feature that it preserves the order in
which classes were added, that can be nice for rendered code readability in
some cases (however, I personally didn't ever needed that). Is there any
significant difference in performance?

-- 
WBR, Dan Korostelev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.zope.org/pipermail/checkins/attachments/20090911/f4575785/attachment.html 


More information about the checkins mailing list