It's better now so...approved.
Some things you should note though:
-Avoid actions that have BJ or "Swapped" in their name. I strongly recommend using JassCraft or JSP to lookup and see if a function is a native or not.
-Always do the nulling on the very end, just before endfunction. It's much easier to keep track of it that way...
-Use the "and" keyword instead of GetBooleanAnd. For example
Some things you should note though:
-Avoid actions that have BJ or "Swapped" in their name. I strongly recommend using JassCraft or JSP to lookup and see if a function is a native or not.
-Always do the nulling on the very end, just before endfunction. It's much easier to keep track of it that way...
-Use the "and" keyword instead of GetBooleanAnd. For example
Code:
function Trig_DOC takes nothing returns boolean
return GetBooleanAnd( Trig_D_O_C(), Trig_D_O_C_2() )
endfunction
Code:
function Trig_DOC takes nothing returns boolean
return Trig_D_O_C() and Trig_D_O_C_2()
endfunction