search

adamqqqplay / dota2ai

Ranked Matchmaking AI: An improved Dota2 AI based on Valve's default AI. Has more than 3 million subscribers on Steam.
http://steamcommunity.com/sharedfiles/filedetails/?id=855965029
GNU General Public License v3.0
310 stars 85 forks source link

Fix Remove() function #108

Closed JoshdanG closed 1 year ago

JoshdanG commented 1 year ago

  1. Previous version would result in nil prematurely terminating array. New implementation fixes:

    • pudge constantly hooks despite units in the way
    • lifestealer won't infest allied heroes
    • abaddon mist coil (with separate commit)
    • not used anywhere else

  2. Also, delete unused Linq.Remove to avoid confusion

JoshdanG commented 1 year ago

Pull requests for pudge and lifestealer are tangentially related but independent. They can be merged in any order or combination.

adamqqqplay commented 1 year ago

Pull requests for pudge and lifestealer are tangentially related but independent. They can be merged in any order or combination.

OK, I will merge #108 first.