From: Jordi - I. - M. <jmi...@or...> - 2001-02-20 19:35:31
|
Absolutely. When I talk 'rewriting things' I'm not saying we should rebuild the code but rewrite what we already have. Now we know all the "ifs" that are goiung to be there, so we can organize the internal code in a better way. Pascal Bestebroer wrote: > 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 > > > > _______________________________________________ > Dynapi-Dev mailing list > Dyn...@li... > http://lists.sourceforge.net/lists/listinfo/dynapi-dev |