<div class="gmail_quote">2009/9/8 Vincent Fretin <span dir="ltr">&lt;<a href="mailto:vincent.fretin@gmail.com">vincent.fretin@gmail.com</a>&gt;</span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Log message for revision 103644:<br>
  Replaced the complicated algorithm by a frozenset<br>
<br>
Changed:<br>
  U   z3c.form/trunk/src/z3c/form/browser/widget.py<br>
<br>
-=-<br>
Modified: z3c.form/trunk/src/z3c/form/browser/widget.py<br>
===================================================================<br>
--- z3c.form/trunk/src/z3c/form/browser/widget.py       2009-09-08 13:14:57 UTC (rev 103643)<br>
+++ z3c.form/trunk/src/z3c/form/browser/widget.py       2009-09-08 13:21:34 UTC (rev 103644)<br>
@@ -54,16 +54,9 @@<br>
         if not self.klass:<br>
             self.klass = unicode(klass)<br>
         else:<br>
-            #make sure items are not repeated<br>
-            parts = self.klass.split()+[unicode(klass)]<br>
-            seen = {}<br>
-            unique = []<br>
-            for item in parts:<br>
-                if item in seen:<br>
-                    continue<br>
-                seen[item]=1<br>
-                unique.append(item)<br>
-            self.klass = u&#39; &#39;.join(unique)<br>
+            # Make sure items are not repeated.<br>
+            parts = self.klass.split() + [unicode(klass)]<br>
+            self.klass = u&#39; &#39;.join(frozenset(parts))<br clear="all"></blockquote></div><br>This change breaks all doctests that have rendered widget classes. Did you not test it before commit? Please, either fix the tests or revert the change.<br>

<br>-- <br>WBR, Dan Korostelev<br>