search

thirdy / durian

Path of Exile Items watcher
http://thirdy.github.io/durian/
GNU General Public License v2.0
35 stars 10 forks source link

Added early Bailout #75

Closed zocke1r closed 8 years ago

zocke1r commented 8 years ago

apparently the last time i added that i added it to the normal affix lookup instead of the maxaffixLookup, still relies on the assumptions that the affix file is sorted

thirdy commented 8 years ago

is this for efficiency?

zocke1r commented 8 years ago

yes, otherwise it would have to run over the entire file for every single mod, just to determine what the max tier is, with this change it stops after finding the first mod that is not part of the correct group after having found the correct mod group,

thirdy commented 8 years ago

great

On Sat, Feb 20, 2016 at 8:39 AM, Dom notifications@github.com wrote:

yes, otherwise it would have to run over the entire file for every single mod, just to determine what the max tier is, with this change it stops after finding the first mod that is not part of the correct group after having found the correct mod group,

— Reply to this email directly or view it on GitHub https://github.com/thirdy/durian/pull/75#issuecomment-186467820.

http://sites.google.com/site/thirdyderivera/