search

nestharus / JASS

MIT License
31 stars 8 forks source link

Approach on new Unit Indexer? #5

Closed nestharus closed 10 years ago

nestharus commented 10 years ago

Check out the new Unit Indexer thus far. It should work ^)^. Original is still up, so don't cry on me.

I updated Trigger so that it wouldn't have the crazy memory constraints that it had. Given the Trigger update, should I make the new UnitIndexer use Trigger instead of Event?

The pro of using Trigger is registering/unregistering both code and triggers. Furthermore, can easily do unit specific deindex events. The unit specific deindex events would be very nice for Unit Indexer =). Could keep Event up for indexing, but kind of want to use Trigger for deindexing.

Oh, I'll also work in GUI support.

GunGunGun commented 10 years ago

Glad to see you will add GUI support to Unit Indexer. :D