From b1dc47b4dfd8c4cf2b27ac0c93f0937acf9e33b1 Mon Sep 17 00:00:00 2001 From: Tigercat2000 Date: Mon, 1 Jun 2015 08:23:31 -0700 Subject: [PATCH] Fix incorrect uses of the shuffle proc. Even before the -tg- proc port, it didn't do inline modification of the list. --- code/game/gamemodes/wizard/raginmages.dm | 2 +- code/modules/admin/verbs/one_click_antag.dm | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/code/game/gamemodes/wizard/raginmages.dm b/code/game/gamemodes/wizard/raginmages.dm index f726b1c2a22..39563a0cbae 100644 --- a/code/game/gamemodes/wizard/raginmages.dm +++ b/code/game/gamemodes/wizard/raginmages.dm @@ -94,7 +94,7 @@ making_mage = 0 return else - shuffle(candidates) + candidates = shuffle(candidates) for(var/mob/i in candidates) if(!i || !i.client) continue //Dont bother removing them from the list since we only grab one wizard diff --git a/code/modules/admin/verbs/one_click_antag.dm b/code/modules/admin/verbs/one_click_antag.dm index 1cfd5174c17..f9940bec995 100644 --- a/code/modules/admin/verbs/one_click_antag.dm +++ b/code/modules/admin/verbs/one_click_antag.dm @@ -167,7 +167,7 @@ client/proc/one_click_antag() sleep(300) if(candidates.len) - shuffle(candidates) + candidates = shuffle(candidates) for(var/mob/i in candidates) if(!i || !i.client) continue //Dont bother removing them from the list since we only grab one wizard