Why does this crash?

I needed a function to convert integer to boolean, and I wrote 1, but now everytime I try to save my map, the editor crashes when saving variables. Please tell me what's wrong with this:

function I2B takes integer a returns boolean
if a=0 then
return false
else
return true
endif
endfunction

a == 0 instead of a = 0

may I place a suggestion?
function I2B takes integer I returns boolean
return i
return false
endfunction

The return bug works between ints and booleans to, as far as I remember 0 is false and anything else is true.

do you really think we should rely on bugs? even if it works...

Thanks, it fixed it, and thanks for return bug suggestion, I didn't think of that. It's not really a bug, just another way of tricking the editor not to complain about it. It's like NoLimits program, it just solves the issue with Editor refusing to save the map because of this. Besides, Cubasis said that Blizzard use return bug in their own maps too.

There has been an official statement by Blizzard that the return bug will not be "fixed".

How exactly does the return bug work in functions like those?

It just returns what the variable contains in the new format.

All types in WC3 are ... 32 bit.

Ints contain their value.
Reals contain their value using normal float standard.
Booleans contain their value as 0=false <everything-else>=true.
Strings contain a address to a certain String array where the specified string is stored.
Handle types contain a address to where it is stored in memory (well, not really, but where in the WC3 memory it is...or whatever). So you'll get a value like... 96000 or sumtin... if you return it as a integer.

So, as they are all 32-bit, we can fool the compiler/game to think that a integer is a boolean... or a location is a integer, and thus, we can f.ex. store a UnitGroup, as a integer (its address), in Game Cache.

And the way it works is that.... the compiler only checks/complains about the "last" return statement in a function. That's why we can return anything we want, just if we put a legit dummy-return statement at the end.

~Cubasis

I see..thank you. So then you just put that integer into a game cache and then take out the integer from game cache and convert it back to unit etc?