ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/deliantra/server/server/spell_util.C
(Generate patch)

Comparing deliantra/server/server/spell_util.C (file contents):
Revision 1.17 by root, Sat Sep 16 22:24:13 2006 UTC vs.
Revision 1.81 by root, Mon Sep 29 10:20:49 2008 UTC

1/* 1/*
2 CrossFire, A Multiplayer game for X-windows 2 * This file is part of Deliantra, the Roguelike Realtime MMORPG.
3 3 *
4 * Copyright (©) 2005,2006,2007,2008 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
4 Copyright (C) 2001 Mark Wedel & Crossfire Development Team 5 * Copyright (©) 2001,2007 Mark Wedel & Crossfire Development Team
5 Copyright (C) 1992 Frank Tore Johansen 6 * Copyright (©) 1992,2007 Frank Tore Johansen
6 7 *
7 This program is free software; you can redistribute it and/or modify 8 * Deliantra is free software: you can redistribute it and/or modify
8 it under the terms of the GNU General Public License as published by 9 * it under the terms of the GNU General Public License as published by
9 the Free Software Foundation; either version 2 of the License, or 10 * the Free Software Foundation, either version 3 of the License, or
10 (at your option) any later version. 11 * (at your option) any later version.
11 12 *
12 This program is distributed in the hope that it will be useful, 13 * This program is distributed in the hope that it will be useful,
13 but WITHOUT ANY WARRANTY; without even the implied warranty of 14 * but WITHOUT ANY WARRANTY; without even the implied warranty of
14 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 15 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15 GNU General Public License for more details. 16 * GNU General Public License for more details.
16 17 *
17 You should have received a copy of the GNU General Public License 18 * You should have received a copy of the GNU General Public License
18 along with this program; if not, write to the Free Software 19 * along with this program. If not, see <http://www.gnu.org/licenses/>.
19 Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. 20 *
20 21 * The authors can be reached via e-mail to <support@deliantra.net>
21 The authors can be reached via e-mail at <crossfire@schmorp.de>
22*/ 22 */
23
24 23
25#include <global.h> 24#include <global.h>
26#include <spells.h> 25#include <spells.h>
27#include <object.h> 26#include <object.h>
28#include <errno.h> 27#include <errno.h>
29#ifndef __CEXTRACT__
30# include <sproto.h> 28#include <sproto.h>
31#endif
32#include <sounds.h> 29#include <sounds.h>
33 30
34extern char *spell_mapping[]; 31extern char *spell_mapping[];
35 32
36/* This returns a random spell from 'ob'. If skill is set, then 33/* This returns a random spell from 'ob'. If skill is set, then
39 */ 36 */
40object * 37object *
41find_random_spell_in_ob (object *ob, const char *skill) 38find_random_spell_in_ob (object *ob, const char *skill)
42{ 39{
43 int k = 0, s; 40 int k = 0, s;
44 object *tmp;
45 41
46 for (tmp = ob->inv; tmp; tmp = tmp->below) 42 for (object *tmp = ob->inv; tmp; tmp = tmp->below)
47 if (tmp->type == SPELL && (!skill || tmp->skill == skill)) 43 if (tmp->type == SPELL && (!skill || tmp->skill == skill))
48 k++; 44 k++;
49 45
50 /* No spells, no need to progess further */ 46 /* No spells, no need to progess further */
51 if (!k) 47 if (!k)
52 return NULL; 48 return NULL;
53 49
54 s = RANDOM () % k; 50 s = RANDOM () % k;
55 51
56 for (tmp = ob->inv; tmp; tmp = tmp->below) 52 for (object *tmp = ob->inv; tmp; tmp = tmp->below)
57 if (tmp->type == SPELL && (!skill || tmp->skill == skill)) 53 if (tmp->type == SPELL && (!skill || tmp->skill == skill))
58 {
59 if (!s) 54 if (!s)
60 return tmp; 55 return tmp;
61 else 56 else
62 s--; 57 s--;
63 } 58
64 /* Should never get here, but just in case */ 59 /* Should never get here, but just in case */
65 return NULL; 60 return 0;
66} 61}
67 62
68/* Relatively simple function that gets used a lot. 63/* Relatively simple function that gets used a lot.
69 * Basically, it sets up the skill pointer for the spell being 64 * Basically, it sets up the skill pointer for the spell being
70 * cast. If op is really casting the spell, then the skill 65 * cast. If op is really casting the spell, then the skill
80 dest->skill = spob->skill; 75 dest->skill = spob->skill;
81 else 76 else
82 dest->skill = caster->skill; 77 dest->skill = caster->skill;
83} 78}
84 79
85/* init_spells: This should really be called check_spells, as that
86 * is what it does. It goes through the spells looking for any
87 * obvious errors. This was most useful in debugging when re-doing
88 * all the spells to catch simple errors. To use it all the time
89 * will result in it spitting out messages that aren't really errors.
90 */
91void
92init_spells (void)
93{
94#ifdef SPELL_DEBUG
95 static int init_spells_done = 0;
96 int i;
97 archetype *at;
98
99 if (init_spells_done)
100 return;
101 LOG (llevDebug, "Checking spells...\n");
102
103 for (at = first_archetype; at; at = at->next)
104 {
105 if (at->clone.type == SPELL)
106 {
107 if (at->clone.skill)
108 {
109 for (i = 1; i < NUM_SKILLS; i++)
110 if (!strcmp (skill_names[i], at->clone.skill))
111 break;
112 if (i == NUM_SKILLS)
113 {
114 LOG (llevError, "Spell %s has improper associated skill %s\n", at->name, at->clone.skill);
115 }
116 }
117 /* other_arch is already checked for in the loader */
118 }
119 }
120
121 i = 0;
122 while (spell_mapping[i])
123 {
124 if (!archetype::find (spell_mapping[i]))
125 {
126 LOG (llevError, "Unable to find spell mapping %s (%i)\n", spell_mapping[i], i);
127 }
128 i++;
129 }
130 LOG (llevDebug, "Checking spells completed.\n");
131#endif
132}
133
134/* Dumps all the spells - now also dumps skill associated with the spell.
135 * not sure what this would be used for, as the data seems pretty
136 * minimal, but easy enough to keep around.
137 */
138void
139dump_spells (void)
140{
141 archetype *at;
142
143 for (at = first_archetype; at; at = at->next)
144 {
145 if (at->clone.type == SPELL)
146 {
147 fprintf (stderr, "%s:%s:%s:%s:%d\n", &at->clone.name,
148 &at->name, at->clone.other_arch ? &at->clone.other_arch->name : "<nil>", &at->clone.skill, at->clone.level);
149 }
150 }
151}
152
153/* pretty basic function - basically just takes 80/* pretty basic function - basically just takes
154 * an object, sets the x,y, and calls insert_ob_in_map 81 * an object, sets the x,y, and calls insert_ob_in_map
155 */ 82 */
156
157void 83void
158spell_effect (object *spob, int x, int y, maptile *map, object *originator) 84spell_effect (object *spob, int x, int y, maptile *map, object *originator)
159{ 85{
160
161 if (spob->other_arch != NULL) 86 if (spob->other_arch)
162 { 87 map->insert (arch_to_object (spob->other_arch), x, y, originator);
163 object *effect = arch_to_object (spob->other_arch); 88}
164 89
165 effect->x = x; 90static int
166 effect->y = y; 91attuned_bonus (object *caster, object *spell, int level)
167 92{
168 insert_ob_in_map (effect, map, originator, 0); 93 // compute the attuned/repelled bonus
169 } 94 // attuned only increases up to 2 times the original level (i.e. bonus <= level) */
95 // repell has no such quarrels
96 return (caster->path_attuned & spell->path_attuned ? min (level, +ATTUNE_REPELL) : 0)
97 + (caster->path_repelled & spell->path_attuned ? -ATTUNE_REPELL : 0);
170} 98}
171 99
172/* 100/*
173 * This function takes a caster and spell and presents the 101 * This function takes a caster and spell and presents the
174 * effective level the caster needs to be to cast the spell. 102 * effective level the caster needs to be to cast the spell.
175 * basically, it just adjusts the spell->level with attuned/repelled 103 * basically, it just adjusts the spell->level with attuned/repelled
176 * spellpaths. Was called path_level_mod 104 * spellpaths. Was called path_level_mod.
177 * 105 *
178 * caster is person casting hte spell. 106 * caster is person casting the spell.
179 * spell is the spell object. 107 * spell is the spell object.
180 * Returns modified level. 108 * Returns modified level.
181 */ 109 */
182int 110int
183min_casting_level (object *caster, object *spell) 111min_casting_level (object *caster, object *spell)
184{ 112{
185 int new_level;
186
187 if (caster->path_denied & spell->path_attuned) 113 if (caster->path_denied & spell->path_attuned)
188 {
189 /* This case is not a bug, just the fact that this function is
190 * usually called BEFORE checking for path_deny. -AV
191 */
192#if 0
193 LOG (llevError, "BUG: path_level_mod (arch %s, name %s): casting denied " "spell\n", caster->arch->name, caster->name);
194#endif
195 return 1; 114 return 500;
196 }
197 new_level = spell->level
198 + ((caster->path_repelled & spell->path_attuned) ? +2 : 0) + ((caster->path_attuned & spell->path_attuned) ? -2 : 0);
199 return (new_level < 1) ? 1 : new_level;
200}
201 115
116 return max (1, spell->level + attuned_bonus (caster, spell, spell->level));
117}
202 118
203/* This function returns the effective level the spell 119/* This function returns the effective level the spell
204 * is being cast at. 120 * is being cast at.
205 * Note that I changed the repelled/attuned bonus to 2 from 5.
206 * This is because the new code compares casting_level against
207 * min_caster_level, so the difference is effectively 4
208 */ 121 */
209
210int 122int
211caster_level (object *caster, object *spell) 123casting_level (object *caster, object *spell)
212{ 124{
213 int level = caster->level; 125 int level = caster->level;
214
215 /* If this is a player, try to find the matching skill */
216 if (caster->type == PLAYER && spell->skill)
217 {
218 int i;
219
220 for (i = 0; i < NUM_SKILLS; i++)
221 if (caster->contr->last_skill_ob[i] && caster->contr->last_skill_ob[i]->skill == spell->skill)
222 {
223 level = caster->contr->last_skill_ob[i]->level;
224 break;
225 }
226 }
227 126
228 /* if a rod is fired by a player, take the use_magic_item skill in consideration. */ 127 /* if a rod is fired by a player, take the use_magic_item skill in consideration. */
229 if (caster->type == ROD && caster->env && caster->env->type == PLAYER) 128 if (caster->type == ROD && caster->env && caster->env->type == PLAYER)
230 { 129 {
231 object *skill = find_skill_by_number (caster->env, SK_USE_MAGIC_ITEM); 130 object *skill = find_skill_by_number (caster->env, SK_USE_MAGIC_ITEM);
232 int sk_level = skill ? skill->level : 1; 131 int sk_level = skill ? skill->level : 1;
233 132
234 level = MIN (level, sk_level + level / 10 + 1); 133 level = min (level, sk_level + level / 10 + 1);
235 } 134 }
135 else if (caster->type == PLAYER) /* If this is a player, try to find the matching skill */
136 if (object *skill = caster->contr->find_skill (spell->skill))
137 level = skill->level;
236 138
237 /* Got valid caster level. Now adjust for attunement */ 139 int bonus = attuned_bonus (caster, spell, level);
238 level += ((caster->path_repelled & spell->path_attuned) ? -2 : 0) + ((caster->path_attuned & spell->path_attuned) ? 2 : 0); 140
141 level += bonus;
142
143 // now scale the effective level from the startinglevel..100 range to 1..100
144 if (level < 100)
145 level = lerp (level, (int)spell->level, 100, 1, 100);
239 146
240 /* Always make this at least 1. If this is zero, we get divide by zero 147 /* Always make this at least 1. If this is zero, we get divide by zero
241 * errors in various places. 148 * errors in various places.
242 */ 149 */
243 if (level < 1)
244 level = 1;
245 return level; 150 return max (level, 1);
246} 151}
247 152
248/* The following function scales the spellpoint cost of 153/* The following function scales the spellpoint cost of
249 * a spell by it's increased effectiveness. Some of the 154 * a spell by it's increased effectiveness. Some of the
250 * lower level spells become incredibly vicious at high 155 * lower level spells become incredibly vicious at high
254 * caster is what is casting the spell, can be op. 159 * caster is what is casting the spell, can be op.
255 * spell is the spell object. 160 * spell is the spell object.
256 * Note that it is now possible for a spell to cost both grace and 161 * Note that it is now possible for a spell to cost both grace and
257 * mana. In that case, we return which ever value is higher. 162 * mana. In that case, we return which ever value is higher.
258 */ 163 */
259
260sint16 164sint16
261SP_level_spellpoint_cost (object *caster, object *spell, int flags) 165SP_level_spellpoint_cost (object *caster, object *spell, int flags)
262{ 166{
263 int sp, grace, level = caster_level (caster, spell); 167 int sp, grace, level = casting_level (caster, spell);
264 168
265 if (settings.spellpoint_level_depend == TRUE) 169 if (settings.spellpoint_level_depend == TRUE)
266 { 170 {
267 if (spell->stats.sp && spell->stats.maxsp) 171 if (spell->stats.sp && spell->stats.maxsp)
268 { 172 {
289 else 193 else
290 { 194 {
291 sp = (int) (spell->stats.sp * PATH_SP_MULT (caster, spell)); 195 sp = (int) (spell->stats.sp * PATH_SP_MULT (caster, spell));
292 if (spell->stats.sp && !sp) 196 if (spell->stats.sp && !sp)
293 sp = 1; 197 sp = 1;
198
294 grace = (int) (spell->stats.grace * PATH_SP_MULT (caster, spell)); 199 grace = (int) (spell->stats.grace * PATH_SP_MULT (caster, spell));
295 if (spell->stats.grace && !grace) 200 if (spell->stats.grace && !grace)
296 grace = 1; 201 grace = 1;
297 } 202 }
203
298 if (flags == SPELL_HIGHEST) 204 if (flags == SPELL_HIGHEST)
299 return MAX (sp, grace); 205 return MAX (sp, grace);
300 else if (flags == SPELL_GRACE) 206 else if (flags == SPELL_GRACE)
301 return grace; 207 return grace;
302 else if (flags == SPELL_MANA) 208 else if (flags == SPELL_MANA)
306 LOG (llevError, "SP_level_spellpoint_cost: Unknown flags passed: %d\n", flags); 212 LOG (llevError, "SP_level_spellpoint_cost: Unknown flags passed: %d\n", flags);
307 return 0; 213 return 0;
308 } 214 }
309} 215}
310 216
217/*
218 * Return the effective casting level of the spell.
219 * To make spells independent of their starting level, this function
220 * scales the range spellstartlevel .. 100 into the range 1..100
221 */
222static int
223SP_casting_level (object *caster, object *spell)
224{
225 return casting_level (caster, spell);
226}
311 227
312/* SP_level_dam_adjust: Returns adjusted damage based on the caster. 228/* SP_level_dam_adjust: Returns adjusted damage based on the caster.
313 * spob is the spell we are adjusting. 229 * spob is the spell we are adjusting.
314 */ 230 */
315int 231int
316SP_level_dam_adjust (object *caster, object *spob) 232SP_level_dam_adjust (object *caster, object *spob)
317{ 233{
318 int level = caster_level (caster, spob);
319 int adj = level - min_casting_level (caster, spob);
320
321 if (adj < 0)
322 adj = 0;
323 if (spob->dam_modifier) 234 if (!spob->dam_modifier)
324 adj /= spob->dam_modifier;
325 else
326 adj = 0;
327 return adj; 235 return 0;
236
237 return SP_casting_level (caster, spob) / spob->dam_modifier;
328} 238}
329 239
330/* Adjust the strength of the spell based on level. 240/* Adjust the strength of the spell based on level.
331 * This is basically the same as SP_level_dam_adjust above, 241 * This is basically the same as SP_level_dam_adjust above,
332 * but instead looks at the level_modifier value. 242 * but instead looks at the level_modifier value.
333 */ 243 */
334int 244int
335SP_level_duration_adjust (object *caster, object *spob) 245SP_level_duration_adjust (object *caster, object *spob)
336{ 246{
337 int level = caster_level (caster, spob);
338 int adj = level - min_casting_level (caster, spob);
339
340 if (adj < 0)
341 adj = 0;
342 if (spob->duration_modifier) 247 if (!spob->duration_modifier)
343 adj /= spob->duration_modifier;
344 else
345 adj = 0;
346
347 return adj; 248 return 0;
249
250 return SP_casting_level (caster, spob) / spob->duration_modifier;
348} 251}
349 252
350/* Adjust the strength of the spell based on level. 253/* Adjust the strength of the spell based on level.
351 * This is basically the same as SP_level_dam_adjust above, 254 * This is basically the same as SP_level_dam_adjust above,
352 * but instead looks at the level_modifier value. 255 * but instead looks at the level_modifier value.
353 */ 256 */
354int 257int
355SP_level_range_adjust (object *caster, object *spob) 258SP_level_range_adjust (object *caster, object *spob)
356{ 259{
357 int level = caster_level (caster, spob);
358 int adj = level - min_casting_level (caster, spob);
359
360 if (adj < 0)
361 adj = 0;
362 if (spob->range_modifier) 260 if (!spob->range_modifier)
363 adj /= spob->range_modifier;
364 else
365 adj = 0;
366
367 return adj; 261 return 0;
262
263 return SP_casting_level (caster, spob) / spob->range_modifier;
368} 264}
369 265
370/* Checks to see if player knows the spell. If the name is the same 266/* Checks to see if player knows the spell. If the name is the same
371 * as an existing spell, we presume they know it. 267 * as an existing spell, we presume they know it.
372 * returns 1 if they know the spell, 0 if they don't. 268 * returns 1 if they know the spell, 0 if they don't.
373 */ 269 */
374object * 270object *
375check_spell_known (object *op, const char *name) 271check_spell_known (object *op, const char *name)
376{ 272{
377 object *spop; 273 object *spop;
274 shstr_cmp name_ (name);
378 275
379 for (spop = op->inv; spop; spop = spop->below) 276 for (spop = op->inv; spop; spop = spop->below)
380 if (spop->type == SPELL && !strcmp (spop->name, name)) 277 if (spop->type == SPELL && spop->name == name)
381 return spop; 278 return spop;
382 279
383 return NULL; 280 return 0;
384} 281}
385
386 282
387/* 283/*
388 * Look at object 'op' and see if they know the spell 284 * Look at object 'op' and see if they know the spell
389 * spname. This is pretty close to check_spell_known 285 * spname. This is pretty close to check_spell_known
390 * above, but it uses a looser matching mechanism. 286 * above, but it uses a looser matching mechanism.
391 * returns the matching spell object, or NULL. 287 * returns the matching spell object, or NULL.
392 * If we match multiple spells but don't get an 288 * If we match multiple spells but don't get an
393 * exact match, we also return NULL. 289 * exact match, we also return NULL.
394 */ 290 */
395
396object * 291object *
397lookup_spell_by_name (object *op, const char *spname) 292lookup_spell_by_name (object *op, const char *spname)
398{ 293{
399 object *spob1 = NULL, *spob2 = NULL, *spob; 294 object *spob1 = NULL, *spob2 = NULL, *spob;
400 int nummatch = 0; 295 int nummatch = 0;
446 * eg, updated for tiled maps. 341 * eg, updated for tiled maps.
447 */ 342 */
448int 343int
449reflwall (maptile *m, int x, int y, object *sp_op) 344reflwall (maptile *m, int x, int y, object *sp_op)
450{ 345{
451 object *op;
452
453 if (OUT_OF_REAL_MAP (m, x, y)) 346 if (OUT_OF_REAL_MAP (m, x, y))
454 return 0; 347 return 0;
455 for (op = get_map_ob (m, x, y); op != NULL; op = op->above) 348
456 if (QUERY_FLAG (op, FLAG_REFL_SPELL) && (!QUERY_FLAG (op, FLAG_ALIVE) || 349 for (object *op = GET_MAP_OB (m, x, y); op; op = op->above)
457 sp_op->type == LIGHTNING || (rndm (0, 99)) < 90 - (sp_op->level / 10))) 350 if (QUERY_FLAG (op, FLAG_REFL_SPELL)
351 && (!QUERY_FLAG (op, FLAG_ALIVE)
352 || (rndm (0, 99)) < 90 - (sp_op->level / 10)))
458 return 1; 353 return 1;
459 354
460 return 0; 355 return 0;
461} 356}
462 357
479 { 374 {
480 new_draw_info (NDI_UNIQUE, 0, op, "Something is in the way."); 375 new_draw_info (NDI_UNIQUE, 0, op, "Something is in the way.");
481 new_draw_info (NDI_UNIQUE, 0, op, "You cast it at your feet."); 376 new_draw_info (NDI_UNIQUE, 0, op, "You cast it at your feet.");
482 dir = 0; 377 dir = 0;
483 } 378 }
484 new_op->x = op->x + freearr_x[dir]; 379
485 new_op->y = op->y + freearr_y[dir]; 380 SET_FLAG (new_op, FLAG_IDENTIFIED);
486 if (dir == 0) 381 op->map->insert (new_op,
487 insert_ob_in_map (new_op, op->map, op, INS_BELOW_ORIGINATOR); 382 op->x + freearr_x[dir], op->y + freearr_y[dir],
488 else 383 op,
489 insert_ob_in_map (new_op, op->map, op, 0); 384 dir ? 0 : INS_BELOW_ORIGINATOR);
385
490 return dir; 386 return dir;
491} 387}
492 388
493/* Returns true if it is ok to put spell *op on the space/may provided. 389/* Returns true if it is ok to put spell *op on the space/may provided.
494 * immune_stop is basically the attacktype of the spell (why 390 * immune_stop is basically the attacktype of the spell (why
496 * has the AT_MAGIC bit set, and there is a counterwall 392 * has the AT_MAGIC bit set, and there is a counterwall
497 * on the space, the object doesn't get placed. if immune stop 393 * on the space, the object doesn't get placed. if immune stop
498 * does not have AT_MAGIC, then counterwalls do not effect the spell. 394 * does not have AT_MAGIC, then counterwalls do not effect the spell.
499 * 395 *
500 */ 396 */
501
502int 397int
503ok_to_put_more (maptile *m, sint16 x, sint16 y, object *op, int immune_stop) 398ok_to_put_more (maptile *m, sint16 x, sint16 y, object *op, int immune_stop)
504{ 399{
505 object *tmp; 400 if (!xy_normalise (m, x, y))
506 int mflags;
507 maptile *mp;
508
509 mp = m;
510 mflags = get_map_flags (m, &mp, x, y, &x, &y);
511
512 if (mflags & P_OUT_OF_MAP)
513 return 0; 401 return 0;
514 402
515 if (OB_TYPE_MOVE_BLOCK (op, GET_MAP_MOVE_BLOCK (mp, x, y))) 403 mapspace &ms = m->at (x, y);
404 ms.update ();
405
406 if (OB_TYPE_MOVE_BLOCK (op, ms.move_block))
516 return 0; 407 return 0;
517 408
518 for (tmp = get_map_ob (mp, x, y); tmp != NULL; tmp = tmp->above) 409 for (object *tmp = ms.bot; tmp; tmp = tmp->above)
519 { 410 {
520 /* If there is a counterspell on the space, and this 411 /* If there is a counterspell on the space, and this
521 * object is using magic, don't progress. I believe we could 412 * object is using magic, don't progress. I believe we could
522 * leave this out and let in progress, and other areas of the code 413 * leave this out and let in progress, and other areas of the code
523 * will then remove it, but that would seem to to use more 414 * will then remove it, but that would seem to to use more
524 * resources, and may not work as well if a player is standing 415 * resources, and may not work as well if a player is standing
525 * on top of a counterwall spell (may hit the player before being 416 * on top of a counterwall spell (may hit the player before being
526 * removed.) On the other hand, it may be more dramatic for the 417 * removed.) On the other hand, it may be more dramatic for the
527 * spell to actually hit the counterwall and be sucked up. 418 * spell to actually hit the counterwall and be sucked up.
528 */ 419 */
529 if ((tmp->attacktype & AT_COUNTERSPELL) && 420 if ((tmp->attacktype & AT_COUNTERSPELL)
530 (tmp->type != PLAYER) && !QUERY_FLAG (tmp, FLAG_MONSTER) && 421 && !QUERY_FLAG (tmp, FLAG_MONSTER)
531 (tmp->type != WEAPON) && (tmp->type != BOW) && (tmp->type != ARROW) && (tmp->type != GOLEM) && (immune_stop & AT_MAGIC)) 422 && (tmp->type != PLAYER)
423 && (tmp->type != WEAPON)
424 && (tmp->type != BOW)
425 && (tmp->type != ARROW)
426 && (tmp->type != GOLEM)
427 && !QUERY_FLAG (tmp, FLAG_IS_FLOOR) // XXX:
428 // we special case floor here because there
429 // are sometimes spell effect floors
430 // which are used to inflict damage
431 // (and those shouldn't go away from
432 // sanctuary) see also: permanent lava
433 && (immune_stop & AT_MAGIC))
532 return 0; 434 return 0;
533 435
534 /* This is to prevent 'out of control' spells. Basically, this 436 /* This is to prevent 'out of control' spells. Basically, this
535 * limits one spell effect per space per spell. This is definately 437 * limits one spell effect per space per spell. This is definately
536 * needed for performance reasons, and just for playability I believe. 438 * needed for performance reasons, and just for playability I believe.
543 /* 445 /*
544 * Combine similar spell effects into one spell effect. Needed for 446 * Combine similar spell effects into one spell effect. Needed for
545 * performance reasons with meteor swarm and the like, but also for 447 * performance reasons with meteor swarm and the like, but also for
546 * playability reasons. 448 * playability reasons.
547 */ 449 */
548 if (tmp->arch == op->arch 450 if (tmp->arch == op->arch /* no harm if not comparing by name here */
549 && tmp->type == op->type 451 && tmp->type == op->type
550 && tmp->subtype == op->subtype 452 && tmp->subtype == op->subtype
453 && tmp->owner == op->owner
551 && tmp->owner == op->owner && ((tmp->subtype == SP_EXPLOSION) || (tmp->subtype == SP_CONE && tmp->stats.sp == op->stats.sp))) 454 && ((tmp->subtype == SP_EXPLOSION) || (tmp->subtype == SP_CONE && tmp->stats.sp == op->stats.sp)))
552 { 455 {
553 tmp->stats.dam = MAX (tmp->stats.dam, op->stats.dam); 456 tmp->stats.dam = MAX (tmp->stats.dam, op->stats.dam);
554 tmp->range = MAX (tmp->range, op->range); 457 tmp->range = MAX (tmp->range, op->range);
555 tmp->duration = MAX (tmp->duration, op->duration); 458 tmp->duration = MAX (tmp->duration, op->duration);
556 return 0; 459 return 0;
558 461
559 /* Perhaps we should also put checks in for no magic and unholy 462 /* Perhaps we should also put checks in for no magic and unholy
560 * ground to prevent it from moving along? 463 * ground to prevent it from moving along?
561 */ 464 */
562 } 465 }
466
563 /* If it passes the above tests, it must be OK */ 467 /* If it passes the above tests, it must be OK */
564 return 1; 468 return 1;
565} 469}
566 470
567/* fire_arch_from_position: fires an archetype. 471/* fire_arch_from_position: fires an archetype.
572 * dir: direction to fire in. 476 * dir: direction to fire in.
573 * spell: spell that is being fired. It uses other_arch for the archetype 477 * spell: spell that is being fired. It uses other_arch for the archetype
574 * to fire. 478 * to fire.
575 * returns 0 on failure, 1 on success. 479 * returns 0 on failure, 1 on success.
576 */ 480 */
577
578int 481int
579fire_arch_from_position (object *op, object *caster, sint16 x, sint16 y, int dir, object *spell) 482fire_arch_from_position (object *op, object *caster, sint16 x, sint16 y, int dir, object *spell)
580{ 483{
581 object *tmp;
582 int mflags;
583 maptile *m;
584
585 if (spell->other_arch == NULL) 484 if (!spell->other_arch)
586 return 0; 485 return 0;
587 486
588 m = op->map; 487 object *tmp = spell->other_arch->instance ();
589 mflags = get_map_flags (m, &m, x, y, &x, &y);
590 if (mflags & P_OUT_OF_MAP)
591 {
592 return 0;
593 }
594 488
595 tmp = arch_to_object (spell->other_arch); 489 if (!tmp)
596
597 if (tmp == NULL)
598 return 0; 490 return 0;
599
600 if (OB_TYPE_MOVE_BLOCK (tmp, GET_MAP_MOVE_BLOCK (m, x, y)))
601 {
602 new_draw_info (NDI_UNIQUE, 0, op, "You can't cast the spell on top of a wall!\n");
603 free_object (tmp);
604 return 0;
605 }
606
607
608 491
609 tmp->stats.dam = spell->stats.dam + SP_level_dam_adjust (caster, spell); 492 tmp->stats.dam = spell->stats.dam + SP_level_dam_adjust (caster, spell);
610 tmp->duration = spell->duration + SP_level_duration_adjust (caster, spell); 493 tmp->duration = spell->duration + SP_level_duration_adjust (caster, spell);
611 /* code in time.c uses food for some things, duration for others */ 494 /* code in time.c uses food for some things, duration for others */
612 tmp->stats.food = tmp->duration; 495 tmp->stats.food = tmp->duration;
613 tmp->range = spell->range + SP_level_range_adjust (caster, spell); 496 tmp->range = spell->range + SP_level_range_adjust (caster, spell);
614 tmp->attacktype = spell->attacktype; 497 tmp->attacktype = spell->attacktype;
615 tmp->x = x;
616 tmp->y = y;
617 tmp->direction = dir; 498 tmp->direction = dir;
618 if (get_owner (op) != NULL) 499 tmp->set_owner (op);
619 copy_owner (tmp, op);
620 else
621 set_owner (tmp, op);
622 tmp->level = caster_level (caster, spell); 500 tmp->level = casting_level (caster, spell);
623 set_spell_skill (op, caster, spell, tmp); 501 set_spell_skill (op, caster, spell, tmp);
624 502
625 /* needed for AT_HOLYWORD,AT_GODPOWER stuff */ 503 /* needed for AT_HOLYWORD,AT_GODPOWER stuff */
626 if (tmp->attacktype & AT_HOLYWORD || tmp->attacktype & AT_GODPOWER) 504 if (tmp->attacktype & AT_HOLYWORD || tmp->attacktype & AT_GODPOWER)
627 {
628 if (!tailor_god_spell (tmp, op)) 505 if (!tailor_god_spell (tmp, op))
629 return 0; 506 return 0;
630 } 507
631 if (QUERY_FLAG (tmp, FLAG_IS_TURNABLE)) 508 if (QUERY_FLAG (tmp, FLAG_IS_TURNABLE))
632 SET_ANIMATION (tmp, dir); 509 SET_ANIMATION (tmp, dir);
633 510
634 if ((tmp = insert_ob_in_map (tmp, m, op, 0)) == NULL) 511 if ((tmp = op->map->insert (tmp, x, y, op)))
635 return 1;
636
637 move_spell_effect (tmp); 512 move_spell_effect (tmp);
638 513
639 return 1; 514 return 1;
640} 515}
641
642
643 516
644/***************************************************************************** 517/*****************************************************************************
645 * 518 *
646 * Code related to rods - perhaps better located in another file? 519 * Code related to rods - perhaps better located in another file?
647 * 520 *
648 ****************************************************************************/ 521 ****************************************************************************/
649
650void 522void
651regenerate_rod (object *rod) 523regenerate_rod (object *rod)
652{ 524{
653 if (rod->stats.hp < rod->stats.maxhp) 525 if (rod->stats.hp < rod->stats.maxhp)
654 { 526 rod->stats.hp = min (rod->stats.maxhp, rod->stats.hp + 1 + rod->stats.maxhp / 10);
655 rod->stats.hp += 1 + rod->stats.maxhp / 10;
656
657 if (rod->stats.hp > rod->stats.maxhp)
658 rod->stats.hp = rod->stats.maxhp;
659 }
660} 527}
661
662 528
663void 529void
664drain_rod_charge (object *rod) 530drain_rod_charge (object *rod)
665{ 531{
666 rod->stats.hp -= SP_level_spellpoint_cost (rod, rod->inv, SPELL_HIGHEST); 532 rod->stats.hp -= SP_level_spellpoint_cost (rod, rod->inv, SPELL_HIGHEST);
667} 533}
668
669
670
671 534
672/* this function is commonly used to find a friendly target for 535/* this function is commonly used to find a friendly target for
673 * spells such as heal or protection or armour 536 * spells such as heal or protection or armour
674 * op is what is looking for the target (which can be a player), 537 * op is what is looking for the target (which can be a player),
675 * dir is the direction we are looking in. Return object found, or 538 * dir is the direction we are looking in. Return object found, or
676 * NULL if no good object. 539 * NULL if no good object.
677 */ 540 */
678
679object * 541object *
680find_target_for_friendly_spell (object *op, int dir) 542find_target_for_friendly_spell (object *op, int dir)
681{ 543{
682 object *tmp; 544 object *tmp;
683 maptile *m;
684 sint16 x, y;
685 int mflags;
686 545
687 /* I don't really get this block - if op isn't a player or rune, 546 /* I don't really get this block - if op isn't a player or rune,
688 * we then make the owner of this object the target. 547 * we then make the owner of this object the target.
689 * The owner could very well be no where near op. 548 * The owner could very well be no where near op.
690 */ 549 */
691 if (op->type != PLAYER && op->type != RUNE) 550 if (op->type != PLAYER && op->type != RUNE)
692 { 551 {
693 tmp = get_owner (op); 552 tmp = op->owner;
694 /* If the owner does not exist, or is not a monster, than apply the spell 553 /* If the owner does not exist, or is not a monster, than apply the spell
695 * to the caster. 554 * to the caster.
696 */ 555 */
697 if (!tmp || !QUERY_FLAG (tmp, FLAG_MONSTER)) 556 if (!tmp || !QUERY_FLAG (tmp, FLAG_MONSTER))
698 tmp = op; 557 tmp = op;
699 } 558 }
700 else 559 else
701 { 560 {
702 m = op->map; 561 maptile *m = op->map;
703 x = op->x + freearr_x[dir]; 562 sint16 x = op->x + freearr_x[dir];
704 y = op->y + freearr_y[dir]; 563 sint16 y = op->y + freearr_y[dir];
705
706 mflags = get_map_flags (m, &m, x, y, &x, &y);
707
708 if (mflags & P_OUT_OF_MAP)
709 tmp = NULL;
710 else
711 { 564
712 for (tmp = get_map_ob (m, x, y); tmp != NULL; tmp = tmp->above) 565 tmp = xy_normalise (m, x, y)
713 if (tmp->type == PLAYER) 566 ? m->at (x, y).player ()
714 break; 567 : 0;
715 }
716 } 568 }
569
717 /* didn't find a player there, look in current square for a player */ 570 /* didn't find a player there, look in current square for a player */
718 if (tmp == NULL) 571 if (!tmp)
719 for (tmp = get_map_ob (op->map, op->x, op->y); tmp != NULL; tmp = tmp->above) 572 tmp = op->ms ().player ();
720 {
721 if (tmp->type == PLAYER)
722 break;
723 }
724 573
725 return tmp; 574 return tmp;
726} 575}
727
728
729 576
730/* raytrace: 577/* raytrace:
731 * spell_find_dir(map, x, y, exclude) will search first the center square 578 * spell_find_dir(map, x, y, exclude) will search first the center square
732 * then some close squares in the given map at the given coordinates for 579 * then some close squares in the given map at the given coordinates for
733 * live objects. 580 * live objects.
736 * monsters/generators only. If not, the spell will hunt players only. 583 * monsters/generators only. If not, the spell will hunt players only.
737 * It returns the direction toward the first/closest live object if it finds 584 * It returns the direction toward the first/closest live object if it finds
738 * any, otherwise -1. 585 * any, otherwise -1.
739 * note that exclude can be NULL, in which case all bets are off. 586 * note that exclude can be NULL, in which case all bets are off.
740 */ 587 */
741
742int 588int
743spell_find_dir (maptile *m, int x, int y, object *exclude) 589spell_find_dir (maptile *m, int x, int y, object *exclude)
744{ 590{
745 int i, max = SIZEOFFREE; 591 int i, max = SIZEOFFREE;
746 sint16 nx, ny; 592 sint16 nx, ny;
760 mp = m; 606 mp = m;
761 mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny); 607 mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny);
762 if (mflags & (P_OUT_OF_MAP | P_BLOCKSVIEW)) 608 if (mflags & (P_OUT_OF_MAP | P_BLOCKSVIEW))
763 continue; 609 continue;
764 610
765 tmp = get_map_ob (mp, nx, ny); 611 tmp = GET_MAP_OB (mp, nx, ny);
766 612
767 while (tmp != NULL && (((owner_type == PLAYER && 613 while (tmp != NULL && (((owner_type == PLAYER &&
768 !QUERY_FLAG (tmp, FLAG_MONSTER) && !QUERY_FLAG (tmp, FLAG_GENERATOR)) || 614 !QUERY_FLAG (tmp, FLAG_MONSTER) && !QUERY_FLAG (tmp, FLAG_GENERATOR)) ||
769 (owner_type != PLAYER && tmp->type != PLAYER)) || (tmp == exclude || (tmp->head && tmp->head == exclude)))) 615 (owner_type != PLAYER && tmp->type != PLAYER)) || (tmp == exclude || (tmp->head && tmp->head == exclude))))
770 tmp = tmp->above; 616 tmp = tmp->above;
773 return freedir[i]; 619 return freedir[i];
774 } 620 }
775 return -1; /* flag for "keep going the way you were" */ 621 return -1; /* flag for "keep going the way you were" */
776} 622}
777 623
778
779
780/* put_a_monster: puts a monster named monstername near by 624/* put_a_monster: puts a monster named monstername near by
781 * op. This creates the treasures for the monsters, and 625 * op. This creates the treasures for the monsters, and
782 * also deals with multipart monsters properly. 626 * also deals with multipart monsters properly.
783 */ 627 */
784
785void 628void
786put_a_monster (object *op, const char *monstername) 629put_a_monster (object *op, const char *monstername)
787{ 630{
788 object *tmp, *head = NULL, *prev = NULL; 631 object *tmp, *head = NULL, *prev = NULL;
789 archetype *at; 632 archetype *at;
796 639
797 /* find a free square nearby 640 /* find a free square nearby
798 * first we check the closest square for free squares 641 * first we check the closest square for free squares
799 */ 642 */
800 643
801 dir = find_first_free_spot (&at->clone, op->map, op->x, op->y); 644 dir = find_first_free_spot (at, op->map, op->x, op->y);
802 if (dir != -1) 645 if (dir != -1)
803 { 646 {
804 /* This is basically grabbed for generate monster. Fixed 971225 to 647 /* This is basically grabbed for generate monster. Fixed 971225 to
805 * insert multipart monsters properly 648 * insert multipart monsters properly
806 */ 649 */
650 //TODO: use expand_tail + ...
807 while (at != NULL) 651 while (at != NULL)
808 { 652 {
809 tmp = arch_to_object (at); 653 tmp = arch_to_object (at);
810 tmp->x = op->x + freearr_x[dir] + at->clone.x; 654 tmp->x = op->x + freearr_x[dir] + at->x;
811 tmp->y = op->y + freearr_y[dir] + at->clone.y; 655 tmp->y = op->y + freearr_y[dir] + at->y;
812 tmp->map = op->map; 656 tmp->map = op->map;
813 if (head) 657 if (head)
814 { 658 {
815 tmp->head = head; 659 tmp->head = head;
816 prev->more = tmp; 660 prev->more = tmp;
817 } 661 }
662
818 if (!head) 663 if (!head)
819 head = tmp; 664 head = tmp;
665
820 prev = tmp; 666 prev = tmp;
667
821 at = at->more; 668 at = (archetype *)at->more;
822 } 669 }
823 670
824 if (head->randomitems) 671 if (head->randomitems)
825 create_treasure (head->randomitems, head, GT_INVISIBLE, op->map->difficulty, 0); 672 create_treasure (head->randomitems, head, GT_INVISIBLE, op->map->difficulty, 0);
826 673
827 insert_ob_in_map (head, op->map, op, 0); 674 insert_ob_in_map (head, op->map, op, 0);
828 675
829 /* thought it'd be cool to insert a burnout, too. */ 676 /* thought it'd be cool to insert a burnout, too. */
830 tmp = get_archetype ("burnout"); 677 op->map->insert (archetype::get (shstr_burnout), op->x + freearr_x[dir], op->y + freearr_y[dir], op);
831 tmp->map = op->map;
832 tmp->x = op->x + freearr_x[dir];
833 tmp->y = op->y + freearr_y[dir];
834 insert_ob_in_map (tmp, op->map, op, 0);
835 } 678 }
836} 679}
837 680
838/* peterm: function which summons hostile monsters and 681/* peterm: function which summons hostile monsters and
839 * places them in nearby squares. 682 * places them in nearby squares.
866{ 709{
867 int attacktype; 710 int attacktype;
868 int face; 711 int face;
869} ATTACKS[22] = 712} ATTACKS[22] =
870{ 713{
871 {
872 AT_PHYSICAL, 0}, 714 { AT_PHYSICAL, 0},
873 {
874 AT_PHYSICAL, 0}, /*face = explosion */ 715 { AT_PHYSICAL, 0}, /*face = explosion */
875 {
876 AT_PHYSICAL, 0}, 716 { AT_PHYSICAL, 0},
877 {
878 AT_MAGIC, 1}, 717 { AT_MAGIC, 1},
879 {
880 AT_MAGIC, 1}, /* face = last-burnout */ 718 { AT_MAGIC, 1}, /* face = last-burnout */
881 {
882 AT_MAGIC, 1}, 719 { AT_MAGIC, 1},
883 {
884 AT_FIRE, 2}, 720 { AT_FIRE, 2},
885 {
886 AT_FIRE, 2}, /* face = fire.... */ 721 { AT_FIRE, 2}, /* face = fire.... */
887 {
888 AT_FIRE, 2}, 722 { AT_FIRE, 2},
889 {
890 AT_ELECTRICITY, 3}, 723 { AT_ELECTRICITY, 3},
891 {
892 AT_ELECTRICITY, 3}, /* ball_lightning */ 724 { AT_ELECTRICITY, 3}, /* ball_lightning */
893 {
894 AT_ELECTRICITY, 3}, 725 { AT_ELECTRICITY, 3},
895 {
896 AT_COLD, 4}, 726 { AT_COLD, 4},
897 {
898 AT_COLD, 4}, /* face=icestorm */ 727 { AT_COLD, 4}, /* face=icestorm */
899 {
900 AT_COLD, 4}, 728 { AT_COLD, 4},
901 {
902 AT_CONFUSION, 5}, 729 { AT_CONFUSION, 5},
903 {
904 AT_POISON, 7}, 730 { AT_POISON, 7},
905 {
906 AT_POISON, 7}, /* face = acid sphere. generator */ 731 { AT_POISON, 7}, /* face = acid sphere. generator */
907 {
908 AT_POISON, 7}, /* poisoncloud face */ 732 { AT_POISON, 7}, /* poisoncloud face */
909 {
910 AT_SLOW, 8}, 733 { AT_SLOW, 8},
911 {
912 AT_PARALYZE, 9}, 734 { AT_PARALYZE, 9},
913 {
914AT_FEAR, 10}}; 735 { AT_FEAR, 10},
915 736};
916
917 737
918/* shuffle_attack: peterm 738/* shuffle_attack: peterm
919 * This routine shuffles the attack of op to one of the 739 * This routine shuffles the attack of op to one of the
920 * ones in the list. It does this at random. It also 740 * ones in the list. It does this at random. It also
921 * chooses a face appropriate to the attack that is 741 * chooses a face appropriate to the attack that is
966 if (failure <= -20 && failure > -40) /* wonder */ 786 if (failure <= -20 && failure > -40) /* wonder */
967 { 787 {
968 new_draw_info_format (NDI_UNIQUE, 0, op, "%s gives a sign to renew your faith.", godname); 788 new_draw_info_format (NDI_UNIQUE, 0, op, "%s gives a sign to renew your faith.", godname);
969 tmp = get_archetype (SPELL_WONDER); 789 tmp = get_archetype (SPELL_WONDER);
970 cast_cone (op, op, 0, tmp); 790 cast_cone (op, op, 0, tmp);
971 free_object (tmp); 791 tmp->destroy (true);
972 } 792 }
973 793
974 else if (failure <= -40 && failure > -60) /* confusion */ 794 else if (failure <= -40 && failure > -60) /* confusion */
975 { 795 {
976 new_draw_info (NDI_UNIQUE, 0, op, "Your diety touches your mind!"); 796 new_draw_info (NDI_UNIQUE, 0, op, "Your diety touches your mind!");
1010 if (failure <= -20 && failure > -40) /* wonder */ 830 if (failure <= -20 && failure > -40) /* wonder */
1011 { 831 {
1012 new_draw_info (NDI_UNIQUE, 0, op, "Your spell causes an unexpected effect."); 832 new_draw_info (NDI_UNIQUE, 0, op, "Your spell causes an unexpected effect.");
1013 tmp = get_archetype (SPELL_WONDER); 833 tmp = get_archetype (SPELL_WONDER);
1014 cast_cone (op, op, 0, tmp); 834 cast_cone (op, op, 0, tmp);
1015 free_object (tmp); 835 tmp->destroy (true);
1016 } 836 }
1017 837
1018 else if (failure <= -40 && failure > -60) /* confusion */ 838 else if (failure <= -40 && failure > -60) /* confusion */
1019 { 839 {
1020 new_draw_info (NDI_UNIQUE, 0, op, "Your magic recoils on you, making you confused!"); 840 new_draw_info (NDI_UNIQUE, 0, op, "Your magic recoils on you, making you confused!");
1039 else 859 else
1040 { 860 {
1041 new_draw_info (NDI_UNIQUE, 0, op, "You lose control of the mana! The uncontrolled magic blasts you!"); 861 new_draw_info (NDI_UNIQUE, 0, op, "You lose control of the mana! The uncontrolled magic blasts you!");
1042 tmp = get_archetype (LOOSE_MANA); 862 tmp = get_archetype (LOOSE_MANA);
1043 tmp->level = skill->level; 863 tmp->level = skill->level;
1044 tmp->x = op->x;
1045 tmp->y = op->y;
1046 864
1047 /* increase the area of destruction a little for more powerful spells */ 865 /* increase the area of destruction a little for more powerful spells */
1048 tmp->range += isqrt (power); 866 tmp->range += isqrt (power);
1049 867
1050 if (power > 25) 868 if (power > 25)
1051 tmp->stats.dam = 25 + isqrt (power); 869 tmp->stats.dam = 25 + isqrt (power);
1052 else 870 else
1053 tmp->stats.dam = power; /* nasty recoils! */ 871 tmp->stats.dam = power; /* nasty recoils! */
1054 872
1055 tmp->stats.maxhp = tmp->count; 873 tmp->stats.maxhp = tmp->count;
1056 insert_ob_in_map (tmp, op->map, NULL, 0); 874
875 tmp->insert_at (op);
1057 } 876 }
1058 } 877 }
1059} 878}
1060 879
1061int 880int
1062cast_party_spell (object *op, object *caster, int dir, object *spell_ob, char *stringarg) 881cast_party_spell (object *op, object *caster, int dir, object *spell_ob, char *stringarg)
1063{ 882{
1064 int success; 883 int success;
1065 player *pl;
1066 object *spell; 884 object *spell;
1067 885
1068 if (!spell_ob->other_arch) 886 if (!spell_ob->other_arch)
1069 { 887 {
1070 LOG (llevError, "cast_party_spell: empty other arch\n"); 888 LOG (llevError, "cast_party_spell: empty other arch\n");
1071 return 0; 889 return 0;
1072 } 890 }
891
1073 spell = arch_to_object (spell_ob->other_arch); 892 spell = arch_to_object (spell_ob->other_arch);
1074 893
1075 /* Always cast spell on caster */ 894 /* Always cast spell on caster */
1076 success = cast_spell (op, caster, dir, spell, stringarg); 895 success = cast_spell (op, caster, dir, spell, stringarg);
1077 896
1078 if (caster->contr->party == NULL) 897 if (caster->contr->party == NULL)
1079 { 898 {
1080 remove_ob (spell); 899 spell->remove ();
1081 return success; 900 return success;
1082 } 901 }
1083 for (pl = first_player; pl != NULL; pl = pl->next) 902
903 for_all_players (pl)
1084 if ((pl->ob->contr->party == caster->contr->party) && (on_same_map (pl->ob, caster))) 904 if ((pl->ob->contr->party == caster->contr->party) && (on_same_map (pl->ob, caster)))
1085 {
1086 cast_spell (pl->ob, caster, pl->ob->facing, spell, stringarg); 905 cast_spell (pl->ob, caster, pl->ob->facing, spell, stringarg);
1087 } 906
1088 remove_ob (spell); 907 spell->remove ();
1089 return success; 908 return success;
1090} 909}
1091 910
1092/* This is where the main dispatch when someone casts a spell. 911/* This is where the main dispatch when someone casts a spell.
1093 * 912 *
1114 * 933 *
1115 * if it is a player casting the spell (op->type == PLAYER, op == caster), 934 * if it is a player casting the spell (op->type == PLAYER, op == caster),
1116 * this function will decrease the mana/grace appropriately. For other 935 * this function will decrease the mana/grace appropriately. For other
1117 * objects, the caller should do what it considers appropriate. 936 * objects, the caller should do what it considers appropriate.
1118 */ 937 */
1119
1120int 938int
1121cast_spell (object *op, object *caster, int dir, object *spell_ob, char *stringarg) 939cast_spell (object *op, object *caster, int dir, object *spell_ob, char *stringarg)
1122{ 940{
1123 const char *godname; 941 const char *godname;
1124 int success = 0, mflags, cast_level = 0, old_shoottype; 942 int success = 0, cast_level = 0;
1125 object *skill = NULL; 943 object *skill = NULL;
1126
1127 old_shoottype = op->contr ? op->contr->shoottype : 0;
1128 944
1129 if (!spell_ob) 945 if (!spell_ob)
1130 { 946 {
1131 LOG (llevError, "cast_spell: null spell object passed\n"); 947 LOG (llevError, "cast_spell: null spell object passed\n");
1132 return 0; 948 return 0;
1143 } 959 }
1144 960
1145 /* if caster is a spell casting object, this normally shouldn't be 961 /* if caster is a spell casting object, this normally shouldn't be
1146 * an issue, because they don't have any spellpaths set up. 962 * an issue, because they don't have any spellpaths set up.
1147 */ 963 */
1148 if (caster->path_denied & spell_ob->path_attuned) 964 if (caster->path_denied & spell_ob->path_attuned && !QUERY_FLAG (caster, FLAG_WIZCAST))
1149 { 965 {
1150 new_draw_info (NDI_UNIQUE, 0, op, "That spell path is denied to you."); 966 new_draw_info (NDI_UNIQUE, 0, op, "That spell path is denied to you.");
1151 return 0; 967 return 0;
1152 } 968 }
1153 969
1160 * need to have the right skill pointer passed, so we need to 976 * need to have the right skill pointer passed, so we need to
1161 * at least process that code. 977 * at least process that code.
1162 */ 978 */
1163 if (op->type == PLAYER && op == caster) 979 if (op->type == PLAYER && op == caster)
1164 { 980 {
1165 cast_level = caster_level (caster, spell_ob);
1166 if (spell_ob->skill) 981 if (spell_ob->skill)
1167 { 982 {
1168 skill = find_skill_by_name (op, spell_ob->skill); 983 skill = find_skill_by_name (op, spell_ob->skill);
984
1169 if (!skill) 985 if (!skill)
1170 { 986 {
1171 new_draw_info_format (NDI_UNIQUE, 0, op, "You need the skill %s to cast %s.", &spell_ob->skill, &spell_ob->name); 987 op->failmsg (format ("You need the skill %s to cast %s! "
988 "H<You either need to learn the skill via a skill scroll "
989 "or you need to wear a talisman or holy symbol.>",
990 &spell_ob->skill, &spell_ob->name));
1172 return 0; 991 return 0;
1173 } 992 }
1174 if (min_casting_level (op, spell_ob) > cast_level && !QUERY_FLAG (op, FLAG_WIZ)) 993
994 const char *msg = "";
995
996 int caster_level = skill->level;
997
998 if (op->path_attuned & spell_ob->path_attuned)
1175 { 999 {
1176 new_draw_info (NDI_UNIQUE, 0, op, "You lack enough skill to cast that spell."); 1000 caster_level += min (cast_level * 2, ATTUNE_REPELL);
1001 msg = " (attuned)";
1002 }
1003
1004 if (op->path_repelled & spell_ob->path_attuned)
1005 {
1006 caster_level = ATTUNE_REPELL; // negative is ok
1007 msg = " (repelled)";
1008 }
1009
1010 if (spell_ob->level > caster_level)
1011 {
1012 op->failmsg (format ("You lack enough skill to cast that spell! "
1013 "H<Your effective cast level is %d%s, but level %d is required.>",
1014 caster_level, msg, spell_ob->level));
1015 if (!op->is_wiz ())
1177 return 0; 1016 return 0;
1178 } 1017 }
1179 } 1018 }
1019
1180 /* If the caster is the wiz, they don't ever fail, and don't have 1020 /* If the caster is the wiz, they don't ever fail, and don't have
1181 * to have sufficient grace/mana. 1021 * to have sufficient grace/mana.
1182 */ 1022 */
1183 if (!QUERY_FLAG (op, FLAG_WIZ)) 1023 if (!QUERY_FLAG (op, FLAG_WIZ))
1184 { 1024 {
1185 if (SP_level_spellpoint_cost (caster, spell_ob, SPELL_MANA) && 1025 if (SP_level_spellpoint_cost (caster, spell_ob, SPELL_MANA) &&
1186 SP_level_spellpoint_cost (caster, spell_ob, SPELL_MANA) > op->stats.sp) 1026 SP_level_spellpoint_cost (caster, spell_ob, SPELL_MANA) > op->stats.sp)
1187 { 1027 {
1188 new_draw_info (NDI_UNIQUE, 0, op, "You don't have enough mana."); 1028 op->failmsg ("You don't have enough mana!");
1189 return 0; 1029 return 0;
1190 } 1030 }
1031
1191 if (SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE) && 1032 if (SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE) &&
1192 SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE) > op->stats.grace) 1033 SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE) > op->stats.grace)
1193 { 1034 {
1194 if (random_roll (0, op->stats.Wis - 1, op, PREFER_HIGH) + op->stats.grace - 1035 if (random_roll (0, op->stats.Wis - 1, op, PREFER_HIGH) + op->stats.grace -
1195 10 * SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE) / op->stats.maxgrace > 0) 1036 10 * SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE) / op->stats.maxgrace > 0)
1196 {
1197 new_draw_info_format (NDI_UNIQUE, 0, op, "%s grants your prayer, though you are unworthy.", godname); 1037 op->statusmsg (format ("%s grants your prayer, though you are unworthy.", godname));
1198 }
1199 else 1038 else
1200 { 1039 {
1201 prayer_failure (op, op->stats.grace, SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE) - op->stats.grace); 1040 prayer_failure (op, op->stats.grace, SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE) - op->stats.grace);
1202 new_draw_info_format (NDI_UNIQUE, 0, op, "%s ignores your prayer.", godname); 1041 op->failmsg (format ("%s ignores your prayer.", godname));
1203 return 0; 1042 return 0;
1204 } 1043 }
1205 } 1044 }
1206 1045
1207 /* player/monster is trying to cast the spell. might fumble it */ 1046 /* player/monster is trying to cast the spell. might fumble it */
1208 if (spell_ob->stats.grace && random_roll (0, 99, op, PREFER_HIGH) < 1047 if (spell_ob->stats.grace && random_roll (0, 99, op, PREFER_HIGH) <
1209 (spell_ob->level / (float) MAX (1, op->level) * cleric_chance[op->stats.Wis])) 1048 (spell_ob->level / (float) MAX (1, op->level) * cleric_chance[op->stats.Wis]))
1210 { 1049 {
1211 play_sound_player_only (op->contr, SOUND_FUMBLE_SPELL, 0, 0); 1050 op->contr->play_sound (sound_find ("fumble_spell"));
1212 new_draw_info (NDI_UNIQUE, 0, op, "You fumble the spell."); 1051 op->failmsg ("You fumble the prayer.");
1213 if (settings.casting_time == TRUE) 1052
1214 {
1215 op->casting_time = -1;
1216 }
1217 op->stats.grace -= random_roll (1, SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE), op, PREFER_LOW); 1053 op->stats.grace -= random_roll (1, SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE), op, PREFER_LOW);
1218 return 0; 1054 return 0;
1219 } 1055 }
1220 else if (spell_ob->stats.sp) 1056 else if (spell_ob->stats.sp)
1221 { 1057 {
1222 int failure = random_roll (0, 199, op, PREFER_HIGH) - op->contr->encumbrance + op->level - spell_ob->level + 35; 1058 int failure = random_roll (0, 199, op, PREFER_HIGH) - op->contr->encumbrance + op->level - spell_ob->level + 35;
1223 1059
1224 if (failure < 0) 1060 if (failure < 0)
1225 { 1061 {
1226 new_draw_info (NDI_UNIQUE, 0, op, "You bungle the spell because you have too much heavy equipment in use."); 1062 op->failmsg ("You bungle the spell because you have too much heavy equipment in use.");
1227 if (settings.spell_failure_effects == TRUE) 1063 if (settings.spell_failure_effects == TRUE)
1228 spell_failure (op, failure, SP_level_spellpoint_cost (caster, spell_ob, SPELL_MANA), skill); 1064 spell_failure (op, failure, SP_level_spellpoint_cost (caster, spell_ob, SPELL_MANA), skill);
1229 op->contr->shoottype = (rangetype) old_shoottype; 1065
1230 op->stats.sp -= random_roll (0, SP_level_spellpoint_cost (caster, spell_ob, SPELL_MANA), op, PREFER_LOW); 1066 op->stats.sp -= random_roll (0, SP_level_spellpoint_cost (caster, spell_ob, SPELL_MANA), op, PREFER_LOW);
1231 return 0; 1067 return 0;
1232 } 1068 }
1233 } 1069 }
1234 } 1070 }
1235 } 1071 }
1236 1072
1237 mflags = get_map_flags (op->map, NULL, op->x, op->y, NULL, NULL); 1073 int mflags = op->ms ().flags ();
1238 1074
1239 /* See if we can cast a spell here. If the caster and op are 1075 /* See if we can cast a spell here. If the caster and op are
1240 * not alive, then this would mean that the mapmaker put the 1076 * not alive, then this would mean that the mapmaker put the
1241 * objects on the space - presume that they know what they are 1077 * objects on the space - presume that they know what they are
1242 * doing. 1078 * doing.
1243 */ 1079 */
1244 1080
1245 if ((mflags & P_SAFE) && !QUERY_FLAG (op, FLAG_WIZCAST)) // There is _ABSOLUTELY_ no magic allowed here (except wizards :-)! 1081 if ((mflags & P_SAFE) && !QUERY_FLAG (op, FLAG_WIZCAST)) // There is _ABSOLUTELY_ no magic allowed here (except wizards :-)!
1246 { 1082 {
1247 new_draw_info (NDI_UNIQUE, 0, op, "This ground is sacred! The gods prevent any magical effects done by you here!."); 1083 op->failmsg ("This ground is sacred! The gods prevent any magical effects done by you here!");
1248 return 0; 1084 return 0;
1249 } 1085 }
1250 1086
1251 if ((spell_ob->type == SPELL) 1087 if ((spell_ob->type == SPELL)
1252 && (caster->type != POTION) 1088 && (caster->type != POTION)
1257 { 1093 {
1258 if (op->type != PLAYER) 1094 if (op->type != PLAYER)
1259 return 0; 1095 return 0;
1260 1096
1261 if ((mflags & P_NO_CLERIC) && spell_ob->stats.grace) 1097 if ((mflags & P_NO_CLERIC) && spell_ob->stats.grace)
1262 new_draw_info_format (NDI_UNIQUE, 0, op, "This ground is unholy! %s ignores you.", godname); 1098 op->failmsg (format ("This ground is unholy! %s ignores you.", godname));
1263 else 1099 else if (object *item = op->contr->ranged_ob)
1264 switch (op->contr->shoottype)
1265 { 1100 {
1266 case range_magic: 1101 if (item->type == SPELL)
1267 new_draw_info (NDI_UNIQUE, 0, op, "Something blocks your spellcasting."); 1102 op->failmsg ("Something blocks your spellcasting.");
1268 break; 1103 else if (item->type == SCROLL)
1269 case range_misc: 1104 op->failmsg ("Something blocks the magic of your scroll.");
1270 new_draw_info (NDI_UNIQUE, 0, op, "Something blocks the magic of your item.");
1271 break;
1272 case range_golem:
1273 new_draw_info (NDI_UNIQUE, 0, op, "Something blocks the magic of your scroll.");
1274 break;
1275 default:
1276 break;
1277 }
1278 return 0;
1279 }
1280
1281 if (caster == op && settings.casting_time == TRUE && spell_ob->type == SPELL)
1282 {
1283 if (op->casting_time == -1)
1284 { /* begin the casting */
1285 op->casting_time = (sint16) (spell_ob->casting_time * PATH_TIME_MULT (op, spell_ob));
1286 op->spell = spell_ob;
1287 /* put the stringarg into the object struct so that when the
1288 * spell is actually cast, it knows about the stringarg.
1289 * necessary for the invoke command spells.
1290 */
1291 if (stringarg)
1292 {
1293 op->spellarg = strdup_local (stringarg);
1294 }
1295 else 1105 else
1296 op->spellarg = NULL; 1106 op->failmsg ("Something blocks the magic of your item.");
1297 return 0;
1298 }
1299 else if (op->casting_time != 0)
1300 {
1301 if (op->type == PLAYER)
1302 new_draw_info (NDI_UNIQUE, 0, op, "You are casting!");
1303 return 0;
1304 } 1107 }
1305 else 1108 else
1306 { /* casting_time == 0 */ 1109 op->failmsg ("Something blocks the spell!");
1307 op->casting_time = -1; 1110
1308 spell_ob = op->spell; 1111 return 0;
1309 stringarg = op->spellarg;
1310 }
1311 }
1312 else
1313 { 1112 }
1113
1314 /* Take into account how long it takes to cast the spell. 1114 /* Take into account how long it takes to cast the spell.
1315 * if the player is casting it, then we use the time in 1115 * if the player is casting it, then we use the time in
1316 * the spell object. If it is a spell object, have it 1116 * the spell object. If it is a spell object, have it
1317 * take two ticks. Things that cast spells on the players 1117 * take two ticks. Things that cast spells on the players
1318 * behalf (eg, altars, and whatever else) shouldn't cost 1118 * behalf (eg, altars, and whatever else) shouldn't cost
1319 * the player any time. 1119 * the player any time.
1320 * Ignore casting time for firewalls 1120 * Ignore casting time for firewalls
1121 */
1122 if (caster == op && caster->type != FIREWALL)
1123 {
1124 op->speed_left -= spell_ob->casting_time * PATH_TIME_MULT (op, spell_ob) * FABS (op->speed);
1125 /* Other portions of the code may also decrement the speed of the player, so
1126 * put a lower limit so that the player isn't stuck here too long
1321 */ 1127 */
1322 if (caster == op && caster->type != FIREWALL)
1323 {
1324 op->speed_left -= spell_ob->casting_time * PATH_TIME_MULT (op, spell_ob) * FABS (op->speed);
1325 /* Other portions of the code may also decrement the speed of the player, so
1326 * put a lower limit so that the player isn't stuck here too long
1327 */
1328 if ((spell_ob->casting_time > 0) && op->speed_left < -spell_ob->casting_time * PATH_TIME_MULT (op, spell_ob) * FABS (op->speed)) 1128 if ((spell_ob->casting_time > 0) && op->speed_left < -spell_ob->casting_time * PATH_TIME_MULT (op, spell_ob) * FABS (op->speed))
1329 op->speed_left = -spell_ob->casting_time * PATH_TIME_MULT (op, spell_ob) * FABS (op->speed); 1129 op->speed_left = -spell_ob->casting_time * PATH_TIME_MULT (op, spell_ob) * FABS (op->speed);
1330 } 1130 }
1331 else if (caster->type == WAND || caster->type == HORN || caster->type == ROD || caster->type == POTION || caster->type == SCROLL) 1131 else if (caster->type == WAND || caster->type == HORN || caster->type == ROD || caster->type == POTION || caster->type == SCROLL)
1332 {
1333 op->speed_left -= 2 * FABS (op->speed); 1132 op->speed_left -= 2 * FABS (op->speed);
1334 }
1335 }
1336 1133
1337 if (op->type == PLAYER && op == caster) 1134 if (op->type == PLAYER && op == caster)
1338 { 1135 {
1339 op->stats.grace -= SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE); 1136 op->stats.grace -= SP_level_spellpoint_cost (caster, spell_ob, SPELL_GRACE);
1340 op->stats.sp -= SP_level_spellpoint_cost (caster, spell_ob, SPELL_MANA); 1137 op->stats.sp -= SP_level_spellpoint_cost (caster, spell_ob, SPELL_MANA);
1347 if (op != caster && !skill && caster->skill) 1144 if (op != caster && !skill && caster->skill)
1348 { 1145 {
1349 skill = find_skill_by_name (op, caster->skill); 1146 skill = find_skill_by_name (op, caster->skill);
1350 if (!skill) 1147 if (!skill)
1351 { 1148 {
1352 new_draw_info_format (NDI_UNIQUE, 0, op, "You lack the skill %s to use the %s", &caster->skill, query_name (caster)); 1149 op->failmsg (format ("You lack the skill %s to use the %s!", &caster->skill, query_name (caster)));
1353 return 0; 1150 return 0;
1354 } 1151 }
1152
1355 change_skill (op, skill, 0); /* needed for proper exp credit */ 1153 op->change_skill (skill); /* needed for proper exp credit */
1356 } 1154 }
1155
1156 if (INVOKE_OBJECT (CAST_SPELL, spell_ob, ARG_OBJECT (op), ARG_OBJECT (caster), ARG_INT (dir), ARG_STRING (stringarg)))
1157 return RESULT_INT (0);
1357 1158
1358 switch (spell_ob->subtype) 1159 switch (spell_ob->subtype)
1359 { 1160 {
1360 /* The order of case statements is same as the order they show up 1161 /* The order of case statements is same as the order they show up
1361 * in in spells.h. 1162 * in in spells.h.
1362 */ 1163 */
1363 case SP_RAISE_DEAD: 1164 case SP_RAISE_DEAD:
1364 success = cast_raise_dead_spell (op, caster, spell_ob, dir, stringarg); 1165 success = cast_raise_dead_spell (op, caster, spell_ob, dir, stringarg);
1365 break; 1166 break;
1366 1167
1367 case SP_RUNE: 1168 case SP_RUNE:
1395 case SP_SMITE: 1196 case SP_SMITE:
1396 success = cast_smite_spell (op, caster, dir, spell_ob); 1197 success = cast_smite_spell (op, caster, dir, spell_ob);
1397 break; 1198 break;
1398 1199
1399 case SP_MAGIC_MISSILE: 1200 case SP_MAGIC_MISSILE:
1400 success = fire_arch_from_position (op, caster, op->x + freearr_x[dir], op->y + freearr_y[dir], dir, spell_ob); 1201 success = fire_arch_from_position (op, caster, op->x, op->y, dir, spell_ob);
1401 break; 1202 break;
1402 1203
1403 case SP_SUMMON_GOLEM: 1204 case SP_SUMMON_GOLEM:
1404 success = summon_golem (op, caster, dir, spell_ob); 1205 success = summon_golem (op, caster, dir, spell_ob);
1405 old_shoottype = range_golem;
1406 break; 1206 break;
1407 1207
1408 case SP_DIMENSION_DOOR: 1208 case SP_DIMENSION_DOOR:
1409 /* dimension door needs the actual caster, because that is what is 1209 /* dimension door needs the actual caster, because that is what is
1410 * moved. 1210 * moved.
1483#ifdef NO_POLYMORPH 1283#ifdef NO_POLYMORPH
1484 /* Not great, but at least provide feedback so if players do have 1284 /* Not great, but at least provide feedback so if players do have
1485 * polymorph (ie, find it as a preset item or left over from before 1285 * polymorph (ie, find it as a preset item or left over from before
1486 * it was disabled), they get some feedback. 1286 * it was disabled), they get some feedback.
1487 */ 1287 */
1488 new_draw_info (NDI_UNIQUE, 0, op, "The spell fizzles"); 1288 op->failmsg ("The spell fizzles!");
1489 success = 0; 1289 success = 0;
1490#else 1290#else
1491 success = cast_polymorph (op, caster, spell_ob, dir); 1291 success = cast_polymorph (op, caster, spell_ob, dir);
1492#endif 1292#endif
1493 break; 1293 break;
1513 break; 1313 break;
1514 1314
1515 case SP_MOVING_BALL: 1315 case SP_MOVING_BALL:
1516 if (spell_ob->path_repelled && (spell_ob->path_repelled & caster->path_attuned) != spell_ob->path_repelled) 1316 if (spell_ob->path_repelled && (spell_ob->path_repelled & caster->path_attuned) != spell_ob->path_repelled)
1517 { 1317 {
1518 new_draw_info_format (NDI_UNIQUE, 0, op, "You lack the proper attunement to cast %s", &spell_ob->name); 1318 op->failmsg (format ("You lack the proper attunement to cast %s!", &spell_ob->name));
1519 success = 0; 1319 success = 0;
1520 } 1320 }
1521 else 1321 else
1522 success = fire_arch_from_position (op, caster, op->x + freearr_x[dir], op->y + freearr_y[dir], dir, spell_ob); 1322 success = fire_arch_from_position (op, caster, op->x, op->y, dir, spell_ob);
1523 break; 1323 break;
1524 1324
1525 case SP_SWARM: 1325 case SP_SWARM:
1526 success = fire_swarm (op, caster, spell_ob, dir); 1326 success = fire_swarm (op, caster, spell_ob, dir);
1527 break; 1327 break;
1543 success = cast_consecrate (op, caster, spell_ob); 1343 success = cast_consecrate (op, caster, spell_ob);
1544 break; 1344 break;
1545 1345
1546 case SP_ANIMATE_WEAPON: 1346 case SP_ANIMATE_WEAPON:
1547 success = animate_weapon (op, caster, spell_ob, dir); 1347 success = animate_weapon (op, caster, spell_ob, dir);
1548 old_shoottype = range_golem;
1549 break; 1348 break;
1550 1349
1551 case SP_LIGHT: 1350 case SP_LIGHT:
1552 success = cast_light (op, caster, spell_ob, dir); 1351 success = cast_light (op, caster, spell_ob, dir);
1553 break; 1352 break;
1566 1365
1567 case SP_AURA: 1366 case SP_AURA:
1568 success = create_aura (op, caster, spell_ob); 1367 success = create_aura (op, caster, spell_ob);
1569 break; 1368 break;
1570 1369
1571 case SP_TOWN_PORTAL:
1572 success = cast_create_town_portal (op, caster, spell_ob, dir);
1573 break;
1574
1575 case SP_PARTY_SPELL: 1370 case SP_PARTY_SPELL:
1576 success = cast_party_spell (op, caster, dir, spell_ob, stringarg); 1371 success = cast_party_spell (op, caster, dir, spell_ob, stringarg);
1577 break; 1372 break;
1578 1373
1579 default: 1374 default:
1580 if (!INVOKE_OBJECT (CAST_SPELL, spell_ob, ARG_OBJECT (op), ARG_OBJECT (caster), ARG_INT (dir), ARG_STRING (stringarg)))
1581 LOG (llevError, "cast_spell: Unhandled spell subtype %d\n", spell_ob->subtype); 1375 LOG (llevError, "cast_spell: Unhandled spell subtype %d\n", spell_ob->subtype);
1582 }
1583
1584 /* FIXME - we need some better sound suppport */
1585 // yes, for example, augment map info with the spell effect
1586 // so clients can calculate the sounds themselves
1587 //play_sound_map(op->map, op->x, op->y, SOUND_CAST_SPELL_0 + spell_ob->subtype);
1588
1589 /* free the spell arg */
1590 if (settings.casting_time == TRUE && stringarg)
1591 { 1376 }
1592 free (stringarg); 1377
1593 stringarg = NULL; 1378 op->play_sound (
1594 } 1379 success
1595 /* perhaps a bit of a hack, but if using a wand, it has to change the skill 1380 ? spell_ob->sound
1596 * to something like use_magic_item, but you really want to be able to fire 1381 ? spell_ob->sound
1597 * it again. 1382 : sound_find ("spell_success")
1598 */ 1383 : sound_find ("fumble_spell")
1599 if (op->contr) 1384 );
1600 op->contr->shoottype = (rangetype) old_shoottype;
1601 1385
1602 return success; 1386 return success;
1603} 1387}
1604 1388
1605
1606/* This is called from time.c/process_object(). That function 1389/* This is called from time.c/process_object(). That function
1607 * calls this for any SPELL_EFFECT type objects. This function 1390 * calls this for any SPELL_EFFECT type objects. This function
1608 * then dispatches them to the appropriate specific routines. 1391 * then dispatches them to the appropriate specific routines.
1609 */ 1392 */
1610void 1393void
1611move_spell_effect (object *op) 1394move_spell_effect (object *op)
1612{ 1395{
1670 check_bullet (op); 1453 check_bullet (op);
1671 return; 1454 return;
1672 } 1455 }
1673} 1456}
1674 1457
1675/* This is called by move_apply. Basically, if someone 1458/* This is called by move_apply. Basically, if someone
1676 * moves onto a spell effect and the walk_on or fly_on flags 1459 * moves onto a spell effect and the walk_on or fly_on flags
1677 * are set, this is called. This should only be called for 1460 * are set, this is called. This should only be called for
1678 * objects of the appropraite type. 1461 * objects of the appropriate type.
1679 */ 1462 */
1680void 1463void
1681apply_spell_effect (object *spell, object *victim) 1464apply_spell_effect (object *spell, object *victim)
1682{ 1465{
1683 switch (spell->subtype) 1466 switch (spell->subtype)
1689 1472
1690 case SP_MAGIC_MISSILE: 1473 case SP_MAGIC_MISSILE:
1691 if (QUERY_FLAG (victim, FLAG_ALIVE)) 1474 if (QUERY_FLAG (victim, FLAG_ALIVE))
1692 { 1475 {
1693 hit_player (victim, spell->stats.dam, spell, spell->attacktype, 1); 1476 hit_player (victim, spell->stats.dam, spell, spell->attacktype, 1);
1694
1695 if (!spell->destroyed ()) 1477 spell->destroy (true);
1696 {
1697 remove_ob (spell);
1698 free_object (spell);
1699 }
1700 } 1478 }
1701 break; 1479 break;
1702 1480
1703 case SP_MOVING_BALL: 1481 case SP_MOVING_BALL:
1704 if (QUERY_FLAG (victim, FLAG_ALIVE)) 1482 if (QUERY_FLAG (victim, FLAG_ALIVE))
1705 hit_player (victim, spell->stats.dam, spell, spell->attacktype, 1); 1483 hit_player (victim, spell->stats.dam, spell, spell->attacktype, 1);
1706 else if (victim->material || victim->materialname) 1484 else if (victim->materialname)
1707 save_throw_object (victim, spell->attacktype, spell); 1485 save_throw_object (victim, spell->attacktype, spell);
1486
1708 break; 1487 break;
1709 } 1488 }
1710} 1489}
1490

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines