On Sat, 3 Jul 2004, bulia byak wrote:
I've checked in a patch to IntLigne::Copy(FloatLigne*) that improves performance on my box.
Thanks Bryce, it does help. The tutorial-basic used to take 3.0 sec on average to load, now it loads in 2.75 sec. I haven't seen any rendering problems. Still, I'd suggest that you postpone further optimizations until after the release because they are rather dangerous and not too urgent (we're already notably faster than 0.38).
Yeah, I actually was planning on just putting the code in the patch tracker for after the release, but mental reviewed the patch and said it was worth committing, so I felt it'd be safe enough. However, yes I have decided to postpone further work until after the release.
Fwiw, I did take a look at a few more routines to see if there were obvious optimizations. Shape::SwapEdges(int,int) scored high, but I didn't see anything obvious. It appears to be mainly a big if/else chain, so I'm wondering if it could be represented as a data structure; if so, possibly the compiler could figure out better optimization paths for it.
Shape::ConvertToShape is another one that could use attention. The reason it is at the top of the profiling list is because it's just such a _long_ function. It probably could benefit from being broken up into some sub-functions, as it is around 800 lines currently.
Bryce