summaryrefslogtreecommitdiff
path: root/src/map
diff options
context:
space:
mode:
authorHaru <haru@dotalux.com>2015-11-14 19:43:38 +0100
committerHaru <haru@dotalux.com>2015-11-14 21:11:42 +0100
commit367c806bccbf242c2025356777b4cf1d194c481a (patch)
tree16e19f7f12fa35be0cf68dec6d9f49293931a21d /src/map
parent5869aadef89270cf38bee53f5ce3959142dde10a (diff)
downloadhercules-367c806bccbf242c2025356777b4cf1d194c481a.tar.gz
hercules-367c806bccbf242c2025356777b4cf1d194c481a.tar.bz2
hercules-367c806bccbf242c2025356777b4cf1d194c481a.tar.xz
hercules-367c806bccbf242c2025356777b4cf1d194c481a.zip
Standardized script-queue related commands
- The commands queueadd, queueremove, queueopt, queuedel and qiclear now all return true (success) or false (failure). - Taken/extended from AnnieRuru's PR #847. - The constants HQO_OnDeath, HQO_OnLogOut, HQO_OnMapChange were renamed to QUEUEOPT_DEATH, QUEUEOPT_LOGOUT, QUEUEOPT_MAPCHANGE respectively. - The name of the variable @QMapChangeTo$ used during the QUEUEOPT_MAPCHANGE event has been renamed to @Queue_Destination_Map$. Signed-off-by: Haru <haru@dotalux.com>
Diffstat (limited to 'src/map')
-rw-r--r--src/map/pc.c2
-rw-r--r--src/map/script.c41
2 files changed, 22 insertions, 21 deletions
diff --git a/src/map/pc.c b/src/map/pc.c
index d3b5498cc..1b7fcd610 100644
--- a/src/map/pc.c
+++ b/src/map/pc.c
@@ -5529,7 +5529,7 @@ int pc_setpos(struct map_session_data* sd, unsigned short map_index, int x, int
for( i = 0; i < sd->queues_count; i++ ) {
struct script_queue *queue = script->queue(sd->queues[i]);
if (queue && queue->event_mapchange[0] != '\0') {
- pc->setregstr(sd, script->add_str("@QMapChangeTo$"), map->list[m].name);
+ pc->setregstr(sd, script->add_str("@Queue_Destination_Map$"), map->list[m].name);
npc->event(sd, queue->event_mapchange, 0);
}
}
diff --git a/src/map/script.c b/src/map/script.c
index f6de7041d..768677d12 100644
--- a/src/map/script.c
+++ b/src/map/script.c
@@ -18918,7 +18918,8 @@ bool script_hqueue_add(int idx, int var)
/**
* Script command queueadd: Adds a new entry to the given queue.
*
- * Returns 1 if already in queue or in case of error, 0 otherwise.
+ * Returns 0 (false) if already in queue or in case of error, 1 (true)
+ * otherwise.
*
* @code{.herc}
* .@size = queuesize(.@queue_id);
@@ -18930,9 +18931,9 @@ BUILDIN(queueadd)
int var = script_getnum(st, 3);
if (script->queue_add(idx, var))
- script_pushint(st, 0);
- else
script_pushint(st, 1);
+ else
+ script_pushint(st, 0);
return true;
}
@@ -18978,7 +18979,7 @@ bool script_hqueue_remove(int idx, int var)
/**
* Script command queueremove: Removes an entry from a queue.
*
- * Returns 0 on success, 1 if the item wasn't in queue.
+ * Returns 1 (true) on success, 0 (false) if the item wasn't in queue.
*
* @code{.herc}
* queueremove(.@queue_id, .@value);
@@ -18990,9 +18991,9 @@ BUILDIN(queueremove)
int var = script_getnum(st, 3);
if (script->queue_remove(idx,var))
- script_pushint(st, 0);
- else
script_pushint(st, 1);
+ else
+ script_pushint(st, 0);
return true;
}
@@ -19002,15 +19003,15 @@ BUILDIN(queueremove)
*
* When the option value isn't provided, the option is removed.
*
- * Returns 0 on success, 1 on failure.
+ * Returns 1 (true) on success, 0 (false) on failure.
*
* The optionType is one of:
- * - HQO_OnDeath
- * - HQO_OnLogOut
- * - HQO_OnMapChange
+ * - QUEUEOPT_DEATH
+ * - QUEUEOPT_LOGOUT
+ * - QUEUEOPT_MAPCHANGE
*
- * When the OnMapChange event is triggered, it sets a temporary character
- * variable \c @QMapChangeTo with the destination map name.
+ * When the QUEUEOPT_MAPCHANGE event is triggered, it sets a temporary
+ * character variable \c @Queue_Destination_Map$ with the destination map name.
*
* @code{.herc}
* queueopt(.@queue_id, optionType, <optional val>);
@@ -19024,7 +19025,7 @@ BUILDIN(queueopt)
if (idx < 0 || idx >= VECTOR_LENGTH(script->hq) || VECTOR_INDEX(script->hq, idx).size == -1) {
ShowWarning("buildin_queueopt: unknown queue id %d\n",idx);
- script_pushint(st, 1);
+ script_pushint(st, 0);
return true;
}
@@ -19051,7 +19052,7 @@ BUILDIN(queueopt)
break;
default:
ShowWarning("buildin_queueopt: unsupported optionType %d\n",var);
- script_pushint(st, 1);
+ script_pushint(st, 0);
return true;
}
script_pushint(st, 1);
@@ -19098,7 +19099,7 @@ bool script_hqueue_del(int idx)
/**
* Script command queuedel: Deletes a queue.
*
- * Returns 0 on success, 1 if the queue doesn't exist.
+ * Returns 1 (true) on success, 0 (false) if the queue doesn't exist.
*
* @code{.herc}
* queuedel(.@queue_id);
@@ -19109,9 +19110,9 @@ BUILDIN(queuedel)
int idx = script_getnum(st, 2);
if (script->queue_del(idx))
- script_pushint(st, 0);
- else
script_pushint(st, 1);
+ else
+ script_pushint(st, 0);
return true;
}
@@ -19270,7 +19271,7 @@ BUILDIN(qicheck)
/**
* Script command qiclear: Destroys a queue iterator.
*
- * Returns 0 on success, 1 on failure.
+ * Returns true (1) on success, false (0) on failure.
*
* @code{.herc}
* qiclear(.@iter);
@@ -19282,12 +19283,12 @@ BUILDIN(qiclear)
if (idx < 0 || idx >= VECTOR_LENGTH(script->hqi)) {
ShowWarning("buildin_qiclear: unknown queue iterator id %d\n",idx);
- script_pushint(st, 1);
+ script_pushint(st, 0);
return true;
}
VECTOR_INDEX(script->hqi, idx).items = -1;
- script_pushint(st, 0);
+ script_pushint(st, 1);
return true;
}