[vJASS] Lib conflict (function redeclared)

[Arcsol]

Hello everyone. As I was adding some libraries and systems to a map, I noticed that upon saving, Jasshelper would give error regarding a GetUnitId function. The error itself is that it says "function redeclared". This happened when I imported AutoIndex. I'm no vJASS expert but I noticed that this same function is used in UnitUserData library, so I assume these two are the ones in conflict.

How can I solve this issue? Thanks in advance.

 

[Adiktuz]

It's an incompatibility issue, there are two ways that I know of

1) Remove/Rename the function on one of the libraries
2) Remove one of the libraries

 

[PurgeandFire]

Since you're going to end up with two libraries manipulating user data, you should remove one and adapt the systems that use them to use the other one. Having two unit indexers is not a good idea.

Remove whichever one is used less often. If you need help making some script use the other indexer, then just post the indexer + the script in question and it should be an easy fix.

 

[Arcsol]

Oh, I see. So not exactly 'easy' way around for us JASS newbies Dx
If I recall correctly, the first library came from here: http://www.hiveworkshop.com/forums/...-a-131394/?prev=search=ABORTspell&d=list&r=20

Then I tried adding BonusMod (which uses AutoIndex): http://www.wc3c.net/showthread.php?t=107940

I haven't used the libraries for anything else but for what it's required in those two.

 

[PurgeandFire]

Oh, okay. That is a very simple fix. Here is the updated spell. You can copy over the libraries from it to replace your current ones, and then delete the "UnitUserData" trigger. It is attached below.

Better yet, if you're up to fixing it yourself, here is how:

Since "UnitUserData" and "AutoIndex" both use GetUnitId, and "UnitUserData" only offers the function "GetUnitId", you can simply delete UnitUserData and make the other libraries require AutoIndex. It will be perfectly compatible. The only one that uses "UnitUserData" is "LastOrder", so simply click the "LastOrder" trigger, and find this line (it is at the very top):

library LastOrder initializer Init needs UnitIndexingUtils

Change that to this:

library LastOrder initializer Init needs AutoIndex

And then you're good to go! Just make sure you delete the "UnitUserData" trigger to avoid any conflicts.

 

[Arcsol]

Yup, that was really easy x3
Thanks a lot!