All these loops are not there in CVS are they ? I removed them myself.
Marc van Leeuwen wrote:
> This mail was originally for Richard Bennett, but before sending, I thought
> it could be interesting for everybody on the list, so "you" == Richard in
> my writing
>
> In order to make ESC work on DynAPI "out of the box", at level 3, I have a
> few "problems". As far as I know you have full CVS access, so maybe if you
> have the time and the will you could help.
>
> There are in the latest release (2.53, I wasn't able to find back the url
> for the "current-snapshot"), ther are many many lines without ending
> semi-colons. ESC corrects some of the problems but not all, and after all,
> it is not its job . I think at some point DynAPI was fully "semi-colon"
> compliant, but some code changed/added recently has broke this. How could
> we find a way to add all those semi-colon back, in a simple manner
> regarding CVS access ?
>
> Another point is, less to do with ESC, that as far as I remember, you have
> once (on my request) removed all the "for i in anArray" expressions, in
> order to offer the possibility to add methods to the Array class. I am now
> busy adding all thos semi-colons in the DynAPI v2.53, and I still have a
> few left:
>
> ext / layer / 36: for (var i in this.parent.children) {
> ext / inline / 36: for (var j in dlyr.doc.images)
> dlyr.doc.images[j].lyrobj=dlyr;
> ext / dragdrop / 13: for (var i in this.children) {
> utils / debug / 28: for(i in this.children)
>
> -----------------------------------------------------------------
> utils / console / 40 : for (var i in obj) {
> event / mouse / 17: if (old) for(var i in old) this[i]=old[i]
> These ones are OK, because it is used properties af a "true" object, not an
> array.
>
> Maybe this could also be fixed in CVS ?
>
> Regards
> Marc
>
> _______________________________________________
> Dynapi-Dev mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/dynapi-dev
_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/dynapi-dev
