On Jun 10, 2006, at 04:22 UTC, Andy Dent wrote: > Note that in RB2006, rgbSurface .pixel is now a virtual function call. > > Whilst waiting for some more generic inlining solution, you may want > to sign on and encourage a special just for this function > > http://www.realsoftware.com/feedback/viewreport.php?reportid=hjhkjuys
In my tests, it wasn't usually RGBSurface.Pixel which typically caused the slowdown in pixel-twiddling loops, buth rather RGBSurface.Red, .Green, and .Blue. These have a LOT of overhead for what they do, and there's no reason for them to be function calls at all, since Color is not a class (ergo they aren't virtual, and can't be). So, IMHO, special-casing these three accessors in the compiler so that it boils down to some inline shifting and masking rather than a function call would be safe (doesn't change behavior at all) and offer a greater benefit. Best, - Joe -- Joe Strout -- [EMAIL PROTECTED] Verified Express, LLC "Making the Internet a Better Place" http://www.verex.com/ _______________________________________________ Unsubscribe or switch delivery mode: <http://www.realsoftware.com/support/listmanager/> Search the archives of this list here: <http://support.realsoftware.com/listarchives/lists.html>
