lep / pjass Goto Github PK

Pjass considers unary + to number as a syntax error, although it works in the game

local real time = LSReal("Time", + .05)

https://github.com/Luashine/wc3-test-maps/tree/master/variable-length-crash

In Jass, variable length of exactly 3959 or above will crash old versions of the game when loading "war3map.j". This is true for locals and globals.

Edit:

Appears to have been fixed in Reforged (1.33.0.19203). I didn't check which version.

Both war3map.j and the map are included here.

Currently: no warning, no error.

https://github.com/lep/pjass/blob/ac58f84a23afe447621dff7ffd84e36a92295817/tests/should-check/modulo.j

I have tested the modulo operator in 1.27 and it's definitely breaking the script:

// OK
function hexadecimalNumber takes nothing returns nothing
	local integer i = 10
	
endfunction

vs

// script doesnt load
function hexadecimalNumber takes nothing returns nothing
	local integer i = 10 % 4
	
endfunction

Is it possible to process multiple files via stdin? Tried concatenating common.j + '\n' + blizzard.j but does not seem to be working (trying to parse wa3map.j from a map with a custom blizzard.j without having to dump the blizzard.j file to disk).

I didn't record it, but I think the crash and error message was exactly the same as in #5 if you put a "mac-style" CR war3map.j file in your map. This leaves us with the only valid line endings being \r\n and \n

Tested 1.27 and Reforged 1.33.0 (Ref doesn't outright crash but at the very least doesn't load the script too)