Changes between Version 1 and Version 2 of Ticket #2022, comment 10


Ignore:
Timestamp:
Dec 29, 2013, 5:50:26 PM (10 years ago)
Author:
wraitii

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #2022, comment 10

    v1 v2  
    22The idea behind the patch is simply to make class members a few variables that otherwise are temporaries in CalculateTerritories(), which is probably called often enough that this makes sense, even though it's not a huge improvement.
    33I've added a few comments starting with "wraitii" to explain a few things. I've inlined the grid functions, and left those he added though we likely don't need them.
     4Kuranes had a liking to save the end of loops in a local variable just in front of the loop. It's fairly ugly, might be or might not be faster (should check assembly code). I've kept most of them.
    45
    56This change was still done fairly quickly so I haven't really checked that pointer behavior is sound and there may be segfaults. But it should be reviewable.