[ scummvm-Patches-1738426 ] Minor tweak to rate.cpp

[SourceForge.net]

Patches item #1738426, was opened at 2007-06-16 17:37
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=1738426&group_id=37116

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Robin Watts (robinwatts)
Assigned to: Nobody/Anonymous (nobody)
Summary: Minor tweak to rate.cpp

Initial Comment:
Fingolfin points out that some compilers are crap at optimising array accesses due to pointer aliasing.

This patch therefore avoids using the 2 place arrays, and instead uses 2 normal variables.

This produces larger source, but any decent compiler should only keep one side of the if when it does the template instantiation, so will work out better overall.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=1738426&group_id=37116