diff options
| author | Haru <haru@dotalux.com> | 2017-03-12 02:43:40 +0100 |
|---|---|---|
| committer | Haru <haru@dotalux.com> | 2017-03-12 02:43:40 +0100 |
| commit | 5321ac0bc44e4429bb443f7e2a2c9fea0e047cd4 (patch) | |
| tree | 0faa9aaf9f9207236c63274258b4d56146ed3596 /src | |
| parent | 386a671bf48dd4e867b0c89df0fe42beb2fe39d7 (diff) | |
| download | hercules-5321ac0bc44e4429bb443f7e2a2c9fea0e047cd4.tar.gz hercules-5321ac0bc44e4429bb443f7e2a2c9fea0e047cd4.tar.bz2 hercules-5321ac0bc44e4429bb443f7e2a2c9fea0e047cd4.tar.xz hercules-5321ac0bc44e4429bb443f7e2a2c9fea0e047cd4.zip | |
Minor optimizations for the warpguild implementation
Follow-up to ebb77e29f343531b508f8c37b32826afbb5f240d
Signed-off-by: Haru <haru@dotalux.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/map/script.c | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/src/map/script.c b/src/map/script.c index 913208fff..6e7598f9b 100644 --- a/src/map/script.c +++ b/src/map/script.c @@ -6680,7 +6680,7 @@ BUILDIN(warpguild) struct guild* g; int type; int i; - int16 map_index = -1; + int16 map_id = -1; const char *str = script_getstr(st, 2); int x = script_getnum(st, 3); @@ -6688,11 +6688,11 @@ BUILDIN(warpguild) int gid = script_getnum(st, 5); if (script_hasdata(st, 6)) { - map_index = map->mapname2mapid(script_getstr(st, 6)); + map_id = map->mapname2mapid(script_getstr(st, 6)); } g = guild->search(gid); - if(g == NULL) + if (g == NULL) return true; type = (strcmp(str, "Random") == 0) ? 0 @@ -6707,27 +6707,28 @@ BUILDIN(warpguild) for (i = 0; i < MAX_GUILD; i++) { if (g->member[i].online && g->member[i].sd != NULL) { + struct map_session_data *pl_sd = g->member[i].sd; - if (map_index >= 0 && map_index != g->member[i].sd->bl.m) + if (map_id >= 0 && map_id != pl_sd->bl.m) continue; switch (type) { case 0: // Random - if (!map->list[g->member[i].sd->bl.m].flag.nowarp) - pc->randomwarp(g->member[i].sd, CLR_TELEPORT); + if (!map->list[pl_sd->bl.m].flag.nowarp) + pc->randomwarp(pl_sd, CLR_TELEPORT); break; case 1: // SavePointAll - if (!map->list[g->member[i].sd->bl.m].flag.noreturn) - pc->setpos(g->member[i].sd, g->member[i].sd->status.save_point.map, g->member[i].sd->status.save_point.x, g->member[i].sd->status.save_point.y, CLR_TELEPORT); + if (!map->list[pl_sd->bl.m].flag.noreturn) + pc->setpos(pl_sd, pl_sd->status.save_point.map, pl_sd->status.save_point.x, pl_sd->status.save_point.y, CLR_TELEPORT); break; case 2: // SavePoint - if (!map->list[g->member[i].sd->bl.m].flag.noreturn) - pc->setpos(g->member[i].sd, sd->status.save_point.map, sd->status.save_point.x, sd->status.save_point.y, CLR_TELEPORT); + if (!map->list[pl_sd->bl.m].flag.noreturn) + pc->setpos(pl_sd, sd->status.save_point.map, sd->status.save_point.x, sd->status.save_point.y, CLR_TELEPORT); break; case 3: // m,x,y - if (!map->list[g->member[i].sd->bl.m].flag.noreturn && !map->list[g->member[i].sd->bl.m].flag.nowarp) - pc->setpos(g->member[i].sd, script->mapindexname2id(st, str), x, y, CLR_TELEPORT); + if (!map->list[pl_sd->bl.m].flag.noreturn && !map->list[pl_sd->bl.m].flag.nowarp) + pc->setpos(pl_sd, script->mapindexname2id(st, str), x, y, CLR_TELEPORT); break; } } |