Re: [Dynapi-Dev] DynLayers elements not destroyed.

[Kevin <ke...@ke...>]

See below:

"Raymond Irving" <xw...@ya...> wrote:

> See below:
> 
> --- Kevin <ke...@ke...> wrote:
> > Hello,
> > 
> > If elements are removed and then some more are added
> > we may have a memory leak. Using Mozilla and
> > example:
> > dynapi.api.dynlayer-destroy.html
> > the elements aren't removed when the Destroy all
> > link is
> > clicked.
> > 
> > Suggested fix
> > dynlayer_base.js:p._remove
> > 
> > -this.elm.innerHTML = "";
> > -this.elm.outerHTML = "";
> > +//this.elm.innerHTML = "";
> > +//this.elm.outerHTML = "";
> > +var p=this.parent.elm;
> > +if(p) p.removeChild(this.elm);
> > 
> > This seems to work fine in IE too..
> 
> I've added the above as well.
>  
> > NS4 has a recycled array in
> > dynlayer_ns4.js:p._remove
> > but I don't see the space reused. Do we have growing
> > memory here and perhaps the Layer should be deleted
> > rather than hidden?
> 
> Well, IMO if they're stored inside a recycle array
> then they should be reused. Any ideas on how we should
> do this?

No ideas as we do, new Layer() or delete and not an OS
specific malloc or free. NS4 may have memory management
bugs as Doug says. Who implemented the recycled array?
Perhaps it was an NS4 bug work around, as doing a delete
on a Layer was proven to be bad news?

-
Kevin
 
> --
> Raymond Irving
> 
> > -
> > Kevin.
> > 
> > 
> > 
> >
> -------------------------------------------------------
> > This SF.net email is sponsored by: ValueWeb: 
> > Dedicated Hosting for just $79/mo with 500 GB of
> > bandwidth! 
> > No other company gives more support or power for
> > your dedicated server
> >
> http://click.atdmt.com/AFF/go/sdnxxaff00300020aff/direct/01/
> > _______________________________________________
> > Dynapi-Dev mailing list
> > Dyn...@li...
> >
> http://www.mail-archive.com/dyn...@li.../
> 
> 
> __________________________________________________
> Do you Yahoo!?
> Yahoo! Tax Center - File online, calculators, forms, and more
> http://tax.yahoo.com
> 
> 
> -------------------------------------------------------
> This SF.net email is sponsored by: ValueWeb: 
> Dedicated Hosting for just $79/mo with 500 GB of bandwidth! 
> No other company gives more support or power for your dedicated server
> http://click.atdmt.com/AFF/go/sdnxxaff00300020aff/direct/01/
> _______________________________________________
> Dynapi-Dev mailing list
> Dyn...@li...
> http://www.mail-archive.com/dyn...@li.../