Return Bug still Works

[Romek]

This was discovered a few days ago. I don't think anybody posted it here though:

scope BlizzardFail initializer Init

    function H2I takes handle h returns integer
        return h
        call DoNothing()
        return 0
    endfunction
    
    private function Init takes nothing returns nothing
        call BJDebugMsg(I2S(H2I(CreateTimer())))
    endfunction

endscope

Fail, Blizzard. Fail.

 

[WolfieeifloW]

Isn't GetHandleId() still better/faster though?

 

[_whelp]

But there are no new natives for the other ones.

function C2I takes code c returns integer
    return c
    call DoNothing()
    return 0
endfunction

 

[Romek]

> Isn't GetHandleId() still better/faster though?
Yes.

There's no reason to use this, as it'll probably be fixed (properly) soon anyway.
I just thought it was interesting that Blizzard made broke most JASS systems, most maps with JASS in them, and annoyed many mappers with need to find+replace, just to fix a bug...

...Which wasn't even fixed.

> But there are no new natives for the other ones.
That's the dangerous one which caused all of this.

 

[Akolyt0r]

strange, when i tested this (abuse) yesterday it didnt work for me...

EDIT: i get syntax error now, "cant convert handle h to integer" ... well i use Zoxc's Parser, which enforces strict type safety ...to laze to switch to pjass for testing this again right now.