Small Code Snippets

When stunning a unit for x seconds, I suggest using Aerial Shackles with infinite duration and then adding an expiration timer, or any timer after casting the spell. As far as I know, aerial shackles cannot be dispelled, but the only way to cancel it is to stun/silence the caster which won't work with a locust unit. So that way you get the same effect like stormbolt, with a duration of x.xx seconds if you like, and you only need one spell level :)
(I have tried it this way in my map "BEER BEER", and so far it worked flawless)

//Edit: Just saw that purplepoot posted something similiar already. Well, I suggest aerial shackles as dummy!

function PreloadSoundPath takes string path returns nothing
    local sound snd = CreateSound(path, false, false, false, 10, 10, "")
    call SetSoundVolume(snd, 0)
    call StartSound(snd)
    call KillSoundWhenDone(snd)
    set snd = null
endfunction

Just a retarded function that preloads a sound in the given path. If someone doesn't know, first time you play a sound nothing happens, because the sound is loaded first, so this function does that loading part.

Maybe we should do something about these snippets Pooty? For example do something about this:

Originally Posted by PurplePoot

Functions

(none yet)

function Trig_Single_Player_Defeat_Actions takes nothing returns nothing
    //do whatever//
endfunction

//===========================================================================
function Trig_Single_Player_Defeat_Conditions takes nothing returns boolean
    return bj_isSinglePlayer == true
endfunction

function InitTrig_Single_Player_Defeat takes nothing returns nothing
    set gg_trg_Single_Player_Defeat = CreateTrigger( )
    call TriggerRegisterTimerEventSingle( gg_trg_Single_Player_Defeat, 15.00 )
    call TriggerAddCondition( gg_trg_Single_Player_Defeat, Condition( function Trig_Single_Player_Defeat_Conditions ) )
    call TriggerAddAction( gg_trg_Single_Player_Defeat, function Trig_Single_Player_Defeat_Actions )
endfunction

That will detect if the game is single player at 15 seconds.

Uhmmm......what?? This is just GUI with a line of custom script converted to JASS.

No it isn't..........

Ah yes, adding some to the listing...

No it isn't..........

No offense, but yes it is.

I have a weird set b = null in my GetClosestUnit function, you can remove that please (since there's no "b" variable). I believe I wanted to null the taken boolexpr, but taken values don't need to be nulled.

function IsDestTree takes destructable dest returns boolean
    local integer d = GetDestructableTypeId(dest)
    return d == 'ATtr' or d == 'BTtw' or d == 'KTtw' or d == 'YTft' or d == 'JTct' or d == 'YTst' or d == 'YTct' or d == 'YTwt' or d == 'JTwt' or d == 'JTwt' or d == 'FTtw' or d == 'CTtr' or d == 'ITtw' or d == 'NTtw' or d == 'OTtw' or d == 'ZTtw' or d == 'WTst' or d == 'LTlt' or d == 'GTsh' or d == 'Xtlt' or d == 'WTtw' or d == 'Attc' or d == 'BTtc' or d == 'CTtc' or d == 'ITtc' or d == 'NTtc' or d == 'ZTtc'
endfunction

Just a function that checks if the given destructable is a tree.

Use 'or'? T'would be better than using elseif, since and/or are short circuit in JASS, apparently.

Ok, I wanted to do that too, but the stupid editor didn't allow such a long line.

*Edited*

Originally Posted by Silvenon

function IsDestTree takes destructable dest returns boolean
    local integer d = GetDestructableTypeId(dest)
    return d == 'ATtr' or d == 'BTtw' or d == 'KTtw' or d == 'YTft' or d == 'JTct' or d == 'YTst' or d == 'YTct' or d == 'YTwt' or d == 'JTwt' or d == 'JTwt' or d == 'FTtw' or d == 'CTtr' or d == 'ITtw' or d == 'NTtw' or d == 'OTtw' or d == 'ZTtw' or d == 'WTst' or d == 'LTlt' or d == 'GTsh' or d == 'Xtlt' or d == 'WTtw' or d == 'Attc' or d == 'BTtc' or d == 'CTtc' or d == 'ITtc' or d == 'NTtc' or d == 'ZTtc'
endfunction

Just a function that checks if the given destructable is a tree.

That looks like you just too out a piece of my code and called it your own code..

I never said I made it, but yes, that was a bitchy thing to do, sorry.

This is what I said when I was using the function in the Coding an efficient knockback tutorial if it makes you feel better:

Originally Posted by Silvenon

This is a function I wrote (actually copied from Hero12341234's Tree Revival tutorial, hehe, sorry Hero....)

You can see for yourself if you don't believe me. I just modified it a bit.

Now I'm going in my room to cry........give me neg rep Purple! (not a joke, I deserve it)

Again, sorry Hero, I forgot to mention it was yours (like I did in the tutorial).

Hello friends. Here is my gift of the month to you:
EDIT: moved here

Not here anymore

I have tested all of these except for the __InGroup versions, but I think its safe to assume that all of them work because of their similarity.

Im also quite sure (lets say 100%) that this is faster than the FirstOfGroup(...) looping alternative.

117 lines of code... I guess this might be better off with a thread of its own but meh. If thats the case Im sure you'll tell me.

That looks awesome, but:

To keep out a potential deluge of threads with very small functions, I've created this thread for you to post relatively short functions you wish to share.

I would rather post that in the JASS functions, not small code snippets.

Crap, this totally beats up my GetClosestUnit function, thanks Disciple <sarcasm> :P

This is really small, and probably already made, but it's worth a shot.

function AddWeatherEffectForPlayer takes rect where, integer effectID, player whichplayer returns nothing
    if GetLocalPlayer() == whichplayer then
        call AddWeatherEffect(where, effectID)
    endif
endfunction