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

Reply via email to