Re: [Dynapi-Dev] Bug with Dynlayer's deleteAllChildren() & deleteChild() methods

[Robert R. <rra...@ya...>]

Here are some of the questions I have with the patch:

DynLayer.prototype.removeChild
------------------------------
"var oChild=this.children[i]"
- this is the same as child, right?  so why do we need it, child does
not change in the loop.  It is set before the for loop.  Also, if the
child does not exist, the oChild is never set and will be null.  Won't
this kill the removeFromArray using a null parameter.

"return arguments[arguments.length-1]"
- we need this (sort of).  This allows you to say something like
addChild(removeChild(child)).  Its not used often, but it is very
similar to addChild(new DynLayer()).



\\Robert
-- 
rra...@ya...