<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> <html> <head> <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type"> </head> <body bgcolor="#ffffff" text="#000000"> On 05/18/2011 10:21 AM, Endi Sukma Dewata wrote: <blockquote cite="mid:4DD3D5DB.3080307@redhat.com" type="cite">A utility class has been added to store objects which can be accessed both as an ordered list and as a map. This class is used to store various objects including entities, fields, columns, and dialogs. The class needs to be defined in a new file called util.js to avoid circular dependency problem. A test suite for this class has been added as well. Ticket #1232 <pre wrap=""> <fieldset class="mimeAttachmentHeader"></fieldset> _______________________________________________ Freeipa-devel mailing list <a class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a> <a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/freeipa-devel">https://www.redhat.com/mailman/listinfo/freeipa-devel</a></pre> </blockquote> Good. The only thing I don't like about it is the Util namespace. I think this should be a JQuery plugin instead. Adding a Util namespace is just ignoring the problem of namespace pollution. Can you move it to JQuery, and rename the file and class ordered_map? Put some documentation in the file as to the contract: we explicitly don't hide access to the internals, but if you don't use the manipulator functions, you will corrupt the map etc. This is where Javascript would really benefit from a macro language. </body> </html>