summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorHaru <haru@dotalux.com>2015-11-03 15:33:10 +0100
committerHaru <haru@dotalux.com>2015-11-12 00:01:32 +0100
commitd0bc38acd10a05bfdc8116b316a6b6eb62b9b580 (patch)
treeda7c06fb135e124c1deb8a5a0f7680718f37a6ec /src
parent57f09704aaa62295a9a967726a0dbabda40770c4 (diff)
downloadhercules-d0bc38acd10a05bfdc8116b316a6b6eb62b9b580.tar.gz
hercules-d0bc38acd10a05bfdc8116b316a6b6eb62b9b580.tar.bz2
hercules-d0bc38acd10a05bfdc8116b316a6b6eb62b9b580.tar.xz
hercules-d0bc38acd10a05bfdc8116b316a6b6eb62b9b580.zip
Split frequently repeated code to add/register new NPCs to its own function
Signed-off-by: Haru <haru@dotalux.com>
Diffstat (limited to 'src')
-rw-r--r--src/map/npc.c92
-rw-r--r--src/map/npc.h1
2 files changed, 35 insertions, 58 deletions
diff --git a/src/map/npc.c b/src/map/npc.c
index e92e3c447..ea73c2f81 100644
--- a/src/map/npc.c
+++ b/src/map/npc.c
@@ -2603,7 +2603,6 @@ struct npc_data* npc_add_warp(char* name, short from_mapid, short from_x, short
struct npc_data *nd;
nd = npc->create_npc(WARP, from_mapid, from_x, from_y, 0, battle_config.warp_point_debug ? WARP_DEBUG_CLASS : WARP_CLASS);
- map->addnpc(from_mapid, nd);
safestrncpy(nd->exname, name, ARRAYLENGTH(nd->exname));
if (npc->name2id(nd->exname) != NULL)
@@ -2621,13 +2620,8 @@ struct npc_data* npc_add_warp(char* name, short from_mapid, short from_x, short
nd->u.warp.y = to_y;
nd->u.warp.xs = xs;
nd->u.warp.ys = xs;
- npc->setcells(nd);
- map->addblock(&nd->bl);
- status->set_viewdata(&nd->bl, nd->class_);
- nd->ud = &npc->base_ud;
- if( map->list[nd->bl.m].users )
- clif->spawn(&nd->bl);
- strdb_put(npc->name_db, nd->exname, nd);
+
+ npc->add_to_location(nd);
return nd;
}
@@ -2664,7 +2658,6 @@ const char* npc_parse_warp(char* w1, char* w2, char* w3, char* w4, const char* s
}
nd = npc->create_npc(WARP, m, x, y, 0, battle_config.warp_point_debug ? WARP_DEBUG_CLASS : WARP_CLASS);
- map->addnpc(m, nd);
npc->parsename(nd, w3, start, buffer, filepath);
nd->path = npc->retainpathreference(filepath);
@@ -2674,13 +2667,8 @@ const char* npc_parse_warp(char* w1, char* w2, char* w3, char* w4, const char* s
nd->u.warp.xs = xs;
nd->u.warp.ys = ys;
npc_warp++;
- npc->setcells(nd);
- map->addblock(&nd->bl);
- status->set_viewdata(&nd->bl, nd->class_);
- nd->ud = &npc->base_ud;
- if( map->list[nd->bl.m].users )
- clif->spawn(&nd->bl);
- strdb_put(npc->name_db, nd->exname, nd);
+
+ npc->add_to_location(nd);
return strchr(start,'\n');// continue
}
@@ -2809,17 +2797,7 @@ const char* npc_parse_shop(char* w1, char* w2, char* w3, char* w4, const char* s
nd->path = npc->retainpathreference(filepath);
++npc_shop;
- if( m >= 0 ) {// normal shop npc
- map->addnpc(m,nd);
- map->addblock(&nd->bl);
- status->set_viewdata(&nd->bl, nd->class_);
- nd->ud = &npc->base_ud;
- if( map->list[nd->bl.m].users )
- clif->spawn(&nd->bl);
- } else {// 'floating' shop
- map->addiddb(&nd->bl);
- }
- strdb_put(npc->name_db, nd->exname, nd);
+ npc->add_to_location(nd);
return strchr(start,'\n');// continue
}
@@ -2997,22 +2975,7 @@ const char* npc_parse_script(char* w1, char* w2, char* w3, char* w4, const char*
nd->u.scr.trader = true;
nd->u.scr.shop = NULL;
++npc_script;
-
- if( m >= 0 ) {
- map->addnpc(m, nd);
- nd->ud = &npc->base_ud;
- npc->setcells(nd);
- map->addblock(&nd->bl);
- if( nd->class_ >= 0 ) {
- status->set_viewdata(&nd->bl, nd->class_);
- if( map->list[nd->bl.m].users )
- clif->spawn(&nd->bl);
- }
- } else {
- // we skip map->addnpc, but still add it to the list of ID's
- map->addiddb(&nd->bl);
- }
- strdb_put(npc->name_db, nd->exname, nd);
+ npc->add_to_location(nd);
//-----------------------------------------
// Loop through labels to export them as necessary
@@ -3045,6 +3008,32 @@ const char* npc_parse_script(char* w1, char* w2, char* w3, char* w4, const char*
}
/**
+ * Registers the NPC and adds it to its location (on map or floating).
+ *
+ * @param nd The NPC to register.
+ */
+void npc_add_to_location(struct npc_data *nd)
+{
+ nullpo_retv(nd);
+
+ if (nd->bl.m > 0) {
+ map->addnpc(nd->bl.m, nd);
+ npc->setcells(nd);
+ map->addblock(&nd->bl);
+ nd->ud = &npc->base_ud;
+ if (nd->class_ >= 0) {
+ status->set_viewdata(&nd->bl, nd->class_);
+ if( map->list[nd->bl.m].users )
+ clif->spawn(&nd->bl);
+ }
+ } else {
+ // we skip map->addnpc, but still add it to the list of ID's
+ map->addiddb(&nd->bl);
+ }
+ strdb_put(npc->name_db, nd->exname, nd);
+}
+
+/**
* Duplicates a warp, shop, cashshop or script.
*
* @param nd An already initialized NPC data. Expects bl->m, bl->x, bl->y,
@@ -3094,21 +3083,7 @@ bool npc_duplicate_sub(struct npc_data *nd, const struct npc_data *snd, int xs,
}
//Add the npc to its location
- if (nd->bl.m >= 0) {
- map->addnpc(nd->bl.m, nd);
- nd->ud = &npc->base_ud;
- npc->setcells(nd);
- map->addblock(&nd->bl);
- if (nd->class_ >= 0) {
- status->set_viewdata(&nd->bl, nd->class_);
- if (map->list[nd->bl.m].users)
- clif->spawn(&nd->bl);
- }
- } else {
- // we skip map->addnpc, but still add it to the list of ID's
- map->addiddb(&nd->bl);
- }
- strdb_put(npc->name_db, nd->exname, nd);
+ npc->add_to_location(nd);
if (nd->subtype != SCRIPT)
return true;
@@ -4780,6 +4755,7 @@ void npc_defaults(void) {
npc->convertlabel_db = npc_convertlabel_db;
npc->skip_script = npc_skip_script;
npc->parse_script = npc_parse_script;
+ npc->add_to_location = npc_add_to_location;
npc->duplicate_sub = npc_duplicate_sub;
npc->parse_duplicate = npc_parse_duplicate;
npc->duplicate4instance = npc_duplicate4instance;
diff --git a/src/map/npc.h b/src/map/npc.h
index bec17fd8d..57000f1d6 100644
--- a/src/map/npc.h
+++ b/src/map/npc.h
@@ -239,6 +239,7 @@ struct npc_interface {
void (*convertlabel_db) (struct npc_label_list *label_list, const char *filepath);
const char* (*skip_script) (const char *start, const char *buffer, const char *filepath, int *retval);
const char* (*parse_script) (char *w1, char *w2, char *w3, char *w4, const char *start, const char *buffer, const char *filepath, int options, int *retval);
+ void (*add_to_location) (struct npc_data *nd);
bool (*duplicate_sub) (struct npc_data *nd, const struct npc_data *snd, int xs, int ys, int options);
const char* (*parse_duplicate) (char* w1, char* w2, char* w3, char* w4, const char* start, const char* buffer, const char* filepath, int options, int *retval);
int (*duplicate4instance) (struct npc_data *snd, int16 m);