From: Pascal B. <pa...@dy...> - 2001-02-20 18:10:54
|
duh! (sorry, just being annoying again :) not all optimisation is for speed! adding stuff to the DynLayer object will make larger memory foot prints, and in most cases it's unneeded. Adding these convenience methods was a nice idea at first.. but they are all adding up to the memory footprint of a single DynLayer. Something "we" should also take under consideration, not just your speed issues. But getting back on that subject again, I've been doing some optimising to the DynLayer, and got it down from 42 secs to 34secs with 500 layers (on a 600mhz machine).. for now only did this on Dynacore, but most stuff can be done on the official as well. I'm still no fan of your canvas ideas, it would require all users to change there code that is using DynAPI now! In my opinion once you have all dynlayer functionality in, your canvas will be slower then it is now (regardless of your addchild ideas) and might actually get as slow as dynlayer (Seeing that most code is still the same). Also, you keep mentioning that the object passing between methods is what's slowing down stuff.. then how are you going to add a child to the parent? aren't you passing the parent object then? If you would compare the speed of this current DynLayer with the one we had a year ago, you would also see big speed differences.. but eventually you need extra functionality and this is the fault with the current dynapi: it needs a big clean-up, not a complete change in code and usage. Everything got pasted onto it, without doing any optimising.. in the createelement there are a few if (is.xx) statements, that can be combined into one, and more of these optimisations. Pascal Bestebroer pa...@dy... http://www.dynamic-core.net > -----Oorspronkelijk bericht----- > Van: dyn...@li... > [mailto:dyn...@li...]Namens Eytan Heidingsfeld > Verzonden: dinsdag 20 februari 2001 16:54 > Aan: dyn...@li... > Onderwerp: RE: [Dynapi-Dev] getParent() > > > That is not what slows down the creation of layers. > It's the problem with the addChild chain that I told you. I can make the > Canvas have all these methods if you like and the time will be about the > same. > 8an > > > _______________________________________________ > Dynapi-Dev mailing list > Dyn...@li... > http://lists.sourceforge.net/lists/listinfo/dynapi-dev > |