View Single Post
Old 10-26-2009, 11:56 AM   #25
grim001
requires vJass
 
grim001's Avatar


Code Moderator
 
Join Date: Nov 2006
Posts: 1,540

Submissions (10)

grim001 is just really nice (277)grim001 is just really nice (277)

Send a message via AIM to grim001
Default

OK, I tested it and everything works fine, I only have two comments:
  • You use xe's preload function in a struct onInit method, so the preloading unit won't exist yet. This is more like a problem with xe, so I made a post in the xe thread asking for vex to fix it.
  • I think it would be better to merge the Table and AutoIndex versions together, although it would be nice if "requires libraryA or libraryB" was possible.

Anyway, I think this is ready for approval now, so I'll approve it and graveyard ADamage.
grim001 is offline   Reply With Quote