Known bugs in JASS

[Pyrogasm]

I can confirm Darkwulfv's bug; UnitDamagePoint will desync Mac users.

[Vexorian]

pretty lame, someone report the bug to blizz' site

[Pyrogasm]

Methinks this thread could use some cleanup... who wants to do it?

[Toadcop]

O.O =)
Boolexpr code MUST return boolean otherwise it may DeSync...
example:

JASS:

  function XXX takes nothing returns nothing
     ...
  endfunction

  function XXX2 takes nothing returns boolean
     ...
     return false
  endfunction

 ...
 ...Condition(function XXX) // - may desync if in this functions are asynchrone actions. (or maybe some other reasons)

 ...
 ...Condition(function XXX2) // will work fine.

well it's only may cause Desync issues. but normaly both examples works well =)

+ it's "Random" bug so it's hard to reproduce it. it may depend on i-net connection etc. (i had this problem in my TcX) after i fix it i have no more desync reports.

[Silvenon]

I didn't know boolexpr codes generated with Condition()/Filter() don't have to return boolean.......weird.

EDIT: There is also one bug, but it's not very JASSish. When using an integer that's over 2147483647, it will change it's sign to -.
(I got this from PurplePoot)

[Toadcop]

There is also one bug, but it's not very JASSish. When using an integer that's over 2147483647, it will change it's sign to -.

overflow =) 31 bit integers. i.e. 2^31 (2147483648) (btw with the bits i am not really sure -_- xD)
but we know it =) keep values under 1 milliard =)

[cohadar]

Originally Posted by Silvenon

I didn't know boolexpr codes generated with Condition()/Filter() don't have to return boolean.......weird.

Boolexpr is a handle, when it is not handled properly it returns null.
On PC null will evaluate to false
On Mac it will ... do something...

[Vexorian]

hmmm toadcop I heard that these cause desyncs on mac users. So they may not be that random.

I think that not returning false might be undefined behavior hence it should create a lot of chaos including desyncing mac users and probably people with different languages in war3, different OS versions, etc, etc.

[botanic]

humm i found something odd with strings when a string is too long the end of it can become a hex value

[cohadar]

GetItemLevel() and GetItemType() do not work on bj_lastCreatedItem
they always return zero.

Ups...
It turns out that UnitAddItemToSlotById() does not set bj_lastCreatedItem, my bad.
Tnx Anitarf.

[Anitarf]

Works fine for me. Did you use CreateItemLoc?

[PurplePoot]

MultiboardDisplay(mb,false), where mb is an arbitrary multiboard, and not null, will close any open multiboard, regardless of whether it's mb or not.

Also, SelectUnit seems to fail to show the command card under very certain <unknown> situations, if the unit added to the selection is initially shown. (ie if you don't clear the selection before, and the tabbed selection is still on a different unit then it will still be able to show)

Not sure what those conditions are, otherwise I would've saved 2 hours of debugging and eventually a different method of doing what I needed to do -.-

[Dil999]

That SelectUnit is a known bug, I believe, most sheep tag games have had problems with it. I believe they fixed it by simply selecting deselecting and selecting, instead of just selecting.

[PurplePoot]

Nono, it only happens if you deselect right before or after.

(Deselect the old unit(s), before selecting the new unit)

[Ammorth]

A bug I ran across from before. Using MultiboardSetRowCount() or MultiboardSetColumnCount() to change a multiboard to a smaller board than it currently is results in errors with setting the text of the multiboard.

Trigger:

MB Glitch

Events

Time - Elapsed game time is 1.00 seconds

Conditions

Actions

-------- Shows a board with 10 columns and 5 rows, all with the letter a --------
Multiboard - Create a multiboard with 10 columns and 5 rows, titled Fun
Multiboard - Set the width for (Last created multiboard) item in column 0, row 0 to 3.00% of the total screen width
Multiboard - Set the text for (Last created multiboard) item in column 0, row 0 to a
Multiboard - Set the display style for (Last created multiboard) item in column 0, row 0 to Show text and Hide icons
Multiboard - Show (Last created multiboard)
Wait 3.00 seconds
-------- Changes the board size to 8 rows and 14 columns and changes the letter to b --------
Multiboard - Change the number of rows for (Last created multiboard) to 8
Multiboard - Change the number of columns for (Last created multiboard) to 14
Multiboard - Set the text for (Last created multiboard) item in column 0, row 0 to b
Multiboard - Set the display style for (Last created multiboard) item in column 0, row 0 to Show text and Hide icons
Wait 3.00 seconds
-------- Should change the number of rows to 6 and columns to 8 and then change the letter to c, but some of the text becomes bc and some stays as b. --------
Multiboard - Change the number of rows for (Last created multiboard) to 6
Multiboard - Change the number of columns for (Last created multiboard) to 8
Multiboard - Set the text for (Last created multiboard) item in column 0, row 0 to c
Multiboard - Set the display style for (Last created multiboard) item in column 0, row 0 to Show text and Hide icons