Here are the files and counts within each where we're using for..in. I have not looked at them yet to make sure they're array accesses, but for anyone that wants to, converting to for(;;) should be fairly trivial.
miker@whirly:~/svn/head-ILS$ gack 'for \(\s*(?:var )?i in' --type=js --ignore-dir=chrome --ignore-dir=xul --ignore-dir=acq --ignore-dir=backend --ignore-dir=conify --ignore-dir=booking --ignore-dir=reports -c |grep -v 0$ Open-ILS/web/opac/common/js/opac_utils.js:1 Open-ILS/web/opac/common/js/jscalendar/calendar-setup.js:1 Open-ILS/web/opac/common/js/jscalendar/calendar.js:1 Open-ILS/web/opac/common/js/org_utils.js:4 Open-ILS/web/opac/skin/default/js/rdetail.js:1 Open-ILS/web/opac/skin/default/js/result_common.js:2 Open-ILS/web/js/dojo/MARC/Field.js:2 Open-ILS/web/js/dojo/MARC/Record.js:2 Open-ILS/web/js/dojo/openils/widget/TranslatorPopup.js:1 Open-ILS/web/js/dojo/openils/I18N.js:1 Open-ILS/web/js/dojo/fieldmapper/dojoData.js:9 Open-ILS/web/js/dojo/fieldmapper/OrgUtils.js:7 Open-ILS/web/js/dojo/fieldmapper/hash.js:2 miker@whirly:~/svn/head-ILS$
I'll take the Dojo stuff ... anyone want to look at the opac proper?
--miker
Here are the files and counts within each where we're using for..in. I have not looked at them yet to make sure they're array accesses, but for anyone that wants to, converting to for(;;) should be fairly trivial.
miker@whirly: ~/svn/head- ILS$ gack 'for \(\s*(?:var )?i in' --type=js --ignore-dir=chrome --ignore-dir=xul --ignore-dir=acq --ignore- dir=backend --ignore-dir=conify --ignore- dir=booking --ignore- dir=reports -c |grep -v 0$ web/opac/ common/ js/opac_ utils.js: 1 web/opac/ common/ js/jscalendar/ calendar- setup.js: 1 web/opac/ common/ js/jscalendar/ calendar. js:1 web/opac/ common/ js/org_ utils.js: 4 web/opac/ skin/default/ js/rdetail. js:1 web/opac/ skin/default/ js/result_ common. js:2 web/js/ dojo/MARC/ Field.js: 2 web/js/ dojo/MARC/ Record. js:2 web/js/ dojo/openils/ widget/ TranslatorPopup .js:1 web/js/ dojo/openils/ I18N.js: 1 web/js/ dojo/fieldmappe r/dojoData. js:9 web/js/ dojo/fieldmappe r/OrgUtils. js:7 web/js/ dojo/fieldmappe r/hash. js:2 ~/svn/head- ILS$
Open-ILS/
Open-ILS/
Open-ILS/
Open-ILS/
Open-ILS/
Open-ILS/
Open-ILS/
Open-ILS/
Open-ILS/
Open-ILS/
Open-ILS/
Open-ILS/
Open-ILS/
miker@whirly:
I'll take the Dojo stuff ... anyone want to look at the opac proper?
--miker