[Laszlo-dev] For Review: Change 20080812-bargull-R01 Summary: replace for-in loops over Arrays

André Bargull andre.bargull at udo.edu
Tue Aug 12 05:34:22 PDT 2008


Change 20080812-bargull-R01 by bargull at dell--p4--2-53 on 2008-08-12 12:09:05
in /home/Admin/src/svn/openlaszlo/trunk
for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: replace for-in loops over Arrays

New Features:

Bugs Fixed: LPP-4435 (only lfc-part)


Technical Reviewer: promanik
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:
Replaced for-in loops over Arrays with a normal for-loop.
This change is essential if you use Array extensions (e.g. from 
Prototype), or you'll end up iterating over the added properties.
Open issue: The Profiler uses Arrays instead of Objects explicitly, not 
sure what to do here, because all Arrays are sparse Arrays, so iterating 
over them in a normal for-loop isn't a good idea. And using 
hasOwnProperty(..) should be avoided too, as it will affect performance.
Plus: Removed the IE-workaround in LzParam, no longer needed due to the 
new class structure.


Tests:

Files:
M WEB-INF/lps/lfc/kernel/swf/LzFontManager.as
M WEB-INF/lps/lfc/services/LzTrack.lzs
M WEB-INF/lps/lfc/services/LzFocus.lzs
M WEB-INF/lps/lfc/debugger/LzDebug.lzs
M WEB-INF/lps/lfc/data/LzHTTPDataProvider.lzs
M WEB-INF/lps/lfc/data/LzDataset.lzs
M WEB-INF/lps/lfc/data/LzParam.lzs

Changeset: 
http://svn.openlaszlo.org/openlaszlo/patches/20080812-bargull-R01.tar




More information about the Laszlo-dev mailing list