Changes between Initial Version and Version 1 of Ticket #3925, comment 8
- Timestamp:
- May 7, 2016, 1:09:50 PM (8 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #3925, comment 8
initial v1 1 1 Looking at the TerrainEdges patch. Why did you take some type definitions outside the loops (like a, b, Edge1 and Edge2)? I don't think this would cause any performance gain (the compiler is smart enough), while it also doesn't reflect the scope of the variables anymore (they're only used in the loop). 2 2 3 Also, while improving performance on a piece of code that should be ripped out is already highly questionable, making such code "look nicer" is even more questionable.