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

Comparing deliantra/server/server/apply.C (file contents):
Revision 1.120 by root, Tue Jul 31 17:33:15 2007 UTC vs.
Revision 1.178 by root, Sun Dec 28 15:28:47 2008 UTC

1/* 1/*
2 * This file is part of Crossfire TRT, the Roguelike Realtime MORPG. 2 * This file is part of Deliantra, the Roguelike Realtime MMORPG.
3 * 3 *
4 * Copyright (©) 2005,2006,2007 Marc Alexander Lehmann / Robin Redeker / the Crossfire TRT team 4 * Copyright (©) 2005,2006,2007,2008 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
5 * Copyright (©) 2001,2007 Mark Wedel & Crossfire Development Team 5 * Copyright (©) 2001,2007 Mark Wedel & Crossfire Development Team
6 * Copyright (©) 1992,2007 Frank Tore Johansen 6 * Copyright (©) 1992,2007 Frank Tore Johansen
7 * 7 *
8 * Crossfire TRT is free software: you can redistribute it and/or modify 8 * Deliantra is free software: you can redistribute it and/or modify
9 * 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
10 * the Free Software Foundation, either version 3 of the License, or 10 * the Free Software Foundation, either version 3 of the License, or
11 * (at your option) any later version. 11 * (at your option) any later version.
12 * 12 *
13 * 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,
… …
16 * GNU General Public License for more details. 16 * GNU General Public License for more details.
17 * 17 *
18 * 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
19 * along with this program. If not, see <http://www.gnu.org/licenses/>. 19 * along with this program. If not, see <http://www.gnu.org/licenses/>.
20 * 20 *
21 * The authors can be reached via e-mail to <crossfire@schmorp.de> 21 * The authors can be reached via e-mail to <support@deliantra.net>
22 */ 22 */
23 23
24#include <cmath> 24#include <cmath>
25 25
26#include <global.h> 26#include <global.h>
… …
53 * subtype 5: arch or race 53 * subtype 5: arch or race
54 * subtype 7: all three 54 * subtype 7: all three
55 */ 55 */
56 if (op->subtype) 56 if (op->subtype)
57 { 57 {
58 arch_flag = (op->subtype & 1); 58 arch_flag = op->subtype & 1;
59 name_flag = (op->subtype & 2); 59 name_flag = op->subtype & 2;
60 race_flag = (op->subtype & 4); 60 race_flag = op->subtype & 4;
61 } 61 }
62 else 62 else
63 { 63 {
64 arch_flag = 1; 64 arch_flag = 1;
65 name_flag = 1; 65 name_flag = 1;
… …
92 * Return value: 1 if money was destroyed, 0 if not. 92 * Return value: 1 if money was destroyed, 0 if not.
93 */ 93 */
94static int 94static int
95apply_id_altar (object *money, object *altar, object *pl) 95apply_id_altar (object *money, object *altar, object *pl)
96{ 96{
97 object *id, *marked; 97 dynbuf_text buf;
98 int success = 0;
99 98
100 if (pl == NULL || pl->type != PLAYER) 99 if (!pl || pl->type != PLAYER)
101 return 0; 100 return 0;
102 101
103 /* Check for MONEY type is a special hack - it prevents 'nothing needs 102 /* Check for MONEY type is a special hack - it prevents 'nothing needs
104 * identifying' from being printed out more than it needs to be. 103 * identifying' from being printed out more than it needs to be.
105 */ 104 */
106 if (!check_altar_sacrifice (altar, money) || money->type != MONEY) 105 if (!check_altar_sacrifice (altar, money) || money->type != MONEY)
107 return 0; 106 return 0;
108 107
109 marked = find_marked_object (pl);
110 /* if the player has a marked item, identify that if it needs to be 108 /* if the player has a marked item, identify that if it needs to be
111 * identified. IF it doesn't, then go through the player inventory. 109 * identified. If it doesn't, then go through the player inventory.
112 */ 110 */
111 if (object *marked = find_marked_object (pl))
113 if (marked && !QUERY_FLAG (marked, FLAG_IDENTIFIED) && need_identify (marked)) 112 if (!QUERY_FLAG (marked, FLAG_IDENTIFIED) && need_identify (marked))
114 { 113 {
115 if (operate_altar (altar, &money)) 114 if (operate_altar (altar, &money))
116 { 115 {
117 identify (marked); 116 identify (marked);
118 new_draw_info_format (NDI_UNIQUE, 0, pl, "You have %s.", long_desc (marked, pl)); 117
118 buf.printf ("You have %s.\r", long_desc (marked, pl));
119 if (marked->msg) 119 if (marked->msg)
120 { 120 buf << "The item has a story:\r" << marked->msg << "\n\n";
121 new_draw_info (NDI_UNIQUE, 0, pl, "The item has a story:"); 121
122 new_draw_info (NDI_UNIQUE, 0, pl, marked->msg); 122 return !money;
123 } 123 }
124 return money == NULL;
125 } 124 }
126 }
127 125
128 for (id = pl->inv; id; id = id->below) 126 for (object *id = pl->inv; id; id = id->below)
129 { 127 {
130 if (!QUERY_FLAG (id, FLAG_IDENTIFIED) && !id->invisible && need_identify (id)) 128 if (!QUERY_FLAG (id, FLAG_IDENTIFIED) && !id->invisible && need_identify (id))
131 { 129 {
132 if (operate_altar (altar, &money)) 130 if (operate_altar (altar, &money))
133 { 131 {
134 identify (id); 132 identify (id);
135 new_draw_info_format (NDI_UNIQUE, 0, pl, "You have %s.", long_desc (id, pl)); 133
134 buf.printf ("You have %s.\r", long_desc (id, pl));
136 if (id->msg) 135 if (id->msg)
137 { 136 buf << "The item has a story:\r" << id->msg << "\n\n";
138 new_draw_info (NDI_UNIQUE, 0, pl, "The item has a story:"); 137
139 new_draw_info (NDI_UNIQUE, 0, pl, id->msg);
140 }
141 success = 1;
142 /* If no more money, might as well quit now */ 138 /* If no more money, might as well quit now */
143 if (money == NULL || !check_altar_sacrifice (altar, money)) 139 if (!money || !check_altar_sacrifice (altar, money))
144 break; 140 break;
145 } 141 }
146 else 142 else
147 { 143 {
148 LOG (llevError, "check_id_altar: Couldn't do sacrifice when we should have been able to\n"); 144 LOG (llevError, "check_id_altar: Couldn't do sacrifice when we should have been able to\n");
149 break; 145 break;
150 } 146 }
151 } 147 }
152 } 148 }
153 if (!success) 149
154 new_draw_info (NDI_UNIQUE, 0, pl, "You have nothing that needs identifying"); 150 if (buf.empty ())
151 pl->failmsg ("You have nothing that needs identifying");
152 else
153 pl->contr->infobox (MSG_CHANNEL ("identify"), buf);
154
155 return money == NULL; 155 return !money;
156} 156}
157 157
158/** 158/**
159 * This checks whether the object has a "on_use_yield" field, and if so generated and drops 159 * This checks whether the object has a "on_use_yield" field, and if so generated and drops
160 * matching item. 160 * matching item.
161 **/ 161 **/
162void 162void
163handle_apply_yield (object *tmp) 163handle_apply_yield (object *tmp)
164{ 164{
165 const char *yield; 165 if (shstr yield = tmp->kv (shstr_on_use_yield))
166 166 archetype::get (yield)->insert_at (tmp, tmp, INS_BELOW_ORIGINATOR);
167 yield = get_ob_key_value (tmp, "on_use_yield");
168 if (yield != NULL)
169 {
170 object *drop = get_archetype (yield);
171
172 if (tmp->env)
173 {
174 drop = insert_ob_in_ob (drop, tmp->env);
175 if (tmp->env->type == PLAYER)
176 esrv_send_item (tmp->env, drop);
177 }
178 else
179 {
180 drop->x = tmp->x;
181 drop->y = tmp->y;
182 insert_ob_in_map (drop, tmp->map, tmp, INS_BELOW_ORIGINATOR);
183 }
184 }
185} 167}
186 168
187/** 169/**
188 * Handles applying a potion. 170 * Handles applying a potion.
189 */ 171 */
… …
195 177
196 floor = GET_MAP_OB (op->map, op->x, op->y); 178 floor = GET_MAP_OB (op->map, op->x, op->y);
197 179
198 if (get_map_flags (op->map, NULL, op->x, op->y, NULL, NULL) & P_SAFE) 180 if (get_map_flags (op->map, NULL, op->x, op->y, NULL, NULL) & P_SAFE)
199 { 181 {
200 if (op->type == PLAYER)
201 new_draw_info (NDI_UNIQUE, 0, op, "Gods prevent you from using this here, it's sacred ground!"); 182 op->failmsg ("Gods prevent you from using this here, it's sacred ground!");
183
202 CLEAR_FLAG (tmp, FLAG_APPLIED); 184 CLEAR_FLAG (tmp, FLAG_APPLIED);
203 return 0; 185 return 0;
204 } 186 }
205 187
206 if (op->type == PLAYER) 188 if (op->type == PLAYER)
… …
217 199
218 if (QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED)) 200 if (QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED))
219 { 201 {
220 op->drain_stat (); 202 op->drain_stat ();
221 op->update_stats (); 203 op->update_stats ();
222 decrease_ob (tmp); 204 tmp->decrease ();
223 return 1; 205 return 1;
224 } 206 }
225 207
226 if (!(at = archetype::find (ARCH_DEPLETION))) 208 if (!(at = archetype::find (ARCH_DEPLETION)))
227 { 209 {
228 LOG (llevError, "Could not find archetype depletion\n"); 210 LOG (llevError, "Could not find archetype depletion\n");
229 return 0; 211 return 0;
230 } 212 }
213
231 depl = present_arch_in_ob (at, op); 214 depl = present_arch_in_ob (at, op);
232 215
233 if (depl) 216 if (depl)
234 { 217 {
235 for (i = 0; i < NUM_STATS; i++) 218 for (i = 0; i < NUM_STATS; i++)
236 if (depl->stats.stat (i)) 219 if (depl->stats.stat (i))
237 new_draw_info (NDI_UNIQUE, 0, op, restore_msg[i]); 220 op->statusmsg (restore_msg[i]);
238 221
239 depl->destroy (); 222 depl->destroy ();
240 op->update_stats (); 223 op->update_stats ();
241 } 224 }
242 else 225 else
243 new_draw_info (NDI_UNIQUE, 0, op, "You potion had no effect."); 226 op->statusmsg ("Your potion had no effect.");
244 227
245 decrease_ob (tmp); 228 tmp->decrease ();
246 return 1; 229 return 1;
247 } 230 }
248 231
249 /* improvement potion - only for players */ 232 /* improvement potion - only for players */
250 if (op->type == PLAYER && tmp->attacktype & AT_GODPOWER) 233 if (op->type == PLAYER && (tmp->attacktype & AT_GODPOWER))
251 { 234 {
252 for (i = 1; i < MIN (11, op->level); i++) 235 for (i = 1; i < MIN (11, op->level); i++)
253 { 236 {
254 if (QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED)) 237 if (QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED))
255 { 238 {
256 if (op->contr->levhp[i] != 1) 239 if (op->contr->levhp[i] != 1)
257 { 240 {
258 op->contr->levhp[i] = 1; 241 op->contr->levhp[i] = 1;
259 break; 242 break;
260 } 243 }
244
261 if (op->contr->levsp[i] != 1) 245 if (op->contr->levsp[i] != 1)
262 { 246 {
263 op->contr->levsp[i] = 1; 247 op->contr->levsp[i] = 1;
264 break; 248 break;
265 } 249 }
250
266 if (op->contr->levgrace[i] != 1) 251 if (op->contr->levgrace[i] != 1)
267 { 252 {
268 op->contr->levgrace[i] = 1; 253 op->contr->levgrace[i] = 1;
269 break; 254 break;
270 } 255 }
… …
274 if (op->contr->levhp[i] < 9) 259 if (op->contr->levhp[i] < 9)
275 { 260 {
276 op->contr->levhp[i] = 9; 261 op->contr->levhp[i] = 9;
277 break; 262 break;
278 } 263 }
264
279 if (op->contr->levsp[i] < 6) 265 if (op->contr->levsp[i] < 6)
280 { 266 {
281 op->contr->levsp[i] = 6; 267 op->contr->levsp[i] = 6;
282 break; 268 break;
283 } 269 }
270
284 if (op->contr->levgrace[i] < 3) 271 if (op->contr->levgrace[i] < 3)
285 { 272 {
286 op->contr->levgrace[i] = 3; 273 op->contr->levgrace[i] = 3;
287 break; 274 break;
288 } 275 }
… …
296 if (!QUERY_FLAG (tmp, FLAG_CURSED) && !QUERY_FLAG (tmp, FLAG_DAMNED)) 283 if (!QUERY_FLAG (tmp, FLAG_CURSED) && !QUERY_FLAG (tmp, FLAG_DAMNED))
297 { 284 {
298 if (got_one) 285 if (got_one)
299 { 286 {
300 op->update_stats (); 287 op->update_stats ();
301 new_draw_info (NDI_UNIQUE, 0, op, "The Gods smile upon you and remake you"); 288 op->statusmsg ("The Gods smile upon you and remake you "
302 new_draw_info (NDI_UNIQUE, 0, op, "a little more in their image."); 289 "a little more in their image. "
303 new_draw_info (NDI_UNIQUE, 0, op, "You feel a little more perfect."); 290 "You feel a little more perfect.", NDI_GREEN);
304 } 291 }
305 else 292 else
306 new_draw_info (NDI_UNIQUE, 0, op, "The potion had no effect - you are already perfect"); 293 op->statusmsg ("The potion had no effect - you are already perfect.");
307 } 294 }
308 else 295 else
309 { /* cursed potion */ 296 { /* cursed potion */
310 if (got_one) 297 if (got_one)
311 { 298 {
312 op->update_stats (); 299 op->update_stats ();
313 new_draw_info (NDI_UNIQUE, 0, op, "The Gods are angry and punish you."); 300 op->failmsg ("The Gods are angry and punish you.");
314 } 301 }
315 else 302 else
316 new_draw_info (NDI_UNIQUE, 0, op, "You are fortunate that you are so pathetic."); 303 op->statusmsg ("You are fortunate that you are so pathetic.", NDI_DK_ORANGE);
317 } 304 }
318 305
319 decrease_ob (tmp); 306 tmp->decrease ();
320 return 1; 307 return 1;
321 } 308 }
322 309
323 310
324 /* A potion that casts a spell. Healing, restore spellpoint (power potion) 311 /* A potion that casts a spell. Healing, restore spellpoint (power potion)
… …
330 { 317 {
331 if (QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED)) 318 if (QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED))
332 { 319 {
333 object *fball; 320 object *fball;
334 321
335 new_draw_info (NDI_UNIQUE, 0, op, "Yech! Your lungs are on fire!"); 322 op->failmsg ("Yech! Your lungs are on fire!");
323
336 /* Explodes a fireball centered at player */ 324 /* Explodes a fireball centered at player */
337 fball = get_archetype (EXPLODING_FIREBALL); 325 fball = get_archetype (EXPLODING_FIREBALL);
338 fball->dam_modifier = random_roll (1, op->level, op, PREFER_LOW) / 5 + 1; 326 fball->dam_modifier = random_roll (1, op->level, op, PREFER_LOW) / 5 + 1;
339 fball->stats.maxhp = random_roll (1, op->level, op, PREFER_LOW) / 10 + 2; 327 fball->stats.maxhp = random_roll (1, op->level, op, PREFER_LOW) / 10 + 2;
340 fball->x = op->x; 328 fball->x = op->x;
… …
342 insert_ob_in_map (fball, op->map, NULL, 0); 330 insert_ob_in_map (fball, op->map, NULL, 0);
343 } 331 }
344 else 332 else
345 cast_spell (op, tmp, op->facing, tmp->inv, NULL); 333 cast_spell (op, tmp, op->facing, tmp->inv, NULL);
346 334
347 decrease_ob (tmp); 335 tmp->decrease ();
336
348 /* if youre dead, no point in doing this... */ 337 /* if youre dead, no point in doing this... */
349 if (!QUERY_FLAG (op, FLAG_REMOVED)) 338 if (!QUERY_FLAG (op, FLAG_REMOVED))
350 op->update_stats (); 339 op->update_stats ();
340
351 return 1; 341 return 1;
352 } 342 }
353 343
354 /* Deal with protection potions */ 344 /* Deal with protection potions */
355 force = NULL; 345 force = NULL;
… …
357 { 347 {
358 if (tmp->resist[i]) 348 if (tmp->resist[i])
359 { 349 {
360 if (!force) 350 if (!force)
361 force = get_archetype (FORCE_NAME); 351 force = get_archetype (FORCE_NAME);
352
362 memcpy (force->resist, tmp->resist, sizeof (tmp->resist)); 353 memcpy (force->resist, tmp->resist, sizeof (tmp->resist));
363 force->type = POTION_EFFECT; 354 force->type = POTION_EFFECT;
364 break; /* Only need to find one protection since we copy entire batch */ 355 break; /* Only need to find one protection since we copy entire batch */
365 } 356 }
366 } 357 }
358
367 /* This is a protection potion */ 359 /* This is a protection potion */
368 if (force) 360 if (force)
369 { 361 {
370 /* cursed items last longer */ 362 /* cursed items last longer */
371 if (QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED)) 363 if (QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED))
… …
379 force->speed_left = -1; 371 force->speed_left = -1;
380 force = insert_ob_in_ob (force, op); 372 force = insert_ob_in_ob (force, op);
381 CLEAR_FLAG (tmp, FLAG_APPLIED); 373 CLEAR_FLAG (tmp, FLAG_APPLIED);
382 SET_FLAG (force, FLAG_APPLIED); 374 SET_FLAG (force, FLAG_APPLIED);
383 change_abil (op, force); 375 change_abil (op, force);
384 decrease_ob (tmp); 376 tmp->decrease ();
385 return 1; 377 return 1;
386 } 378 }
387 379
388 /* Only thing left are the stat potions */ 380 /* Only thing left are the stat potions */
389 if (op->type == PLAYER) 381 if (op->type == PLAYER)
390 { /* only for players */ 382 { /* only for players */
391 if ((QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED)) && tmp->value != 0) 383 if ((QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED)) && tmp->value != 0)
392 CLEAR_FLAG (tmp, FLAG_APPLIED); 384 CLEAR_FLAG (tmp, FLAG_APPLIED);
393 else 385 else
394 SET_FLAG (tmp, FLAG_APPLIED); 386 SET_FLAG (tmp, FLAG_APPLIED);
387
395 if (!change_abil (op, tmp)) 388 if (!change_abil (op, tmp))
396 new_draw_info (NDI_UNIQUE, 0, op, "Nothing happened."); 389 op->statusmsg ("Nothing happened.");
397 } 390 }
398 391
399 /* CLEAR_FLAG is so that if the character has other potions 392 /* CLEAR_FLAG is so that if the character has other potions
400 * that were grouped with the one consumed, his 393 * that were grouped with the one consumed, his
401 * stat will not be raised by them. fix_player just clears 394 * stat will not be raised by them. fix_player just clears
402 * up all the stats. 395 * up all the stats.
403 */ 396 */
404 CLEAR_FLAG (tmp, FLAG_APPLIED); 397 CLEAR_FLAG (tmp, FLAG_APPLIED);
405 op->update_stats (); 398 op->update_stats ();
406 decrease_ob (tmp); 399 tmp->decrease ();
407 return 1; 400 return 1;
408} 401}
409 402
410/**************************************************************************** 403/****************************************************************************
411 * Weapon improvement code follows 404 * Weapon improvement code follows
… …
417static int 410static int
418check_item (object *op, const char *item) 411check_item (object *op, const char *item)
419{ 412{
420 int count = 0; 413 int count = 0;
421 414
422 415 if (!item)
423 if (item == NULL)
424 return 0; 416 return 0;
425 417
426 op = op->below; 418 for (op = op->below; op; op = op->below)
427 while (op != NULL)
428 { 419 {
429 if (strcmp (op->arch->archname, item) == 0) 420 if (strcmp (op->arch->archname, item) == 0)
430 { 421 {
431 if (!QUERY_FLAG (op, FLAG_CURSED) && !QUERY_FLAG (op, FLAG_DAMNED) 422 if (!QUERY_FLAG (op, FLAG_CURSED) && !QUERY_FLAG (op, FLAG_DAMNED)
432 /* Loophole bug? -FD- */ && !QUERY_FLAG (op, FLAG_UNPAID)) 423 /* Loophole bug? -FD- */ && !QUERY_FLAG (op, FLAG_UNPAID))
… …
435 count++; 426 count++;
436 else 427 else
437 count += op->nrof; 428 count += op->nrof;
438 } 429 }
439 } 430 }
440
441 op = op->below;
442 } 431 }
443 432
444 return count; 433 return count;
445} 434}
446 435
… …
456 object *prev; 445 object *prev;
457 446
458 prev = op; 447 prev = op;
459 op = op->below; 448 op = op->below;
460 449
461 while (op != NULL) 450 while (op)
462 { 451 {
463 if (strcmp (op->arch->archname, item) == 0) 452 if (strcmp (op->arch->archname, item) == 0)
464 { 453 {
465 if (op->nrof >= nrof) 454 if (op->nrof >= nrof)
466 { 455 {
467 decrease_ob_nr (op, nrof); 456 op->decrease (nrof);
468 return; 457 return;
469 } 458 }
470 else 459 else
471 { 460 {
472 decrease_ob_nr (op, op->nrof); 461 op->decrease (nrof);
473 nrof -= op->nrof; 462 nrof -= op->nrof;
474 } 463 }
464
475 op = prev; 465 op = prev;
476 } 466 }
467
477 prev = op; 468 prev = op;
478 op = op->below; 469 op = op->below;
479 } 470 }
480} 471}
481 472
… …
487 * we return 1 (true) if the player can use the weapon. 478 * we return 1 (true) if the player can use the weapon.
488 */ 479 */
489static int 480static int
490check_weapon_power (const object *who, int improvs) 481check_weapon_power (const object *who, int improvs)
491{ 482{
492
493/* Old code is below (commented out). Basically, since weapons are the only 483/* Old code is below (commented out). Basically, since weapons are the only
494 * object players really have any control to improve, it's a bit harsh to 484 * object players really have any control to improve, it's a bit harsh to
495 * require high level in some combat skill, so we just use overall level. 485 * require high level in some combat skill, so we just use overall level.
496 */ 486 */
497#if 1 487#if 1
… …
538static int 528static int
539check_sacrifice (object *op, const object *improver) 529check_sacrifice (object *op, const object *improver)
540{ 530{
541 int count = 0; 531 int count = 0;
542 532
543 if (improver->slaying != NULL) 533 if (improver->slaying)
544 { 534 {
545 count = check_item (op, improver->slaying); 535 count = check_item (op, improver->slaying);
546 if (count < 1) 536 if (count < 1)
547 { 537 {
548 char buf[200];
549
550 sprintf (buf, "The gods want more %ss", &improver->slaying); 538 op->failmsg (format ("The gods want more %ss", &improver->slaying));
551 new_draw_info (NDI_UNIQUE, 0, op, buf);
552 return 0; 539 return 0;
553 } 540 }
554 } 541 }
555 else 542 else
556 count = 1; 543 count = 1;
… …
559} 546}
560 547
561/** 548/**
562 * Actually improves the weapon, and tells user. 549 * Actually improves the weapon, and tells user.
563 */ 550 */
564int 551static int
565improve_weapon_stat (object *op, object *improver, object *weapon, signed char *stat, int sacrifice_count, const char *statname) 552improve_weapon_stat (object *op, object *improver, object *weapon, sint8 &stat, int sacrifice_count, const char *statname)
566{ 553{
567
568 new_draw_info (NDI_UNIQUE, 0, op, "Your sacrifice was accepted.");
569 *stat += sacrifice_count; 554 stat += sacrifice_count;
570 weapon->last_eat++; 555 weapon->last_eat++;
571 new_draw_info_format (NDI_UNIQUE, 0, op, "Weapon's bonus to %s improved by %d", statname, sacrifice_count); 556 improver->decrease ();
572 decrease_ob (improver);
573 557
574 /* So it updates the players stats and the window */ 558 /* So it updates the players stats and the window */
575 op->update_stats (); 559 op->update_stats ();
560
561 op->statusmsg (format (
562 "Your sacrifice was accepted.\n"
563 "Weapon's bonus to %s improved by %d.",
564 statname, sacrifice_count
565 ));
566
576 return 1; 567 return 1;
577} 568}
578 569
579/* Types of improvements, hidden in the sp field. */ 570/* Types of improvements, hidden in the sp field. */
580#define IMPROVE_PREPARE 1 571#define IMPROVE_PREPARE 1
… …
599 int sacrifice_count, i; 590 int sacrifice_count, i;
600 char buf[MAX_BUF]; 591 char buf[MAX_BUF];
601 592
602 if (weapon->level != 0) 593 if (weapon->level != 0)
603 { 594 {
604 new_draw_info (NDI_UNIQUE, 0, op, "Weapon already prepared."); 595 op->failmsg ("Weapon is already prepared!");
605 return 0; 596 return 0;
606 } 597 }
607 598
608 for (i = 0; i < NROFATTACKS; i++) 599 for (i = 0; i < NROFATTACKS; i++)
609 if (weapon->resist[i]) 600 if (weapon->resist[i])
… …
615 if (i < NROFATTACKS || weapon->stats.hp || /* regeneration */ 606 if (i < NROFATTACKS || weapon->stats.hp || /* regeneration */
616 (weapon->stats.sp && weapon->type == WEAPON) || /* sp regeneration */ 607 (weapon->stats.sp && weapon->type == WEAPON) || /* sp regeneration */
617 weapon->stats.exp || /* speed */ 608 weapon->stats.exp || /* speed */
618 weapon->stats.ac) /* AC - only taifu's I think */ 609 weapon->stats.ac) /* AC - only taifu's I think */
619 { 610 {
620 new_draw_info (NDI_UNIQUE, 0, op, "Cannot prepare magic weapons."); 611 op->failmsg ("You cannot prepare magic weapons. "
612 "H<A weapon is considered magical if it changes regeneration, "
613 "speed or ac, or has other protections.>");
621 return 0; 614 return 0;
622 } 615 }
623 616
624 sacrifice_count = check_sacrifice (op, improver); 617 sacrifice_count = check_sacrifice (op, improver);
625 if (sacrifice_count <= 0) 618 if (sacrifice_count <= 0)
626 return 0; 619 return 0;
627 620
628 weapon->level = isqrt (sacrifice_count); 621 weapon->level = isqrt (sacrifice_count);
629 new_draw_info (NDI_UNIQUE, 0, op, "Your sacrifice was accepted.");
630 eat_item (op, improver->slaying, sacrifice_count); 622 eat_item (op, improver->slaying, sacrifice_count);
631 623
632 new_draw_info_format (NDI_UNIQUE, 0, op, "Your *%s may be improved %d times.", &weapon->name, weapon->level); 624 op->statusmsg (format (
625 "Your sacrifice was accepted."
626 "Your *%s may be improved %d times.",
627 &weapon->name, weapon->level
628 ));
633 629
634 sprintf (buf, "%s's %s", &op->name, &weapon->name); 630 sprintf (buf, "%s's %s", &op->name, &weapon->name);
635 weapon->name = weapon->name_pl = buf; 631 weapon->name = weapon->name_pl = buf;
636 weapon->nrof = 0; /* prevents preparing n weapons in the same 632 weapon->nrof = 0; /* prevents preparing n weapons in the same
637 slot at once! */ 633 slot at once! */
638 decrease_ob (improver); 634 improver->decrease ();
639 weapon->last_eat = 0; 635 weapon->last_eat = 0;
640 return 1; 636 return 1;
641} 637}
642
643 638
644/** 639/**
645 * Does the dirty job for 'improve weapon' scroll, prepare or add something. 640 * Does the dirty job for 'improve weapon' scroll, prepare or add something.
646 * This is the new improve weapon code. 641 * This is the new improve weapon code.
647 * Returns 0 if it was not able to work for some reason. 642 * Returns 0 if it was not able to work for some reason.
… …
656improve_weapon (object *op, object *improver, object *weapon) 651improve_weapon (object *op, object *improver, object *weapon)
657{ 652{
658 int sacrifice_count, sacrifice_needed = 0; 653 int sacrifice_count, sacrifice_needed = 0;
659 654
660 if (improver->stats.sp == IMPROVE_PREPARE) 655 if (improver->stats.sp == IMPROVE_PREPARE)
661 {
662 return prepare_weapon (op, improver, weapon); 656 return prepare_weapon (op, improver, weapon);
663 } 657
664 if (weapon->level == 0) 658 if (weapon->level == 0)
665 { 659 {
666 new_draw_info (NDI_UNIQUE, 0, op, "This weapon has not been prepared."); 660 op->failmsg ("This weapon has not been prepared. H<You first ahve to prepare a weapon with a prepare weapon scroll.>");
667 return 0; 661 return 0;
668 } 662 }
663
669 if (weapon->level == weapon->last_eat && weapon->item_power >= 100) 664 if (weapon->level == weapon->last_eat && weapon->item_power >= 100)
670 { 665 {
671 new_draw_info (NDI_UNIQUE, 0, op, "This weapon cannot be improved any more."); 666 op->failmsg ("This weapon cannot be improved any more.");
672 return 0; 667 return 0;
673 } 668 }
669
674 if (QUERY_FLAG (weapon, FLAG_APPLIED) && !check_weapon_power (op, weapon->last_eat + 1)) 670 if (QUERY_FLAG (weapon, FLAG_APPLIED) && !check_weapon_power (op, weapon->last_eat + 1))
675 { 671 {
676 new_draw_info (NDI_UNIQUE, 0, op, "Improving the weapon will make it too"); 672 op->failmsg ("Improving the weapon will make it too "
677 new_draw_info (NDI_UNIQUE, 0, op, "powerful for you to use. Unready it if you"); 673 "powerful for you to use. Unready it if you "
678 new_draw_info (NDI_UNIQUE, 0, op, "really want to improve it."); 674 "really want to improve it.");
679 return 0; 675 return 0;
680 } 676 }
677
681 /* This just increases damage by 5 points, no matter what. No sacrifice 678 /* This just increases damage by 5 points, no matter what. No sacrifice
682 * is needed. Since stats.dam is now a 16 bit value and not 8 bit, 679 * is needed. Since stats.dam is now a 16 bit value and not 8 bit,
683 * don't put any maximum value on damage - the limit is how much the 680 * don't put any maximum value on damage - the limit is how much the
684 * weapon can be improved. 681 * weapon can be improved.
685 */ 682 */
686 if (improver->stats.sp == IMPROVE_DAMAGE) 683 if (improver->stats.sp == IMPROVE_DAMAGE)
687 { 684 {
688 weapon->stats.dam += 5; 685 weapon->stats.dam += 5;
689 weapon->weight += 5000; /* 5 KG's */ 686 weapon->weight += 5000; /* 5 KG's */
690 new_draw_info_format (NDI_UNIQUE, 0, op, "Damage has been increased by 5 to %d", weapon->stats.dam); 687 op->statusmsg (format ("Damage has been increased by 5 to %d.", weapon->stats.dam));
691 weapon->last_eat++; 688 weapon->last_eat++;
692 689
693 weapon->item_power++; 690 weapon->item_power++;
694 decrease_ob (improver); 691 improver->decrease ();
695 return 1; 692 return 1;
696 } 693 }
694
697 if (improver->stats.sp == IMPROVE_WEIGHT) 695 if (improver->stats.sp == IMPROVE_WEIGHT)
698 { 696 {
699 /* Reduce weight by 20% */ 697 /* Reduce weight by 20% */
700 weapon->weight = (weapon->weight * 8) / 10; 698 weapon->weight = (weapon->weight * 8) / 10;
701 if (weapon->weight < 1) 699 if (weapon->weight < 1)
702 weapon->weight = 1; 700 weapon->weight = 1;
701
703 new_draw_info_format (NDI_UNIQUE, 0, op, "Weapon weight reduced to %6.1f kg", (float) weapon->weight / 1000.0); 702 op->statusmsg (format ("Weapon weight reduced to %6.1fkg.", (float) weapon->weight / 1000.0));
704 weapon->last_eat++; 703 weapon->last_eat++;
705 weapon->item_power++; 704 weapon->item_power++;
706 decrease_ob (improver); 705 improver->decrease ();
707 return 1; 706 return 1;
708 } 707 }
708
709 if (improver->stats.sp == IMPROVE_ENCHANT) 709 if (improver->stats.sp == IMPROVE_ENCHANT)
710 { 710 {
711 weapon->magic++; 711 weapon->magic++;
712 weapon->last_eat++; 712 weapon->last_eat++;
713 new_draw_info_format (NDI_UNIQUE, 0, op, "Weapon magic increased to %d", weapon->magic); 713 op->statusmsg (format ("Weapon magic increased to %d.", weapon->magic));
714 decrease_ob (improver); 714 improver->decrease ();
715 weapon->item_power++; 715 weapon->item_power++;
716 return 1; 716 return 1;
717 } 717 }
718 718
719 sacrifice_needed = weapon->stats.Str + weapon->stats.Int + weapon->stats.Dex + 719 sacrifice_needed = weapon->stats.Str + weapon->stats.Int + weapon->stats.Dex +
… …
724 sacrifice_needed *= 2; 724 sacrifice_needed *= 2;
725 725
726 sacrifice_count = check_sacrifice (op, improver); 726 sacrifice_count = check_sacrifice (op, improver);
727 if (sacrifice_count < sacrifice_needed) 727 if (sacrifice_count < sacrifice_needed)
728 { 728 {
729 new_draw_info_format (NDI_UNIQUE, 0, op, "You need at least %d %s", sacrifice_needed, &improver->slaying); 729 op->failmsg (format ("You need at least %d %s.", sacrifice_needed, &improver->slaying));
730 return 0; 730 return 0;
731 } 731 }
732
732 eat_item (op, improver->slaying, sacrifice_needed); 733 eat_item (op, improver->slaying, sacrifice_needed);
733 weapon->item_power++; 734 weapon->item_power++;
734 735
735 switch (improver->stats.sp) 736 switch (improver->stats.sp)
736 { 737 {
737 case IMPROVE_STR:
738 return improve_weapon_stat (op, improver, weapon, (signed char *) &(weapon->stats.Str), 1, "strength"); 738 case IMPROVE_STR: return improve_weapon_stat (op, improver, weapon, weapon->stats.Str, 1, "strength");
739 case IMPROVE_DEX:
740 return improve_weapon_stat (op, improver, weapon, (signed char *) &(weapon->stats.Dex), 1, "dexterity"); 739 case IMPROVE_DEX: return improve_weapon_stat (op, improver, weapon, weapon->stats.Dex, 1, "dexterity");
741 case IMPROVE_CON:
742 return improve_weapon_stat (op, improver, weapon, (signed char *) &(weapon->stats.Con), 1, "constitution"); 740 case IMPROVE_CON: return improve_weapon_stat (op, improver, weapon, weapon->stats.Con, 1, "constitution");
743 case IMPROVE_WIS:
744 return improve_weapon_stat (op, improver, weapon, (signed char *) &(weapon->stats.Wis), 1, "wisdom"); 741 case IMPROVE_WIS: return improve_weapon_stat (op, improver, weapon, weapon->stats.Wis, 1, "wisdom");
745 case IMPROVE_CHA:
746 return improve_weapon_stat (op, improver, weapon, (signed char *) &(weapon->stats.Cha), 1, "charisma"); 742 case IMPROVE_CHA: return improve_weapon_stat (op, improver, weapon, weapon->stats.Cha, 1, "charisma");
747 case IMPROVE_INT:
748 return improve_weapon_stat (op, improver, weapon, (signed char *) &(weapon->stats.Int), 1, "intelligence"); 743 case IMPROVE_INT: return improve_weapon_stat (op, improver, weapon, weapon->stats.Int, 1, "intelligence");
749 case IMPROVE_POW:
750 return improve_weapon_stat (op, improver, weapon, (signed char *) &(weapon->stats.Pow), 1, "power"); 744 case IMPROVE_POW: return improve_weapon_stat (op, improver, weapon, weapon->stats.Pow, 1, "power");
751 default: 745 default:
752 new_draw_info (NDI_UNIQUE, 0, op, "Unknown improvement type."); 746 op->failmsg ("Unknown improvement type.");
753 } 747 }
748
754 LOG (llevError, "improve_weapon: Got to end of function\n"); 749 LOG (llevError, "improve_weapon: Got to end of function\n");
755 return 0; 750 return 0;
756} 751}
757 752
758/** 753/**
… …
768 if (op->type != PLAYER) 763 if (op->type != PLAYER)
769 return 0; 764 return 0;
770 765
771 if (!QUERY_FLAG (op, FLAG_WIZCAST) && (get_map_flags (op->map, NULL, op->x, op->y, NULL, NULL) & P_NO_MAGIC)) 766 if (!QUERY_FLAG (op, FLAG_WIZCAST) && (get_map_flags (op->map, NULL, op->x, op->y, NULL, NULL) & P_NO_MAGIC))
772 { 767 {
773 new_draw_info (NDI_UNIQUE, 0, op, "Something blocks the magic of the scroll."); 768 op->failmsg ("Something blocks the magic of the scroll!");
774 return 0; 769 return 0;
775 } 770 }
776 771
777 otmp = find_marked_object (op); 772 otmp = find_marked_object (op);
773
778 if (!otmp) 774 if (!otmp)
779 { 775 {
780 new_draw_info (NDI_UNIQUE, 0, op, "You need to mark a weapon object."); 776 op->failmsg ("You need to mark a weapon object. H<Use the mark command or the mark option from the item popup menu.>");
781 return 0; 777 return 0;
782 } 778 }
783 779
784 if (otmp->type != WEAPON && otmp->type != BOW) 780 if (otmp->type != WEAPON && otmp->type != BOW)
785 { 781 {
786 new_draw_info (NDI_UNIQUE, 0, op, "Marked item is not a weapon or bow"); 782 op->failmsg ("Marked item is not a weapon or bow!");
787 return 0; 783 return 0;
788 } 784 }
789 785
790 new_draw_info (NDI_UNIQUE, 0, op, "Applied weapon builder."); 786 op->statusmsg ("Applied weapon builder.");
787
791 improve_weapon (op, tmp, otmp); 788 improve_weapon (op, tmp, otmp);
792 esrv_send_item (op, otmp); 789 esrv_send_item (op, otmp);
793 return 1; 790 return 1;
794} 791}
795 792
… …
820{ 817{
821 object *tmp; 818 object *tmp;
822 819
823 if (armour->magic >= settings.armor_max_enchant) 820 if (armour->magic >= settings.armor_max_enchant)
824 { 821 {
825 new_draw_info (NDI_UNIQUE, 0, op, "This armour can not be enchanted any further."); 822 op->failmsg ("This armour can not be enchanted any further!");
826 return 0; 823 return 0;
827 } 824 }
825
828 /* Dealing with random artifact armor is a lot trickier (in terms of value, weight, 826 /* Dealing with random artifact armor is a lot trickier (in terms of value, weight,
829 * etc), so take the easy way out and don't worry about it. 827 * etc), so take the easy way out and don't worry about it.
830 * Note - maybe add scrolls which make the random artifact versions (eg, armour 828 * Note - maybe add scrolls which make the random artifact versions (eg, armour
831 * of gnarg and what not?) 829 * of gnarg and what not?)
832 */ 830 */
833 if (armour->title) 831 if (armour->title)
834 { 832 {
835 new_draw_info (NDI_UNIQUE, 0, op, "This armour will not accept further enchantment."); 833 op->failmsg ("This armour will not accept further enchantment.");
836 return 0; 834 return 0;
837 } 835 }
838 836
839 /* Split objects if needed. Can't insert tmp until the 837 /* Split objects if needed. Can't insert tmp until the
840 * end of this function - otherwise it will just re-merge. 838 * end of this function - otherwise it will just re-merge.
841 */ 839 */
842 if (armour->nrof > 1) 840 tmp = armour->nrof > 1 ? armour->split (armour->nrof - 1) : 0;
843 tmp = get_split_ob (armour, armour->nrof - 1);
844 else
845 tmp = NULL;
846 841
847 armour->magic++; 842 armour->magic++;
848 843
849 if (!settings.armor_speed_linear) 844 if (!settings.armor_speed_linear)
850 { 845 {
… …
887 armour->item_power = get_power_from_ench (armour->arch->item_power + armour->magic); 882 armour->item_power = get_power_from_ench (armour->arch->item_power + armour->magic);
888 883
889 if (op->type == PLAYER) 884 if (op->type == PLAYER)
890 { 885 {
891 esrv_send_item (op, armour); 886 esrv_send_item (op, armour);
887
892 if (QUERY_FLAG (armour, FLAG_APPLIED)) 888 if (QUERY_FLAG (armour, FLAG_APPLIED))
893 op->update_stats (); 889 op->update_stats ();
894 } 890 }
895 decrease_ob (improver); 891
892 improver->decrease ();
893
896 if (tmp) 894 if (tmp)
897 { 895 op->insert (tmp);
898 insert_ob_in_ob (tmp, op); 896
899 esrv_send_item (op, tmp);
900 }
901 return 1; 897 return 1;
902} 898}
903
904 899
905/* 900/*
906 * convert_item() returns 1 if anything was converted, 0 if the item was not 901 * convert_item() returns 1 if anything was converted, 0 if the item was not
907 * what the converter wants, -1 if the converter is broken. 902 * what the converter wants, -1 if the converter is broken.
908 */ 903 *
909#define CONV_FROM(xyz) xyz->slaying
910#define CONV_TO(xyz) xyz->other_arch
911#define CONV_NR(xyz) xyz->stats.sp
912#define CONV_NEED(xyz) xyz->stats.food
913
914/* Takes one items and makes another. 904 * Takes one type of items and makes another.
915 * converter is the object that is doing the conversion. 905 * converter is the object that is doing the conversion.
916 * item is the object that triggered the converter - if it is not 906 * item is the object that triggered the converter - if it is not
917 * what the converter wants, this will not do anything. 907 * what the converter wants, this will not do anything.
918 */ 908 */
919int 909static int
920convert_item (object *item, object *converter) 910convert_item (object *item, object *converter)
921{ 911{
922 int nr = 0;
923 uint32 price_in; 912 sint64 nr, price_in;
913
914 if (item->flag [FLAG_UNPAID])
915 return 0;
916
917 shstr conv_from = converter->slaying;
918 archetype *conv_to = converter->other_arch;
919 sint64 need = converter->stats.food;
920 sint64 give = converter->stats.sp;
924 921
925 /* We make some assumptions - we assume if it takes money as it type, 922 /* We make some assumptions - we assume if it takes money as it type,
926 * it wants some amount. We don't make change (ie, if something costs 923 * it wants some amount. We don't make change (ie, if something costs
927 * 3 gp and player drops a platinum, tough luck) 924 * 3 gp and player drops a platinum, tough luck)
928 */ 925 */
929 if (!strcmp (CONV_FROM (converter), "money")) 926 if (conv_from == shstr_money)
930 { 927 {
931 int cost;
932
933 if (item->type != MONEY) 928 if (item->type != MONEY)
934 return 0; 929 return 0;
935 930
936 nr = (item->nrof * item->value) / CONV_NEED (converter); 931 nr = sint64 (item->nrof) * item->value / need;
937 if (!nr) 932 if (!nr)
938 return 0; 933 return 0;
939 cost = nr * CONV_NEED (converter) / item->value; 934
940 /* take into account rounding errors */ 935 converter->play_sound (sound_find ("shop_buy"));
941 if (nr * CONV_NEED (converter) % item->value) 936
942 cost++; 937 sint64 cost = (nr * need + item->value - 1) / item->value;
943 decrease_ob_nr (item, cost); 938
939 item->decrease (cost);
944 940
945 price_in = cost * item->value; 941 price_in = cost * item->value;
946 } 942 }
947 else 943 else
948 { 944 {
949 if (item->type == PLAYER || CONV_FROM (converter) != item->arch->archname || 945 if (item->type == PLAYER
950 (CONV_NEED (converter) && CONV_NEED (converter) > (uint16) item->nrof)) 946 || conv_from != item->arch->archname
947 || (need && need > (uint16) item->nrof))
951 return 0; 948 return 0;
952 949
953 if (CONV_NEED (converter)) 950 converter->play_sound (sound_find ("convert_item"));
951
952 if (need)
954 { 953 {
955 nr = item->nrof / CONV_NEED (converter); 954 nr = sint64 (item->nrof) / need;
956 decrease_ob_nr (item, nr * CONV_NEED (converter)); 955 item->decrease (nr * need);
957 price_in = nr * CONV_NEED (converter) * item->value; 956 price_in = nr * need * item->value;
958 } 957 }
959 else 958 else
960 { 959 {
961 price_in = item->value; 960 price_in = item->value;
962 item->destroy (); 961 item->destroy ();
963 } 962 }
964 } 963 }
965 964
966 if (converter->inv != NULL) 965 if (converter->inv)
967 { 966 {
968 object *ob; 967 object *ob;
969 int i; 968 int i;
970 object *ob_to_copy; 969 object *ob_to_copy;
971 970
972 /* select random object from inventory to copy */ 971 /* select random object from inventory to copy */
973 ob_to_copy = converter->inv; 972 ob_to_copy = converter->inv;
974 for (ob = converter->inv->below, i = 1; ob != NULL; ob = ob->below, i++) 973 for (ob = converter->inv->below, i = 1; ob; ob = ob->below, i++)
975 {
976 if (rndm (0, i) == 0) 974 if (rndm (0, i) == 0)
977 {
978 ob_to_copy = ob; 975 ob_to_copy = ob;
979 } 976
980 } 977 item = ob_to_copy->deep_clone ();
981 item = object_create_clone (ob_to_copy);
982 CLEAR_FLAG (item, FLAG_IS_A_TEMPLATE); 978 CLEAR_FLAG (item, FLAG_IS_A_TEMPLATE);
983 unflag_inv (item, FLAG_IS_A_TEMPLATE); 979 unflag_inv (item, FLAG_IS_A_TEMPLATE);
984 } 980 }
985 else 981 else
986 { 982 {
987 if (converter->other_arch == NULL) 983 if (!conv_to)
988 { 984 {
989 LOG (llevError, "move_creator: Converter doesn't have other arch set: %s (%s, %d, %d)\n", 985 LOG (llevError, "move_creator: Converter doesn't have other arch set: %s (%s, %d, %d)\n",
990 &converter->name, &converter->map->path, converter->x, converter->y); 986 &converter->name, &converter->map->path, converter->x, converter->y);
991 return -1; 987 return -1;
992 } 988 }
993 989
994 item = object_create_arch (converter->other_arch); 990 item = object_create_arch (conv_to);
995 fix_generated_item (item, converter, 0, 0, GT_MINIMAL); 991 fix_generated_item (item, converter, 0, 0, GT_MINIMAL);
996 } 992 }
997 993
998 if (CONV_NR (converter)) 994 if (give)
999 item->nrof = CONV_NR (converter); 995 item->nrof = give;
996
1000 if (nr) 997 if (nr)
1001 item->nrof *= nr; 998 item->nrof *= nr;
1002 if (is_in_shop (converter)) 999
1000 if (converter->flag [FLAG_PRECIOUS])
1003 SET_FLAG (item, FLAG_UNPAID); 1001 SET_FLAG (item, FLAG_UNPAID);
1002
1003 if (is_in_shop (converter))
1004 {
1005 // converters on shop floors don't work anymore, bug lets check for it
1006 // and report in case someone still does it.
1007 LOG (llevDebug, "ITEMBUG: broken converter, converters on shop floor don't work: %s\n",
1008 converter->debug_desc ());
1009 SET_FLAG (item, FLAG_UNPAID);
1010 }
1004 else if (price_in < item->nrof * item->value) 1011 else if (price_in < sint64 (item->nrof) * item->value)
1005 { 1012 {
1006 LOG (llevDebug, "converter output price higher than input: %s at %s (%d, %d) in value %d, out value %d for %s\n", 1013 LOG (llevDebug, "converter output price higher than input: %s at %s (%d, %d) in value %d, out value %d for %s\n",
1007 &converter->name, &converter->map->path, converter->x, converter->y, price_in, item->nrof * item->value, &item->name); 1014 &converter->name, &converter->map->path, converter->x, converter->y, price_in, item->nrof * item->value, &item->name);
1008
1009 /** 1015 /**
1010 * elmex: we are going to let the game continue, as the mapcreator 1016 * elmex: we are going to let the game continue, as the mapcreator
1011 * probably had something in mind when doing this 1017 * hopefully had something in mind when doing this.
1012 */ 1018 */
1013 } 1019 }
1014 SET_FLAG (item, FLAG_IDENTIFIED); 1020
1021 // elmex: only identify if we need to, for example so that generated money doesn't
1022 // get an 'identified' flag so easily.
1023 if (need_identify (item))
1024 identify (item);
1025
1015 insert_ob_in_map_at (item, converter->map, converter, 0, converter->x, converter->y); 1026 insert_ob_in_map_at (item, converter->map, converter, 0, converter->x, converter->y);
1016 return 1; 1027 return 1;
1017} 1028}
1018 1029
1019/** 1030/**
… …
1036 1047
1037 op->contr->last_used = 0; 1048 op->contr->last_used = 0;
1038 1049
1039 if (sack->env && sack->env != op) 1050 if (sack->env && sack->env != op)
1040 { 1051 {
1041 new_draw_info (NDI_UNIQUE, 0, op, "You must put it onto the floor or into your inventory first."); 1052 op->failmsg ("You must put it onto the floor or into your inventory first.");
1042 return 1; 1053 return 1;
1043 } 1054 }
1044 1055
1045 // already applied == open on ground, or open in inv, or active in inv 1056 // already applied == open on ground, or open in inv, or active in inv
1046 if (sack->flag [FLAG_APPLIED]) 1057 if (sack->flag [FLAG_APPLIED])
… …
1052 return 1; 1063 return 1;
1053 } 1064 }
1054 else if (!sack->env) 1065 else if (!sack->env)
1055 { 1066 {
1056 // active, but not ours: some other player has opened it 1067 // active, but not ours: some other player has opened it
1057 new_draw_info_format (NDI_UNIQUE, 0, op, "%s is already occupied.", query_name (sack)); 1068 op->failmsg (format ("Somebody else is using the %s already.", query_name (sack)));
1058 return 1; 1069 return 1;
1059 } 1070 }
1060 1071
1061 // fall through to opening it (active in inv) 1072 // fall through to opening it (active in inv)
1062 } 1073 }
… …
1064 { 1075 {
1065 // it is in our env, so activate it, do not open yet 1076 // it is in our env, so activate it, do not open yet
1066 op->close_container (); 1077 op->close_container ();
1067 sack->flag [FLAG_APPLIED] = 1; 1078 sack->flag [FLAG_APPLIED] = 1;
1068 esrv_update_item (UPD_FLAGS, op, sack); 1079 esrv_update_item (UPD_FLAGS, op, sack);
1069 new_draw_info_format (NDI_UNIQUE, 0, op, "You ready %s.", query_name (sack)); 1080 op->statusmsg (format ("You ready %s.", query_name (sack)));
1070 return 1; 1081 return 1;
1071 } 1082 }
1072 1083
1073 // it's locked? 1084 // it's locked?
1074 if (sack->slaying) 1085 if (sack->slaying)
1075 { 1086 {
1076 if (object *tmp = find_key (op, op, sack)) 1087 if (object *tmp = find_key (op, op, sack))
1077 new_draw_info_format (NDI_UNIQUE, 0, op, "You unlock %s with %s.", query_name (sack), query_name (tmp)); 1088 op->statusmsg (format ("You unlock %s with %s.", query_name (sack), query_name (tmp)));
1078 else 1089 else
1079 { 1090 {
1080 new_draw_info_format (NDI_UNIQUE, 0, op, "You don't have the key to unlock %s.", query_name (sack)); 1091 op->statusmsg (format ("You don't have the key to unlock %s.", query_name (sack)));
1081 return 1; 1092 return 1;
1082 } 1093 }
1083 } 1094 }
1084 1095
1085 op->open_container (sack); 1096 op->open_container (sack);
… …
1104 * with an altar. We call it a Potion - altars are stationary - it 1115 * with an altar. We call it a Potion - altars are stationary - it
1105 * is up to map designers to use them properly. 1116 * is up to map designers to use them properly.
1106 */ 1117 */
1107 if (altar->inv && altar->inv->type == SPELL) 1118 if (altar->inv && altar->inv->type == SPELL)
1108 { 1119 {
1109 new_draw_info_format (NDI_BLACK, 0, originator, "The altar casts %s.", &altar->inv->name); 1120 originator->statusmsg (format ("The altar casts %s.", &altar->inv->name));
1110 cast_spell (originator, altar, 0, altar->inv, NULL); 1121 cast_spell (originator, altar, 0, altar->inv, NULL);
1111 /* If it is connected, push the button. Fixes some problems with 1122 /* If it is connected, push the button. Fixes some problems with
1112 * old maps. 1123 * old maps.
1113 */ 1124 */
1114 1125
1115/* push_button (altar);*/ 1126/* push_button (altar);*/
1116 } 1127 }
… …
1140 double opinion; 1151 double opinion;
1141 object *tmp, *next; 1152 object *tmp, *next;
1142 1153
1143 SET_FLAG (op, FLAG_NO_APPLY); /* prevent loops */ 1154 SET_FLAG (op, FLAG_NO_APPLY); /* prevent loops */
1144 1155
1145 if (op->type != PLAYER) 1156 bool has_unpaid = false;
1157
1158 // quite inefficient to do this here twice, but the api doesn'T lend itself to
1159 // a quick and small change :(
1160 for (object::depth_iterator item = op->begin (); item != op->end (); ++item)
1161 if (item->flag [FLAG_UNPAID])
1162 {
1163 has_unpaid = true;
1164 break;
1165 }
1166
1167 if (!op->is_player ())
1146 { 1168 {
1147 /* Remove all the unpaid objects that may be carried here. 1169 /* Remove all the unpaid objects that may be carried here.
1148 * This could be pets or monsters that are somehow in 1170 * This could be pets or monsters that are somehow in
1149 * the shop. 1171 * the shop.
1150 */ 1172 */
… …
1154 1176
1155 if (QUERY_FLAG (tmp, FLAG_UNPAID)) 1177 if (QUERY_FLAG (tmp, FLAG_UNPAID))
1156 { 1178 {
1157 int i = find_free_spot (tmp, op->map, op->x, op->y, 1, 9); 1179 int i = find_free_spot (tmp, op->map, op->x, op->y, 1, 9);
1158 1180
1181 if (i >= 0)
1159 tmp->remove (); 1182 tmp->move (i);
1160
1161 if (i == -1)
1162 i = 0;
1163
1164 tmp->map = op->map;
1165 tmp->x = op->x + freearr_x[i];
1166 tmp->y = op->y + freearr_y[i];
1167 insert_ob_in_map (tmp, op->map, op, 0);
1168 } 1183 }
1169 } 1184 }
1170 1185
1171 /* Don't teleport things like spell effects */ 1186 /* Don't teleport things like spell effects */
1172 if (QUERY_FLAG (op, FLAG_NO_PICK)) 1187 if (QUERY_FLAG (op, FLAG_NO_PICK))
… …
1175 /* unpaid objects, or non living objects, can't transfer by 1190 /* unpaid objects, or non living objects, can't transfer by
1176 * shop mats. Instead, put it on a nearby space. 1191 * shop mats. Instead, put it on a nearby space.
1177 */ 1192 */
1178 if (QUERY_FLAG (op, FLAG_UNPAID) || !QUERY_FLAG (op, FLAG_ALIVE)) 1193 if (QUERY_FLAG (op, FLAG_UNPAID) || !QUERY_FLAG (op, FLAG_ALIVE))
1179 { 1194 {
1180
1181 /* Somebody dropped an unpaid item, just move to an adjacent place. */ 1195 /* Somebody dropped an unpaid item, just move to an adjacent place. */
1182 int i = find_free_spot (op, op->map, op->x, op->y, 1, 9); 1196 int i = find_free_spot (op, op->map, op->x, op->y, 1, 9);
1183 1197
1184 if (i != -1) 1198 if (i != -1)
1185 rv = transfer_ob (op, op->x + freearr_x[i], op->y + freearr_y[i], 0, shop_mat); 1199 rv = transfer_ob (op, op->x + freearr_x[i], op->y + freearr_y[i], 0, shop_mat);
1186 1200
1187 return 0; 1201 return 0;
1188 } 1202 }
1203
1189 /* Removed code that checked for multipart objects - it appears that 1204 /* Removed code that checked for multipart objects - it appears that
1190 * the teleport function should be able to handle this just fine. 1205 * the teleport function should be able to handle this just fine.
1191 */ 1206 */
1192 rv = teleport (shop_mat, SHOP_MAT, op); 1207 rv = teleport (shop_mat, SHOP_MAT, op);
1193 } 1208 }
1194 else if (can_pay (op) && get_payment (op)) 1209 else if (can_pay (op) && get_payment (op))
1195 { 1210 {
1196 /* this is only used for players */ 1211 /* this is only used for players */
1197 rv = teleport (shop_mat, SHOP_MAT, op); 1212 rv = teleport (shop_mat, SHOP_MAT, op);
1198 1213
1214 if (has_unpaid)
1215 op->contr->play_sound (sound_find ("shop_buy"));
1216 else if (is_in_shop (op))
1217 op->contr->play_sound (sound_find ("shop_enter"));
1218 else
1219 op->contr->play_sound (sound_find ("shop_leave"));
1220
1199 if (shop_mat->msg) 1221 if (shop_mat->msg)
1200 new_draw_info (NDI_UNIQUE, 0, op, shop_mat->msg); 1222 op->statusmsg (shop_mat->msg);
1201 /* This check below is a bit simplistic - generally it should be correct, 1223 /* This check below is a bit simplistic - generally it should be correct,
1202 * but there is never a guarantee that the bottom space on the map is 1224 * but there is never a guarantee that the bottom space on the map is
1203 * actually the shop floor. 1225 * actually the shop floor.
1204 */ 1226 */
1205 else if (!rv && !is_in_shop (op)) 1227 else if (!rv && !is_in_shop (op))
1206 { 1228 {
1207 opinion = shopkeeper_approval (op->map, op); 1229 opinion = shopkeeper_approval (op->map, op);
1208 1230
1209 if (opinion > 0.9) 1231 op->statusmsg (
1210 new_draw_info (NDI_UNIQUE, 0, op, "The shopkeeper gives you a friendly wave."); 1232 opinion >= 0.90 ? "The shopkeeper gives you a friendly wave."
1211 else if (opinion > 0.75) 1233 : opinion >= 0.75 ? "The shopkeeper waves to you."
1212 new_draw_info (NDI_UNIQUE, 0, op, "The shopkeeper waves to you."); 1234 : opinion >= 0.50 ? "The shopkeeper ignores you."
1213 else if (opinion > 0.5) 1235 : "The shopkeeper glares at you with contempt."
1214 new_draw_info (NDI_UNIQUE, 0, op, "The shopkeeper ignores you.");
1215 else 1236 );
1216 new_draw_info (NDI_UNIQUE, 0, op, "The shopkeeper glares at you with contempt.");
1217 } 1237 }
1218 } 1238 }
1219 else 1239 else
1220 { 1240 {
1221 /* if we get here, a player tried to leave a shop but was not able 1241 /* if we get here, a player tried to leave a shop but was not able
1222 * to afford the items he has. We try to move the player so that 1242 * to afford the items he has. We try to move the player so that
1223 * they are not on the mat anymore 1243 * they are not on the mat anymore
1224 */ 1244 */
1225 int i = find_free_spot (op, op->map, op->x, op->y, 1, 9); 1245 int i = find_free_spot (op, op->map, op->x, op->y, 1, 9);
1226 1246
1227 if (i == -1) 1247 if (i == -1)
1228 {
1229 LOG (llevError, "Internal shop-mat problem.\n"); 1248 LOG (llevError, "Internal shop-mat problem.\n");
1230 }
1231 else 1249 else
1232 { 1250 {
1233 op->remove (); 1251 op->remove ();
1234 op->x += freearr_x[i]; 1252 op->x += freearr_x[i];
1235 op->y += freearr_y[i]; 1253 op->y += freearr_y[i];
… …
1245 * Handles applying a sign. 1263 * Handles applying a sign.
1246 */ 1264 */
1247static void 1265static void
1248apply_sign (object *op, object *sign, int autoapply) 1266apply_sign (object *op, object *sign, int autoapply)
1249{ 1267{
1250 readable_message_type *msgType; 1268 if (sign->has_dialogue ())
1251
1252 if (sign->msg == NULL)
1253 { 1269 {
1254 new_draw_info (NDI_UNIQUE, 0, op, "Nothing is written on it."); 1270 op->statusmsg (format ("Maybe you should I<talk> to the %s instead?", &sign->name));
1255 return; 1271 return;
1256 } 1272 }
1257 1273
1274 if (!sign->msg)
1275 {
1276 op->contr->infobox (MSG_CHANNEL ("examine"),
1277 format ("T<%s>\n\n Nothing %sis written on it.",
1278 &sign->name,
1279 sign->name == sign->arch->name ? "" : "else "));
1280 return;
1281 }
1282
1258 if (sign->stats.food) 1283 if (sign->stats.food)
1259 { 1284 {
1260 if (sign->last_eat >= sign->stats.food) 1285 if (sign->last_eat >= sign->stats.food)
1261 { 1286 {
1262 if (!sign->move_on) 1287 if (!sign->move_on)
1263 new_draw_info (NDI_UNIQUE, 0, op, "You cannot read it anymore."); 1288 op->failmsg ("You cannot read it anymore.");
1264 1289
1265 return; 1290 return;
1266 } 1291 }
1267 1292
1268 if (!QUERY_FLAG (op, FLAG_WIZPASS)) 1293 if (!QUERY_FLAG (op, FLAG_WIZPASS))
… …
1274 * move_on is zero, it needs to be manually applied (doesn't talk 1299 * move_on is zero, it needs to be manually applied (doesn't talk
1275 * to us). 1300 * to us).
1276 */ 1301 */
1277 if (QUERY_FLAG (op, FLAG_BLIND) && !QUERY_FLAG (op, FLAG_WIZ) && !sign->move_on) 1302 if (QUERY_FLAG (op, FLAG_BLIND) && !QUERY_FLAG (op, FLAG_WIZ) && !sign->move_on)
1278 { 1303 {
1279 new_draw_info (NDI_UNIQUE, 0, op, "You are unable to read while blind."); 1304 op->failmsg ("You are unable to read while blind!");
1280 return; 1305 return;
1281 } 1306 }
1282 1307
1283 if (op->contr) 1308 if (op->contr)
1284 if (client *ns = op->contr->ns) 1309 if (client *ns = op->contr->ns)
1285 { 1310 {
1311 if (sign->sound)
1286 ns->play_sound (sign->sound); 1312 ns->play_sound (sign->sound);
1287 msgType = get_readable_message_type (sign); 1313 else if (autoapply)
1314 ns->play_sound (sound_find ("msg_voice"));
1288 1315
1289 if (ns->can_msg) 1316 op->contr->infobox (MSG_CHANNEL ("examine"), format ("T<%s>\n\n%s", &sign->name, &sign->msg));
1290 ns->send_msg (NDI_NAVY, msgType->msgtype, &sign->msg);
1291 else
1292 {
1293 char newbuf[HUGE_BUF];
1294 snprintf (newbuf, sizeof (newbuf), "%u %s", autoapply ? 1 : 0, &sign->msg);
1295 draw_ext_info (NDI_UNIQUE | NDI_NAVY, 0, op, msgType->message_type, msgType->message_subtype, newbuf, &sign->msg);
1296 }
1297 } 1317 }
1318}
1319
1320static void
1321move_apply_hole (object *trap, object *victim)
1322{
1323 /* Hole not open? */
1324 if (trap->stats.wc > 0)
1325 return;
1326
1327 /* Is this a multipart monster and not the head? If so, return.
1328 * Processing will happen if the head runs into the pit
1329 */
1330 if (victim->head)
1331 return;
1332
1333 // now find all possible locations and randomly pick one
1334 int dir = find_free_spot (victim, trap->map, EXIT_X (trap), EXIT_Y (trap), 0,
1335 trap->range >= 3 ? SIZEOFFREE3 + 1
1336 : trap->range >= 2 ? SIZEOFFREE2 + 1
1337 : trap->range >= 1 ? SIZEOFFREE1 + 1
1338 : SIZEOFFREE0 + 1);
1339
1340 if (dir < 0)
1341 return;
1342
1343 victim->play_sound (trap->sound ? trap->sound : sound_find ("fall_hole"));
1344 victim->statusmsg ("You fall through the hole!", NDI_RED);
1345
1346 transfer_ob (victim,
1347 EXIT_X (trap) + freearr_x[dir],
1348 EXIT_Y (trap) + freearr_y[dir],
1349 0, victim);
1298} 1350}
1299 1351
1300/** 1352/**
1301 * 'victim' moves onto 'trap' 1353 * 'victim' moves onto 'trap'
1302 * 'victim' leaves 'trap' 1354 * 'victim' leaves 'trap'
… …
1414 1466
1415 if (!trap->value) 1467 if (!trap->value)
1416 { 1468 {
1417 int tot; 1469 int tot;
1418 1470
1419 for (ab = trap->above, tot = 0; ab != NULL; ab = ab->above) 1471 for (ab = trap->above, tot = 0; ab; ab = ab->above)
1420 if ((ab->move_type && trap->move_on) || ab->move_type == 0) 1472 if ((ab->move_type && trap->move_on) || ab->move_type == 0)
1421 tot += (ab->nrof ? ab->nrof : 1) * ab->weight + ab->carrying; 1473 tot += ab->head_ ()->total_weight ();
1422 1474
1423 if (!(trap->value = (tot > trap->weight) ? 1 : 0)) 1475 if (!(trap->value = (tot > trap->weight) ? 1 : 0))
1424 goto leave; 1476 goto leave;
1425 1477
1426 SET_ANIMATION (trap, trap->value); 1478 SET_ANIMATION (trap, trap->value);
… …
1436 1488
1437 if ((ab->move_type && trap->move_on) || ab->move_type == 0) 1489 if ((ab->move_type && trap->move_on) || ab->move_type == 0)
1438 { 1490 {
1439 if (!sound_was_played) 1491 if (!sound_was_played)
1440 { 1492 {
1441 trap->play_sound (sound_find ("fall_hole")); 1493 trap->play_sound (trap->sound ? trap->sound : sound_find ("fall_hole"));
1442 sound_was_played = 1; 1494 sound_was_played = 1;
1443 } 1495 }
1444 1496
1445 new_draw_info (NDI_UNIQUE, 0, ab, "You fall into a trapdoor!"); 1497 ab->statusmsg ("You fall into a trapdoor!", NDI_RED);
1446 transfer_ob (ab, (int) EXIT_X (trap), (int) EXIT_Y (trap), 0, ab); 1498 transfer_ob (ab, EXIT_X (trap), EXIT_Y (trap), 0, ab);
1447 } 1499 }
1448 } 1500 }
1449 goto leave; 1501 goto leave;
1450 } 1502 }
1451 1503
1452
1453 case CONVERTER: 1504 case CONVERTER:
1454 if (convert_item (victim, trap) < 0) 1505 if (convert_item (victim, trap) < 0)
1455 { 1506 {
1456 new_draw_info_format (NDI_UNIQUE, 0, originator, "The %s seems to be broken!", query_name (trap)); 1507 originator->failmsg (format ("The %s seems to be broken!", query_name (trap)));
1457 get_archetype ("burnout")->insert_at (trap, trap); 1508 archetype::get (shstr_burnout)->insert_at (trap, trap);
1458 } 1509 }
1459 1510
1460 goto leave; 1511 goto leave;
1461 1512
1462 case TRIGGER_BUTTON: 1513 case TRIGGER_BUTTON:
… …
1472 case CHECK_INV: 1523 case CHECK_INV:
1473 check_inv (victim, trap); 1524 check_inv (victim, trap);
1474 goto leave; 1525 goto leave;
1475 1526
1476 case HOLE: 1527 case HOLE:
1477 /* Hole not open? */ 1528 move_apply_hole (trap, victim);
1478 if (trap->stats.wc > 0)
1479 goto leave;
1480
1481 /* Is this a multipart monster and not the head? If so, return.
1482 * Processing will happen if the head runs into the pit
1483 */
1484 if (victim->head)
1485 goto leave;
1486
1487 victim->play_sound (sound_find ("fall_hole"));
1488 new_draw_info (NDI_UNIQUE, 0, victim, "You fall through the hole!\n");
1489 transfer_ob (victim, EXIT_X (trap), EXIT_Y (trap), 1, victim);
1490 goto leave; 1529 goto leave;
1491 1530
1492 case EXIT: 1531 case EXIT:
1493 if (victim->type == PLAYER && EXIT_PATH (trap)) 1532 if (victim->type == PLAYER && EXIT_PATH (trap))
1494 { 1533 {
1495 /* Basically, don't show exits leading to random maps the 1534 /* Basically, don't show exits leading to random maps the
1496 * players output. 1535 * players output.
1497 */ 1536 */
1498 if (trap->msg && strncmp (EXIT_PATH (trap), "/!", 2)) 1537 if (trap->msg && strncmp (EXIT_PATH (trap), "/!", 2))
1499 new_draw_info (NDI_NAVY, 0, victim, trap->msg); 1538 victim->statusmsg (trap->msg, NDI_NAVY);
1500 1539
1540 trap->play_sound (trap->sound);
1501 victim->enter_exit (trap); 1541 victim->enter_exit (trap);
1502 } 1542 }
1503 goto leave; 1543 goto leave;
1504 1544
1505 case ENCOUNTER: 1545 case ENCOUNTER:
… …
1527 goto leave; 1567 goto leave;
1528 1568
1529 case RUNE: 1569 case RUNE:
1530 case TRAP: 1570 case TRAP:
1531 if (trap->level && QUERY_FLAG (victim, FLAG_ALIVE)) 1571 if (trap->level && QUERY_FLAG (victim, FLAG_ALIVE))
1532 {
1533 spring_trap (trap, victim); 1572 spring_trap (trap, victim);
1534 }
1535 goto leave; 1573 goto leave;
1536 1574
1537 default: 1575 default:
1538 LOG (llevDebug, "name %s, arch %s, type %d with fly/walk on/off not " 1576 LOG (llevDebug, "name %s, arch %s, type %d with fly/walk on/off not "
1539 "handled in move_apply()\n", &trap->name, &trap->arch->archname, trap->type); 1577 "handled in move_apply()\n", &trap->name, &trap->arch->archname, trap->type);
… …
1553 int lev_diff; 1591 int lev_diff;
1554 object *skill_ob; 1592 object *skill_ob;
1555 1593
1556 if (QUERY_FLAG (op, FLAG_BLIND) && !QUERY_FLAG (op, FLAG_WIZ)) 1594 if (QUERY_FLAG (op, FLAG_BLIND) && !QUERY_FLAG (op, FLAG_WIZ))
1557 { 1595 {
1558 new_draw_info (NDI_UNIQUE, 0, op, "You are unable to read while blind."); 1596 op->failmsg ("You are unable to read while blind!");
1559 return; 1597 return;
1560 } 1598 }
1561 1599
1562 if (!tmp->msg) 1600 if (!tmp->msg)
1563 { 1601 {
1564 new_draw_info_format (NDI_UNIQUE, 0, op, "You open the %s and find it empty.", &tmp->name); 1602 op->failmsg (format ("You open the %s and find it empty.", &tmp->name));
1565 return; 1603 return;
1566 } 1604 }
1567 1605
1568 /* need a literacy skill to read stuff! */ 1606 /* need a literacy skill to read stuff! */
1569 skill_ob = find_skill_by_name (op, tmp->skill); 1607 skill_ob = find_skill_by_name (op, tmp->skill);
1570 if (!skill_ob) 1608 if (!skill_ob)
1571 { 1609 {
1572 new_draw_info (NDI_UNIQUE, 0, op, "You are unable to decipher the strange symbols. H<You lack the skill to read this>"); 1610 op->failmsg (format ("You are unable to decipher the strange symbols. H<You lack the %s skill to read this.>", &tmp->skill));
1573 return; 1611 return;
1574 } 1612 }
1575 1613
1576 lev_diff = tmp->level - (skill_ob->level + 5); 1614 lev_diff = tmp->level - (skill_ob->level + 5);
1577 if (!QUERY_FLAG (op, FLAG_WIZ) && lev_diff > 0) 1615 if (!QUERY_FLAG (op, FLAG_WIZ) && lev_diff > 0)
1578 { 1616 {
1579 if (lev_diff < 2) 1617 op->failmsg (lev_diff < 2 ? "This book is just barely beyond your comprehension."
1580 new_draw_info (NDI_UNIQUE, 0, op, "This book is just barely beyond your comprehension."); 1618 : lev_diff < 3 ? "This book is slightly beyond your comprehension."
1581 else if (lev_diff < 3) 1619 : lev_diff < 5 ? "This book is beyond your comprehension."
1582 new_draw_info (NDI_UNIQUE, 0, op, "This book is slightly beyond your comprehension."); 1620 : lev_diff < 8 ? "This book is quite a bit beyond your comprehension."
1583 else if (lev_diff < 5) 1621 : lev_diff < 15 ? "This book is way beyond your comprehension."
1584 new_draw_info (NDI_UNIQUE, 0, op, "This book is beyond your comprehension."); 1622 : "This book is totally beyond your comprehension.");
1585 else if (lev_diff < 8)
1586 new_draw_info (NDI_UNIQUE, 0, op, "This book is quite a bit beyond your comprehension.");
1587 else if (lev_diff < 15)
1588 new_draw_info (NDI_UNIQUE, 0, op, "This book is way beyond your comprehension.");
1589 else
1590 new_draw_info (NDI_UNIQUE, 0, op, "This book is totally beyond your comprehension.");
1591 return; 1623 return;
1592 } 1624 }
1593 1625
1594 readable_message_type *msgType = get_readable_message_type (tmp); 1626 readable_message_type *msgType = get_readable_message_type (tmp);
1595 1627
1596 if (player *pl = op->contr) 1628 if (player *pl = op->contr)
1597 if (client *ns = pl->ns) 1629 if (client *ns = pl->ns)
1598 if (ns->can_msg) 1630 pl->infobox (MSG_CHANNEL ("book"), format ("T<%s>\n\n%s", (char *)long_desc (tmp, op), &tmp->msg));
1599 {
1600 dynbuf_text buf;
1601 buf << long_desc (tmp, op)
1602 << "\n\n"
1603 << tmp->msg
1604 << '\0';
1605 ns->send_msg (NDI_NAVY, msgType->msgtype, buf.linearise ());
1606 }
1607 else
1608 draw_ext_info_format (NDI_UNIQUE | NDI_NAVY, 0, op,
1609 msgType->message_type, msgType->message_subtype,
1610 "You open the %s and start reading.\n%s", (char *)"%s\n%s",
1611 long_desc (tmp, op), &tmp->msg);
1612 1631
1613 /* gain xp from reading */ 1632 /* gain xp from reading */
1614 if (!QUERY_FLAG (tmp, FLAG_NO_SKILL_IDENT)) 1633 if (!QUERY_FLAG (tmp, FLAG_NO_SKILL_IDENT))
1615 { /* only if not read before */ 1634 { /* only if not read before */
1616 int exp_gain = calc_skill_exp (op, tmp, skill_ob); 1635 int exp_gain = calc_skill_exp (op, tmp, skill_ob);
… …
1618 if (!QUERY_FLAG (tmp, FLAG_IDENTIFIED)) 1637 if (!QUERY_FLAG (tmp, FLAG_IDENTIFIED))
1619 { 1638 {
1620 /*exp_gain *= 2; because they just identified it too */ 1639 /*exp_gain *= 2; because they just identified it too */
1621 SET_FLAG (tmp, FLAG_IDENTIFIED); 1640 SET_FLAG (tmp, FLAG_IDENTIFIED);
1622 1641
1623 /* If in a container, update how it looks */ 1642 if (object *pl = tmp->visible_to ())
1624 if (tmp->env)
1625 esrv_update_item (UPD_FLAGS | UPD_NAME, op, tmp); 1643 esrv_update_item (UPD_FLAGS | UPD_NAME, pl, tmp);
1626 else
1627 op->contr->ns->floorbox_update ();
1628 } 1644 }
1629 1645
1630 change_exp (op, exp_gain, skill_ob->skill, 0); 1646 change_exp (op, exp_gain, skill_ob->skill, 0);
1631 SET_FLAG (tmp, FLAG_NO_SKILL_IDENT); /* so no more xp gained from this book */ 1647 SET_FLAG (tmp, FLAG_NO_SKILL_IDENT); /* so no more xp gained from this book */
1632 } 1648 }
… …
1637 * op is the person learning the skill, tmp is the skill scroll object 1653 * op is the person learning the skill, tmp is the skill scroll object
1638 */ 1654 */
1639static void 1655static void
1640apply_skillscroll (object *op, object *tmp) 1656apply_skillscroll (object *op, object *tmp)
1641{ 1657{
1642 switch ((int) learn_skill (op, tmp)) 1658 switch (learn_skill (op, tmp))
1643 { 1659 {
1644 case 0: 1660 case 0:
1645 new_draw_info (NDI_UNIQUE, 0, op, "You already possess the knowledge "); 1661 op->play_sound (sound_find ("generic_fail"));
1646 new_draw_info_format (NDI_UNIQUE, 0, op, "held within the %s.\n", query_name (tmp)); 1662 op->failmsg (format ("You already possess the knowledge held within the %s.", query_name (tmp)));
1647 return; 1663 break;
1648 1664
1649 case 1: 1665 case 1:
1650 new_draw_info_format (NDI_UNIQUE, 0, op, "You succeed in learning %s", &tmp->skill);
1651 decrease_ob (tmp); 1666 tmp->decrease ();
1667 op->play_sound (sound_find ("skill_learn"));
1668 op->statusmsg (format ("You succeed in learning %s", &tmp->skill));
1652 return; 1669 break;
1653 1670
1654 default: 1671 default:
1672 tmp->decrease ();
1673 op->play_sound (sound_find ("generic_fail"));
1655 new_draw_info_format (NDI_UNIQUE, 0, op, "You fail to learn the knowledge of the %s.\n", query_name (tmp)); 1674 op->failmsg (format ("You fail to learn the knowledge of the %s.\n", query_name (tmp)));
1656 decrease_ob (tmp);
1657 return; 1675 break;
1658 } 1676 }
1659} 1677}
1660 1678
1661/** 1679/**
1662 * Actually makes op learn spell. 1680 * Actually makes op learn spell.
… …
1712 { 1730 {
1713 LOG (llevError, "BUG: do_forget_spell(): spell not known\n"); 1731 LOG (llevError, "BUG: do_forget_spell(): spell not known\n");
1714 return; 1732 return;
1715 } 1733 }
1716 1734
1717 new_draw_info_format (NDI_UNIQUE | NDI_NAVY, 0, op, "You lose knowledge of %s.", spell); 1735 op->failmsg (format ("You lose knowledge of %s.", spell));
1718 player_unready_range_ob (op->contr, spob); 1736 player_unready_range_ob (op->contr, spob);
1719 esrv_remove_spell (op->contr, spob); 1737 esrv_remove_spell (op->contr, spob);
1720 spob->destroy (); 1738 spob->destroy ();
1721} 1739}
1722 1740
… …
1730{ 1748{
1731 object *skop, *spell, *spell_skill; 1749 object *skop, *spell, *spell_skill;
1732 1750
1733 if (QUERY_FLAG (op, FLAG_BLIND) && !QUERY_FLAG (op, FLAG_WIZ)) 1751 if (QUERY_FLAG (op, FLAG_BLIND) && !QUERY_FLAG (op, FLAG_WIZ))
1734 { 1752 {
1735 new_draw_info (NDI_UNIQUE, 0, op, "You are unable to read while blind."); 1753 op->failmsg ("You are unable to read while blind.");
1736 return; 1754 return;
1737 } 1755 }
1738 1756
1739 /* artifact_spellbooks have 'slaying' field point to a spell name, 1757 /* artifact_spellbooks have 'slaying' field point to a spell name,
1740 * instead of having their spell stored in stats.sp. These are 1758 * instead of having their spell stored in stats.sp. These are
… …
1743 if (tmp->slaying) 1761 if (tmp->slaying)
1744 { 1762 {
1745 spell = arch_to_object (find_archetype_by_object_name (tmp->slaying)); 1763 spell = arch_to_object (find_archetype_by_object_name (tmp->slaying));
1746 if (!spell) 1764 if (!spell)
1747 { 1765 {
1748 new_draw_info_format (NDI_UNIQUE, 0, op, "The book's formula for %s is incomplete", &tmp->slaying); 1766 op->failmsg (format ("The book's formula for %s is incomplete.", &tmp->slaying));
1749 return; 1767 return;
1750 } 1768 }
1751 else 1769 else
1752 insert_ob_in_ob (spell, tmp); 1770 insert_ob_in_ob (spell, tmp);
1753 1771
… …
1758 1776
1759 /* need a literacy skill to learn spells. Also, having a literacy level 1777 /* need a literacy skill to learn spells. Also, having a literacy level
1760 * lower than the spell will make learning the spell more difficult */ 1778 * lower than the spell will make learning the spell more difficult */
1761 if (!skop) 1779 if (!skop)
1762 { 1780 {
1763 new_draw_info (NDI_UNIQUE, 0, op, "You can't read! Your attempt fails. H<You lack the literacy skill.>"); 1781 op->failmsg (format ("You can't read! Your attempt fails. H<You lack the %s skill.>", &tmp->skill));
1764 return; 1782 return;
1765 } 1783 }
1766 1784
1767 spell = tmp->inv; 1785 spell = tmp->inv;
1768 1786
1769 if (!spell) 1787 if (!spell)
1770 { 1788 {
1771 LOG (llevError, "apply_spellbook: Book %s has no spell in it!\n", &tmp->name); 1789 LOG (llevError, "apply_spellbook: Book %s has no spell in it!\n", &tmp->name);
1772 new_draw_info (NDI_UNIQUE, 0, op, "The spellbook symbols make no sense. This is a bug, please report!"); 1790 op->failmsg ("The spellbook symbols make no sense. This is a bug, please report!");
1773 return; 1791 return;
1774 } 1792 }
1775 1793
1776 if (skop->level < int (sqrtf (spell->level) * 1.5f)) 1794 if (skop->level < int (sqrtf (spell->level) * 1.5f))
1777 { 1795 {
1778 new_draw_info (NDI_UNIQUE, 0, op, "You are unable to decipher the strange symbols. H<Your literacy level is too low.>"); 1796 op->failmsg (format ("You are unable to decipher the strange symbols. H<Your %s level is too low.>", &tmp->skill));
1779 return; 1797 return;
1780 } 1798 }
1781 1799
1782 new_draw_info_format (NDI_UNIQUE, 0, op, "The spellbook contains the %s level spell %s.", get_levelnumber (spell->level), &spell->name); 1800 op->statusmsg (format ("The spellbook contains the %s level spell %s.", get_levelnumber (spell->level), &spell->name));
1783 1801
1784 if (!QUERY_FLAG (tmp, FLAG_IDENTIFIED)) 1802 if (!QUERY_FLAG (tmp, FLAG_IDENTIFIED))
1785 {
1786 identify (tmp); 1803 identify (tmp);
1787
1788 if (tmp->env)
1789 esrv_update_item (UPD_FLAGS | UPD_NAME, op, tmp);
1790 else
1791 op->contr->ns->floorbox_update ();
1792 }
1793 1804
1794 /* I removed the check for special_prayer_mark here - it didn't make 1805 /* I removed the check for special_prayer_mark here - it didn't make
1795 * a lot of sense - special prayers are not found in spellbooks, and 1806 * a lot of sense - special prayers are not found in spellbooks, and
1796 * if the player doesn't know the spell, doesn't make a lot of sense that 1807 * if the player doesn't know the spell, doesn't make a lot of sense that
1797 * they would have a special prayer mark. 1808 * they would have a special prayer mark.
1798 */ 1809 */
1799 if (check_spell_known (op, spell->name)) 1810 if (check_spell_known (op, spell->name))
1800 { 1811 {
1801 new_draw_info (NDI_UNIQUE, 0, op, "You already know that spell. H<It makes no sense to learn spells twice, and would only waste the spellbook.>\n"); 1812 op->statusmsg ("You already know that spell. H<It makes no sense to learn spells twice, and would only waste the spellbook.>\n");
1802 return; 1813 return;
1803 } 1814 }
1804 1815
1805 if (spell->skill) 1816 if (spell->skill)
1806 { 1817 {
1807 spell_skill = find_skill_by_name (op, spell->skill); 1818 spell_skill = find_skill_by_name (op, spell->skill);
1808 1819
1809 if (!spell_skill) 1820 if (!spell_skill)
1810 { 1821 {
1811 new_draw_info_format (NDI_UNIQUE, 0, op, "You lack the skill %s to use this spell.", &spell->skill); 1822 op->failmsg (format ("You lack the skill %s to use this spell.", &spell->skill));
1812 return; 1823 return;
1813 } 1824 }
1814 1825
1815 if (spell_skill->level < spell->level) 1826 if (spell_skill->level < spell->level)
1816 { 1827 {
1817 new_draw_info_format (NDI_UNIQUE, 0, op, "You need to be level %d in %s to learn this spell.", spell->level, &spell->skill); 1828 op->failmsg (format ("You need to be level %d in %s to learn this spell.", spell->level, &spell->skill));
1818 return; 1829 return;
1819 } 1830 }
1820 } 1831 }
1821 1832
1822 /* Logic as follows 1833 /* Logic as follows
… …
1831 * Overall, chances are the same but a player will find having a high 1842 * Overall, chances are the same but a player will find having a high
1832 * literacy rate very useful! -b.t. 1843 * literacy rate very useful! -b.t.
1833 */ 1844 */
1834 if (QUERY_FLAG (op, FLAG_CONFUSED)) 1845 if (QUERY_FLAG (op, FLAG_CONFUSED))
1835 { 1846 {
1836 new_draw_info (NDI_UNIQUE, 0, op, "In your confused state you flub the wording of the text!"); 1847 op->failmsg ("In your confused state you flub the wording of the text!");
1837 scroll_failure (op, 0 - random_roll (0, spell->level, op, PREFER_LOW), MAX (spell->stats.sp, spell->stats.grace)); 1848 scroll_failure (op, 0 - random_roll (0, spell->level, op, PREFER_LOW), MAX (spell->stats.sp, spell->stats.grace));
1838 } 1849 }
1839 else if (QUERY_FLAG (tmp, FLAG_STARTEQUIP) || 1850 else if (QUERY_FLAG (tmp, FLAG_STARTEQUIP) ||
1840 (random_roll (0, 100, op, PREFER_LOW) - (5 * skop->level)) < learn_spell[spell->stats.grace ? op->stats.Wis : op->stats.Int]) 1851 (random_roll (0, 100, op, PREFER_LOW) - (5 * skop->level)) < learn_spell[spell->stats.grace ? op->stats.Wis : op->stats.Int])
1841 { 1852 {
1842 1853 op->statusmsg ("You succeed in learning the spell!", NDI_GREEN);
1843 new_draw_info (NDI_UNIQUE, 0, op, "You succeed in learning the spell!");
1844 do_learn_spell (op, spell, 0); 1854 do_learn_spell (op, spell, 0);
1845 1855
1846 /* xp gain to literacy for spell learning */ 1856 /* xp gain to literacy for spell learning */
1847 if (!QUERY_FLAG (tmp, FLAG_STARTEQUIP)) 1857 if (!QUERY_FLAG (tmp, FLAG_STARTEQUIP))
1848 change_exp (op, calc_skill_exp (op, tmp, skop), skop->skill, 0); 1858 change_exp (op, calc_skill_exp (op, tmp, skop), skop->skill, 0);
1849 } 1859 }
1850 else 1860 else
1851 { 1861 {
1852 op->contr->play_sound (sound_find ("fumble_spell")); 1862 op->contr->play_sound (sound_find ("fumble_spell"));
1853 new_draw_info (NDI_UNIQUE, 0, op, "You fail to learn the spell. H<Wis (priests) or Int (wizards) governs the chance of learning a prayer or spell.>\n"); 1863 op->failmsg ("You fail to learn the spell. H<Wis (priests) or Int (wizards) governs the chance of learning a prayer or spell.>\n");
1854 } 1864 }
1855 1865
1856 decrease_ob (tmp); 1866 tmp->decrease ();
1857} 1867}
1858 1868
1859/** 1869/**
1860 * Handles applying a spell scroll. 1870 * Handles applying a spell scroll.
1861 */ 1871 */
… …
1864{ 1874{
1865 object *skop; 1875 object *skop;
1866 1876
1867 if (QUERY_FLAG (op, FLAG_BLIND) && !QUERY_FLAG (op, FLAG_WIZ)) 1877 if (QUERY_FLAG (op, FLAG_BLIND) && !QUERY_FLAG (op, FLAG_WIZ))
1868 { 1878 {
1869 new_draw_info (NDI_UNIQUE, 0, op, "You are unable to read while blind."); 1879 op->failmsg ("You are unable to read while blind.");
1870 return; 1880 return;
1871 } 1881 }
1872 1882
1873 if (!tmp->inv || tmp->inv->type != SPELL) 1883 if (!tmp->inv || tmp->inv->type != SPELL)
1874 { 1884 {
1875 new_draw_info (NDI_UNIQUE, 0, op, "The scroll just doesn't make sense! H<...and never will make sense.>"); 1885 op->failmsg ("The scroll just doesn't make sense! H<...and never will make sense.>");
1876 return; 1886 return;
1877 } 1887 }
1878 1888
1879 if (op->type == PLAYER) 1889 if (op->type == PLAYER)
1880 { 1890 {
… …
1886 */ 1896 */
1887 skop = find_skill_by_name (op, skill_names[SK_LITERACY]); 1897 skop = find_skill_by_name (op, skill_names[SK_LITERACY]);
1888 1898
1889 if (!skop) 1899 if (!skop)
1890 { 1900 {
1891 new_draw_info (NDI_UNIQUE, 0, op, "You are unable to decipher the strange symbols. H<You lack the literacy skill.>"); 1901 op->failmsg (format ("You are unable to decipher the strange symbols. H<You lack the %s skill.>", &skill_names[SK_LITERACY]));
1892 return; 1902 return;
1893 } 1903 }
1894 1904
1895 if ((exp_gain = calc_skill_exp (op, tmp, skop))) 1905 if ((exp_gain = calc_skill_exp (op, tmp, skop)))
1896 change_exp (op, exp_gain, skop->skill, 0); 1906 change_exp (op, exp_gain, skop->skill, 0);
1897 } 1907 }
1898 1908
1899 if (!QUERY_FLAG (tmp, FLAG_IDENTIFIED)) 1909 if (!QUERY_FLAG (tmp, FLAG_IDENTIFIED))
1900 identify (tmp); 1910 identify (tmp);
1901 1911
1902 new_draw_info_format (NDI_BLACK, 0, op, "The scroll of %s turns to dust.", &tmp->inv->name); 1912 op->statusmsg (format ("The scroll of %s turns to dust.", &tmp->inv->name));
1903 1913
1904 cast_spell (op, tmp, dir, tmp->inv, NULL); 1914 cast_spell (op, tmp, dir, tmp->inv, NULL);
1905 decrease_ob (tmp); 1915 tmp->decrease ();
1906} 1916}
1907 1917
1908/** 1918/**
1909 * Applies a treasure object - by default, chest. op 1919 * Applies a treasure object - by default, chest. op
1910 * is the person doing the applying, tmp is the treasure 1920 * is the person doing the applying, tmp is the treasure
1911 * chest. 1921 * chest.
1912 */ 1922 */
1913static void 1923static void
1914apply_treasure (object *op, object *tmp) 1924apply_treasure (object *op, object *tmp)
1915{ 1925{
1916 /* Nice side effect of new treasure creation method is that the treasure 1926 /* Nice side effect of this treasure creation method is that the treasure
1917 * for the chest is done when the chest is created, and put into the chest 1927 * for the chest is done when the chest is created, and put into the chest
1918 * inventory. So that when the chest burns up, the items still exist. Also 1928 * inventory. So that when the chest burns up, the items still exist. Also
1919 * prevents people fromt moving chests to more difficult maps to get better 1929 * prevents people from moving chests to more difficult maps to get better
1920 * treasure 1930 * treasure
1921 */ 1931 */
1922 object *treas = tmp->inv; 1932 object *treas = tmp->inv;
1923 1933
1924 if (!treas) 1934 if (!treas)
1925 { 1935 {
1926 new_draw_info (NDI_UNIQUE, 0, op, "The chest was empty."); 1936 op->statusmsg ("The chest was empty.");
1927 decrease_ob (tmp); 1937 tmp->decrease ();
1928 return; 1938 return;
1929 } 1939 }
1930 1940
1931 while (tmp->inv) 1941 while (tmp->inv)
1932 { 1942 {
1933 treas = tmp->inv; 1943 treas = tmp->inv;
1934
1935 treas->remove (); 1944 treas->remove ();
1936 new_draw_info_format (NDI_UNIQUE, 0, op, "You find %s in the chest.", query_name (treas));
1937 1945
1938 treas->x = op->x; 1946 treas->x = op->x;
1939 treas->y = op->y; 1947 treas->y = op->y;
1940 treas = insert_ob_in_map (treas, op->map, op, INS_BELOW_ORIGINATOR); 1948 treas = insert_ob_in_map (treas, op->map, op, INS_BELOW_ORIGINATOR);
1941 1949
1942 if (treas && (treas->type == RUNE || treas->type == TRAP) && treas->level && QUERY_FLAG (op, FLAG_ALIVE)) 1950 if (treas && (treas->type == RUNE || treas->type == TRAP) && treas->level && QUERY_FLAG (op, FLAG_ALIVE))
1943 spring_trap (treas, op); 1951 spring_trap (treas, op);
1944 1952
1945 /* If either player or container was destroyed, no need to do 1953 /* If either player or container was destroyed, no need to do
1946 * further processing. I think this should be enclused with 1954 * further processing. I think this should be enclused with
1947 * spring trap above, as I don't think there is otherwise 1955 * spring trap above, as I don't think there is otherwise
1948 * any way for the treasure chest or player to get killed 1956 * any way for the treasure chest or player to get killed.
1949 */ 1957 */
1950 if (op->destroyed () || tmp->destroyed ()) 1958 if (op->destroyed () || tmp->destroyed ())
1951 break; 1959 break;
1952 } 1960 }
1953 1961
1954 if (!tmp->destroyed () && tmp->inv == NULL) 1962 if (!tmp->destroyed () && !tmp->inv)
1955 decrease_ob (tmp); 1963 tmp->decrease (true);
1956
1957} 1964}
1958 1965
1959/** 1966/**
1960 * op eats food. 1967 * op eats food.
1961 * If player, takes care of messages and dragon special food. 1968 * If player, takes care of messages and dragon special food.
… …
1976 { 1983 {
1977 /* usual case - no dragon meal: */ 1984 /* usual case - no dragon meal: */
1978 if (op->stats.food + tmp->stats.food > 999) 1985 if (op->stats.food + tmp->stats.food > 999)
1979 { 1986 {
1980 if (tmp->type == FOOD || tmp->type == FLESH) 1987 if (tmp->type == FOOD || tmp->type == FLESH)
1981 new_draw_info (NDI_UNIQUE, 0, op, "You feel full, but what a waste of food!"); 1988 op->failmsg ("You feel full, but what a waste of food!");
1982 else 1989 else
1983 new_draw_info (NDI_UNIQUE, 0, op, "Most of the drink goes down your face not your throat!"); 1990 op->statusmsg ("Most of the drink goes down your face not your throat!");
1984 } 1991 }
1992
1993 tmp->play_sound (
1994 tmp->sound
1995 ? tmp->sound
1996 : tmp->type == DRINK
1997 ? sound_find ("eat_drink")
1998 : sound_find ("eat_food")
1999 );
1985 2000
1986 if (!QUERY_FLAG (tmp, FLAG_CURSED)) 2001 if (!QUERY_FLAG (tmp, FLAG_CURSED))
1987 { 2002 {
1988 char buf[MAX_BUF]; 2003 const char *buf;
1989 2004
1990 if (!is_dragon_pl (op)) 2005 if (!is_dragon_pl (op))
1991 { 2006 {
1992 /* eating message for normal players */ 2007 /* eating message for normal players */
1993 if (tmp->type == DRINK) 2008 if (tmp->type == DRINK)
1994 sprintf (buf, "Ahhh...that %s tasted good.", &tmp->name); 2009 buf = format ("Ahhh...that %s tasted good.", &tmp->name);
1995 else 2010 else
1996 sprintf (buf, "The %s tasted %s", &tmp->name, tmp->type == FLESH ? "terrible!" : "good."); 2011 buf = format ("The %s tasted %s", &tmp->name, tmp->type == FLESH ? "terrible!" : "good.");
1997 } 2012 }
1998 else 2013 else
1999 {
2000 /* eating message for dragon players */ 2014 /* eating message for dragon players */
2001 sprintf (buf, "The %s tasted terrible!", &tmp->name); 2015 buf = format ("The %s tasted terrible!", &tmp->name);
2002 }
2003 2016
2004 new_draw_info (NDI_UNIQUE, 0, op, buf); 2017 op->statusmsg (buf);
2018
2005 capacity_remaining = 999 - op->stats.food; 2019 capacity_remaining = 999 - op->stats.food;
2006 op->stats.food += tmp->stats.food; 2020 op->stats.food += tmp->stats.food;
2007 if (capacity_remaining < tmp->stats.food) 2021 if (capacity_remaining < tmp->stats.food)
2008 op->stats.hp += capacity_remaining / 50; 2022 op->stats.hp += capacity_remaining / 50;
2009 else 2023 else
2010 op->stats.hp += tmp->stats.food / 50; 2024 op->stats.hp += tmp->stats.food / 50;
2025
2011 if (op->stats.hp > op->stats.maxhp) 2026 if (op->stats.hp > op->stats.maxhp)
2012 op->stats.hp = op->stats.maxhp; 2027 op->stats.hp = op->stats.maxhp;
2013 if (op->stats.food > 999) 2028 if (op->stats.food > 999)
2014 op->stats.food = 999; 2029 op->stats.food = 999;
2015 } 2030 }
… …
2019 eat_special_food (op, tmp); 2034 eat_special_food (op, tmp);
2020 } 2035 }
2021 } 2036 }
2022 2037
2023 handle_apply_yield (tmp); 2038 handle_apply_yield (tmp);
2024 decrease_ob (tmp); 2039 tmp->decrease ();
2025} 2040}
2026 2041
2027/** 2042/**
2028 * A dragon is eating some flesh. If the flesh contains resistances, 2043 * A dragon is eating some flesh. If the flesh contains resistances,
2029 * there is a chance for the dragon's skin to get improved. 2044 * there is a chance for the dragon's skin to get improved.
… …
2039{ 2054{
2040 object *skin = NULL; /* pointer to dragon skin force */ 2055 object *skin = NULL; /* pointer to dragon skin force */
2041 object *abil = NULL; /* pointer to dragon ability force */ 2056 object *abil = NULL; /* pointer to dragon ability force */
2042 object *tmp = NULL; /* tmp. object */ 2057 object *tmp = NULL; /* tmp. object */
2043 2058
2044 char buf[MAX_BUF]; /* tmp. string buffer */
2045 double chance; /* improvement-chance of one resistance type */ 2059 double chance; /* improvement-chance of one resistance type */
2046 double totalchance = 1; /* total chance of gaining one resistance */ 2060 double totalchance = 1; /* total chance of gaining one resistance */
2047 double bonus = 0; /* level bonus (improvement is easier at lowlevel) */ 2061 double bonus = 0; /* level bonus (improvement is easier at lowlevel) */
2048 double mbonus = 0; /* monster bonus */ 2062 double mbonus = 0; /* monster bonus */
2049 int atnr_winner[NROFATTACKS]; /* winning candidates for resistance improvement */ 2063 int atnr_winner[NROFATTACKS]; /* winning candidates for resistance improvement */
… …
2071 /* now start by filling stomache and health, according to food-value */ 2085 /* now start by filling stomache and health, according to food-value */
2072 if ((999 - op->stats.food) < meal->stats.food) 2086 if ((999 - op->stats.food) < meal->stats.food)
2073 op->stats.hp += (999 - op->stats.food) / 50; 2087 op->stats.hp += (999 - op->stats.food) / 50;
2074 else 2088 else
2075 op->stats.hp += meal->stats.food / 50; 2089 op->stats.hp += meal->stats.food / 50;
2090
2076 if (op->stats.hp > op->stats.maxhp) 2091 if (op->stats.hp > op->stats.maxhp)
2077 op->stats.hp = op->stats.maxhp; 2092 op->stats.hp = op->stats.maxhp;
2078 2093
2079 op->stats.food = MIN (999, op->stats.food + meal->stats.food); 2094 op->stats.food = MIN (999, op->stats.food + meal->stats.food);
2080 2095
… …
2125 } 2140 }
2126 } 2141 }
2127 2142
2128 /* inverse totalchance as until now we have the failure-chance */ 2143 /* inverse totalchance as until now we have the failure-chance */
2129 totalchance = 100 - totalchance * 100; 2144 totalchance = 100 - totalchance * 100;
2145
2130 /* print message according to totalchance */ 2146 /* print message according to totalchance */
2147 const char *buf;
2131 if (totalchance > 50.) 2148 if (totalchance > 50.)
2132 sprintf (buf, "Hmm! The %s tasted delicious!", &meal->name); 2149 buf = format ("Hmm! The %s tasted delicious!", &meal->name);
2133 else if (totalchance > 10.) 2150 else if (totalchance > 10.)
2134 sprintf (buf, "The %s tasted very good.", &meal->name); 2151 buf = format ("The %s tasted very good.", &meal->name);
2135 else if (totalchance > 1.) 2152 else if (totalchance > 1.)
2136 sprintf (buf, "The %s tasted good.", &meal->name); 2153 buf = format ("The %s tasted good.", &meal->name);
2137 else if (totalchance > 0.1) 2154 else if (totalchance > 0.1)
2138 sprintf (buf, "The %s tasted bland.", &meal->name); 2155 buf = format ("The %s tasted bland.", &meal->name);
2139 else if (totalchance >= 0.01) 2156 else if (totalchance >= 0.01)
2140 sprintf (buf, "The %s had a boring taste.", &meal->name); 2157 buf = format ("The %s had a boring taste.", &meal->name);
2141 else if (meal->last_eat > 0 && atnr_is_dragon_enabled (meal->last_eat)) 2158 else if (meal->last_eat > 0 && atnr_is_dragon_enabled (meal->last_eat))
2142 sprintf (buf, "The %s tasted strange.", &meal->name); 2159 buf = format ("The %s tasted strange.", &meal->name);
2143 else 2160 else
2144 sprintf (buf, "The %s had no taste.", &meal->name); 2161 buf = format ("The %s had no taste.", &meal->name);
2145 new_draw_info (NDI_UNIQUE, 0, op, buf); 2162
2163 op->statusmsg (buf);
2146 2164
2147 /* now choose a winner if we have any */ 2165 /* now choose a winner if we have any */
2148 i = -1; 2166 i = -1;
2149 if (winners > 0) 2167 if (winners > 0)
2150 i = atnr_winner[RANDOM () % winners]; 2168 i = atnr_winner [rndm (winners)];
2151 2169
2152 if (i >= 0 && i < NROFATTACKS && skin->resist[i] < 95) 2170 if (i >= 0 && i < NROFATTACKS && skin->resist[i] < 95)
2153 { 2171 {
2154 /* resistance increased! */ 2172 /* resistance increased! */
2155 skin->resist[i]++; 2173 skin->resist[i]++;
2156 op->update_stats (); 2174 op->update_stats ();
2157 2175
2158 sprintf (buf, "Your skin is now more resistant to %s!", change_resist_msg[i]); 2176 op->statusmsg (format ("Your skin is now more resistant to %s!", change_resist_msg[i]));
2159 new_draw_info (NDI_UNIQUE | NDI_RED, 0, op, buf);
2160 } 2177 }
2161 2178
2162 /* if this flesh contains a new ability focus, we mark it 2179 /* if this flesh contains a new ability focus, we mark it
2163 into the ability_force and it will take effect on next level */ 2180 into the ability_force and it will take effect on next level */
2164 if (meal->last_eat > 0 && atnr_is_dragon_enabled (meal->last_eat) && meal->last_eat != abil->last_eat) 2181 if (meal->last_eat > 0 && atnr_is_dragon_enabled (meal->last_eat) && meal->last_eat != abil->last_eat)
2165 { 2182 {
2166 abil->last_eat = meal->last_eat; /* write: last_eat <new attnr focus> */ 2183 abil->last_eat = meal->last_eat; /* write: last_eat <new attnr focus> */
2167 2184
2168 if (meal->last_eat != abil->stats.exp) 2185 if (meal->last_eat != abil->stats.exp)
2186 op->statusmsg (format (
2187 "Your metabolism prepares to focus on %s!\n"
2188 "The change will happen at level %d.",
2189 change_resist_msg[meal->last_eat],
2190 abil->level + 1
2169 { 2191 ));
2170 sprintf (buf, "Your metabolism prepares to focus on %s!", change_resist_msg[meal->last_eat]);
2171 new_draw_info (NDI_UNIQUE, 0, op, buf);
2172 sprintf (buf, "The change will happen at level %d", abil->level + 1);
2173 new_draw_info (NDI_UNIQUE, 0, op, buf);
2174 }
2175 else 2192 else
2176 { 2193 {
2177 sprintf (buf, "Your metabolism will continue to focus on %s.", change_resist_msg[meal->last_eat]); 2194 op->statusmsg (format ("Your metabolism will continue to focus on %s.", change_resist_msg[meal->last_eat]));
2178 new_draw_info (NDI_UNIQUE, 0, op, buf);
2179 abil->last_eat = 0; 2195 abil->last_eat = 0;
2180 } 2196 }
2181 } 2197 }
2198
2182 return 1; 2199 return 1;
2183} 2200}
2184 2201
2185/** 2202/**
2186 * Handles applying an improve armor scroll. 2203 * Handles applying an improve armor scroll.
… …
2191{ 2208{
2192 object *armor; 2209 object *armor;
2193 2210
2194 if (!QUERY_FLAG (op, FLAG_WIZCAST) && (get_map_flags (op->map, 0, op->x, op->y, 0, 0) & P_NO_MAGIC)) 2211 if (!QUERY_FLAG (op, FLAG_WIZCAST) && (get_map_flags (op->map, 0, op->x, op->y, 0, 0) & P_NO_MAGIC))
2195 { 2212 {
2196 new_draw_info (NDI_UNIQUE, 0, op, "Something blocks the magic of the scroll. H<The area prevents magic effects.>"); 2213 op->failmsg ("Something blocks the magic of the scroll. H<This area prevents magic effects.>");
2197 return; 2214 return;
2198 } 2215 }
2199 2216
2200 armor = find_marked_object (op); 2217 armor = find_marked_object (op);
2201 2218
2202 if (!armor) 2219 if (!armor)
2203 { 2220 {
2204 new_draw_info (NDI_UNIQUE, 0, op, "You need to mark an armor object. Use the right mouse button popup or the mark command to do this."); 2221 op->failmsg ("You need to mark an armor object. Use the right mouse button popup or the mark command to do this.");
2205 return; 2222 return;
2206 } 2223 }
2207 2224
2208 if (armor->type != ARMOUR 2225 if (armor->type != ARMOUR
2209 && armor->type != CLOAK 2226 && armor->type != CLOAK
2210 && armor->type != BOOTS && armor->type != GLOVES && armor->type != BRACERS && armor->type != SHIELD && armor->type != HELMET) 2227 && armor->type != BOOTS && armor->type != GLOVES && armor->type != BRACERS && armor->type != SHIELD && armor->type != HELMET)
2211 { 2228 {
2212 new_draw_info (NDI_UNIQUE, 0, op, "Your marked item is not armour!\n"); 2229 op->failmsg ("Your marked item is not armour!\n");
2213 return; 2230 return;
2214 } 2231 }
2215 2232
2216 new_draw_info (NDI_UNIQUE, 0, op, "Applying armour enchantment."); 2233 op->statusmsg ("Applying armour enchantment.");
2217 improve_armour (op, tmp, armor); 2234 improve_armour (op, tmp, armor);
2218} 2235}
2219 2236
2220extern void 2237void
2221apply_poison (object *op, object *tmp) 2238apply_poison (object *op, object *tmp)
2222{ 2239{
2240 // need to do it now when it is still on the map
2241 handle_apply_yield (tmp);
2242
2243 object *poison = tmp->split (1);
2244
2223 if (op->type == PLAYER) 2245 if (op->type == PLAYER)
2224 { 2246 {
2225 op->contr->play_sound (sound_find ("drink_poison")); 2247 op->contr->play_sound (sound_find ("drink_poison"));
2226 new_draw_info (NDI_UNIQUE, 0, op, "Yech! That tasted poisonous!"); 2248 op->failmsg ("Yech! That tasted poisonous!");
2227 strcpy (op->contr->killer, "poisonous booze"); 2249 op->contr->killer = poison;
2228 } 2250 }
2229 2251
2230 if (tmp->stats.hp > 0) 2252 if (poison->stats.hp > 0)
2231 { 2253 {
2232 LOG (llevDebug, "Trying to poison player/monster for %d hp\n", tmp->stats.hp); 2254 LOG (llevDebug, "Trying to poison player/monster for %d hp\n", poison->stats.hp);
2233 hit_player (op, tmp->stats.hp, tmp, AT_POISON, 1); 2255 hit_player (op, poison->stats.hp, tmp, AT_POISON, 1);
2234 } 2256 }
2235 2257
2236 op->stats.food -= op->stats.food / 4; 2258 op->stats.food -= op->stats.food / 4;
2237 handle_apply_yield (tmp); 2259 poison->destroy ();
2238 decrease_ob (tmp);
2239} 2260}
2240 2261
2241/** 2262/**
2242 * This function return true if the exit is not a 2 ways one or it is 2 ways, valid exit. 2263 * This function return true if the exit is not a 2 ways one or it is 2 ways, valid exit.
2243 * A valid 2 way exit means: 2264 * A valid 2 way exit means:
… …
2258 2279
2259#if 0 //TODO 2280#if 0 //TODO
2260 if (!has_been_loaded (EXIT_PATH (exit)) && exit->race) 2281 if (!has_been_loaded (EXIT_PATH (exit)) && exit->race)
2261 return 0; /* This is a reset town portal */ 2282 return 0; /* This is a reset town portal */
2262#endif 2283#endif
2284
2285 LOG (llevError | logBacktrace, "sync map load due to %s\n", exit->debug_desc ());
2263 2286
2264 maptile *exitmap = maptile::find_sync (EXIT_PATH (exit), exit->map); 2287 maptile *exitmap = maptile::find_sync (EXIT_PATH (exit), exit->map);
2265 2288
2266 if (exitmap) 2289 if (exitmap)
2267 { 2290 {
… …
2343 * them in this function - they are passed to apply_special 2366 * them in this function - they are passed to apply_special
2344 */ 2367 */
2345int 2368int
2346manual_apply (object *op, object *tmp, int aflag) 2369manual_apply (object *op, object *tmp, int aflag)
2347{ 2370{
2348 if (tmp->head)
2349 tmp = tmp->head; 2371 tmp = tmp->head_ ();
2350 2372
2351 if (QUERY_FLAG (tmp, FLAG_UNPAID) && !QUERY_FLAG (tmp, FLAG_APPLIED)) 2373 if (QUERY_FLAG (tmp, FLAG_UNPAID) && !QUERY_FLAG (tmp, FLAG_APPLIED))
2352 { 2374 {
2353 if (op->type == PLAYER) 2375 if (op->type == PLAYER)
2354 { 2376 {
2355 new_draw_info (NDI_UNIQUE, 0, op, "You should pay for it first. H<You cannot use items marked as unpaid.>"); 2377 op->failmsg ("You should pay for it first! H<You cannot use items marked as unpaid.>");
2356 return 1; 2378 return 1;
2357 } 2379 }
2358 else 2380 else
2359 return 0; /* monsters just skip unpaid items */ 2381 return 0; /* monsters just skip unpaid items */
2360 } 2382 }
… …
2363 return RESULT_INT (0); 2385 return RESULT_INT (0);
2364 2386
2365 switch (tmp->type) 2387 switch (tmp->type)
2366 { 2388 {
2367 case CF_HANDLE: 2389 case CF_HANDLE:
2368 new_draw_info (NDI_UNIQUE, 0, op, "You turn the handle.");
2369 op->play_sound (sound_find ("turn_handle")); 2390 op->play_sound (sound_find ("turn_handle"));
2391 op->statusmsg ("You turn the handle.");
2370 tmp->value = tmp->value ? 0 : 1; 2392 tmp->value = tmp->value ? 0 : 1;
2371 SET_ANIMATION (tmp, tmp->value); 2393 SET_ANIMATION (tmp, tmp->value);
2372 update_object (tmp, UP_OBJ_FACE); 2394 update_object (tmp, UP_OBJ_FACE);
2373 push_button (tmp); 2395 push_button (tmp);
2374 return 1; 2396 return 1;
2375 2397
2376 case TRIGGER: 2398 case TRIGGER:
2377 if (check_trigger (tmp, op)) 2399 if (check_trigger (tmp, op))
2378 { 2400 {
2379 new_draw_info (NDI_UNIQUE, 0, op, "You turn the handle."); 2401 op->statusmsg ("You turn the handle.");
2380 op->play_sound (sound_find ("turn_handle")); 2402 op->play_sound (sound_find ("turn_handle"));
2381 } 2403 }
2382 else 2404 else
2383 new_draw_info (NDI_UNIQUE, 0, op, "The handle doesn't move."); 2405 op->failmsg ("The handle doesn't move.");
2384 2406
2385 return 1; 2407 return 1;
2386 2408
2387 case EXIT: 2409 case EXIT:
2388 if (op->type != PLAYER) 2410 if (op->type != PLAYER)
2389 return 0; 2411 return 0;
2390 2412
2391 if (!EXIT_PATH (tmp) || !is_legal_2ways_exit (op, tmp)) 2413 if (!EXIT_PATH (tmp) || !is_legal_2ways_exit (op, tmp))
2392 new_draw_info_format (NDI_UNIQUE, 0, op, "The %s is closed.", query_name (tmp)); 2414 op->failmsg (format ("The %s is closed.", query_name (tmp)));
2393 else 2415 else
2394 { 2416 {
2395 /* Don't display messages for random maps. */ 2417 /* Don't display messages for random maps. */
2396 if (tmp->msg && strncmp (EXIT_PATH (tmp), "/!", 2)) 2418 if (tmp->msg && strncmp (EXIT_PATH (tmp), "/!", 2))
2397 new_draw_info (NDI_NAVY, 0, op, tmp->msg); 2419 op->statusmsg (tmp->msg, NDI_NAVY);
2398 2420
2399 op->enter_exit (tmp); 2421 op->enter_exit (tmp);
2400 } 2422 }
2401 2423
2424 return 1;
2425
2426 case INSCRIBABLE:
2427 op->statusmsg (tmp->msg);
2428 // maybe show a spell menu to chose from or something like that
2402 return 1; 2429 return 1;
2403 2430
2404 case SIGN: 2431 case SIGN:
2405 apply_sign (op, tmp, 0); 2432 apply_sign (op, tmp, 0);
2406 return 1; 2433 return 1;
… …
2515 { 2542 {
2516 char buf[MAX_BUF]; 2543 char buf[MAX_BUF];
2517 timeofday_t tod; 2544 timeofday_t tod;
2518 2545
2519 get_tod (&tod); 2546 get_tod (&tod);
2520 sprintf (buf, "It is %d minute%s past %d o'clock %s",
2521 tod.minute + 1, ((tod.minute + 1 < 2) ? "" : "s"),
2522 ((tod.hour % 14 == 0) ? 14 : ((tod.hour) % 14)), ((tod.hour >= 14) ? "pm" : "am"));
2523 op->play_sound (sound_find ("sound_clock")); 2547 op->play_sound (sound_find ("sound_clock"));
2524 new_draw_info (NDI_UNIQUE, 0, op, buf); 2548 op->statusmsg (format (
2549 "It is %d minute%s past %d o'clock %s",
2550 tod.minute + 1, ((tod.minute + 1 < 2) ? "" : "s"),
2551 ((tod.hour % 14 == 0) ? 14 : ((tod.hour) % 14)), ((tod.hour >= 14) ? "pm" : "am")
2552 ));
2525 return 1; 2553 return 1;
2526 } 2554 }
2527 else 2555 else
2528 return 0; 2556 return 0;
2529 2557
… …
2556 default: 2584 default:
2557 return 0; 2585 return 0;
2558 } 2586 }
2559} 2587}
2560 2588
2561
2562/* quiet suppresses the "don't know how to apply" and "you must get it first" 2589/* quiet suppresses the "don't know how to apply" and "you must get it first"
2563 * messages as needed by player_apply_below(). But there can still be 2590 * messages as needed by player_apply_below(). But there can still be
2564 * "but you are floating high above the ground" messages. 2591 * "but you are floating high above the ground" messages.
2565 * 2592 *
2566 * Same return value as apply() function. 2593 * Same return value as apply() function.
2567 */ 2594 */
2568int 2595int
2569player_apply (object *pl, object *op, int aflag, int quiet) 2596player_apply (object *pl, object *op, int aflag, int quiet)
2570{ 2597{
2571 int tmp;
2572
2573 if (op->env && (pl->move_type & MOVE_FLYING)) 2598 if (!op->env && (pl->move_type & MOVE_FLYING))
2574 { 2599 {
2575 /* player is flying and applying object not in inventory */ 2600 /* player is flying and applying object not in inventory */
2576 if (!QUERY_FLAG (pl, FLAG_WIZ) && !(op->move_type & MOVE_FLYING)) 2601 if (!QUERY_FLAG (pl, FLAG_WIZ) && !(op->move_type & MOVE_FLYING))
2577 { 2602 {
2578 new_draw_info (NDI_UNIQUE, 0, pl, "But you are floating high above the ground! H<You have to stop levitating first, if you can.>"); 2603 pl->failmsg ("But you are floating high above the ground! "
2604 "H<You have to stop levitating first, if you can, either by using your levitation skill, "
2605 "or waiting till the levitation effect wears off.>");
2579 return 0; 2606 return 0;
2580 } 2607 }
2581 } 2608 }
2582 2609
2583 pl->contr->last_used = op; 2610 pl->contr->last_used = op;
2584 2611
2585 tmp = manual_apply (pl, op, aflag); 2612 int tmp = manual_apply (pl, op, aflag);
2613
2586 if (!quiet) 2614 if (!quiet)
2587 { 2615 {
2588 if (tmp == 0) 2616 if (tmp == 0)
2589 new_draw_info_format (NDI_UNIQUE, 0, pl, "I don't know how to apply the %s.", query_name (op)); 2617 pl->statusmsg (format ("I don't know how to apply the %s.", query_name (op)));
2590 else if (tmp == 2) 2618 else if (tmp == 2)
2591 new_draw_info_format (NDI_UNIQUE, 0, pl, "You must get it first!\n"); 2619 pl->failmsg ("You must get it first!\n");
2592 } 2620 }
2593 2621
2594 return tmp; 2622 return tmp;
2595} 2623}
2596 2624
… …
2625 * person moving on it, also activate. Added code to make it 2653 * person moving on it, also activate. Added code to make it
2626 * so that at least one of players movement types be that which 2654 * so that at least one of players movement types be that which
2627 * the item needs. 2655 * the item needs.
2628 */ 2656 */
2629 if (!tmp->invisible || (tmp->move_on & pl->move_type)) 2657 if (!tmp->invisible || (tmp->move_on & pl->move_type))
2630 {
2631 if (player_apply (pl, tmp, 0, 1) == 1) 2658 if (player_apply (pl, tmp, 0, 1) == 1)
2632 return; 2659 return;
2633 } 2660
2634 if (floors >= 2) 2661 if (floors >= 2)
2635 return; /* process at most two floor objects */ 2662 return; /* process at most two floor objects */
2636 } 2663 }
2637} 2664}
2638 2665
… …
2673 { 2700 {
2674 pl->combat_ob = 0; 2701 pl->combat_ob = 0;
2675 who->change_weapon (pl->ranged_ob); 2702 who->change_weapon (pl->ranged_ob);
2676 } 2703 }
2677 2704
2678 new_draw_info_format (NDI_UNIQUE, 0, who, "You unwield %s.", query_name (op)); 2705 who->statusmsg (format ("You unwield %s.", query_name (op)));
2679 2706
2680 change_abil (who, op); 2707 change_abil (who, op);
2681 CLEAR_FLAG (who, FLAG_READY_WEAPON); 2708 CLEAR_FLAG (who, FLAG_READY_WEAPON);
2682 break; 2709 break;
2683 2710
2684 case SKILL: 2711 case SKILL:
2685 if (who->contr) 2712 if (who->contr)
2686 { 2713 {
2714 if (IS_COMBAT_SKILL (op->subtype))
2715 who->change_weapon (who->contr->combat_ob = 0);
2716 else if (IS_RANGED_SKILL (op->subtype))
2717 who->change_weapon (who->contr->ranged_ob = 0);
2718
2687 if (!op->invisible) 2719 if (op->invisible)
2688 new_draw_info_format (NDI_UNIQUE, 0, who, "You stop using the %s.", query_name (op)); 2720 who->statusmsg (format ("You can no longer use the skill: %s.", &op->skill));
2689 else 2721 else
2690 new_draw_info_format (NDI_UNIQUE, 0, who, "You can no longer use the skill: %s.", &op->skill); 2722 who->statusmsg (format ("You stop using the %s.", query_name (op)));
2691 } 2723 }
2692 2724
2693 change_abil (who, op); 2725 change_abil (who, op);
2694 CLEAR_FLAG (who, FLAG_READY_SKILL); 2726 CLEAR_FLAG (who, FLAG_READY_SKILL);
2695 break; 2727 break;
… …
2702 case GLOVES: 2734 case GLOVES:
2703 case AMULET: 2735 case AMULET:
2704 case GIRDLE: 2736 case GIRDLE:
2705 case BRACERS: 2737 case BRACERS:
2706 case CLOAK: 2738 case CLOAK:
2707 new_draw_info_format (NDI_UNIQUE, 0, who, "You unwear %s.", query_name (op)); 2739 who->statusmsg (format ("You unwear %s.", query_name (op)));
2708 change_abil (who, op); 2740 change_abil (who, op);
2709 break; 2741 break;
2710 2742
2711 case LAMP: 2743 case LAMP:
2712 { 2744 {
2713 new_draw_info_format (NDI_UNIQUE, 0, who, "You turn off your %s.", &op->name); 2745 who->statusmsg (format ("You turn off your %s.", &op->name));
2714 2746
2715 object *tmp2 = arch_to_object (op->other_arch); 2747 object *tmp2 = arch_to_object (op->other_arch);
2716 tmp2->x = op->x; 2748 tmp2->x = op->x;
2717 tmp2->y = op->y; 2749 tmp2->y = op->y;
2718 tmp2->map = op->map; 2750 tmp2->map = op->map;
… …
2722 CLEAR_FLAG (tmp2, FLAG_APPLIED); 2754 CLEAR_FLAG (tmp2, FLAG_APPLIED);
2723 2755
2724 if (QUERY_FLAG (op, FLAG_INV_LOCKED)) 2756 if (QUERY_FLAG (op, FLAG_INV_LOCKED))
2725 SET_FLAG (tmp2, FLAG_INV_LOCKED); 2757 SET_FLAG (tmp2, FLAG_INV_LOCKED);
2726 2758
2759 op->destroy ();
2760 who->insert (tmp2);
2761 who->update_stats ();
2762
2727 if (who->contr) 2763 if (who->contr)
2728 esrv_del_item (who->contr, op->count);
2729
2730 op->destroy ();
2731 insert_ob_in_ob (tmp2, who);
2732 who->update_stats ();
2733
2734 if (QUERY_FLAG (op, FLAG_CURSED) || QUERY_FLAG (op, FLAG_DAMNED)) 2764 if (QUERY_FLAG (op, FLAG_CURSED) || QUERY_FLAG (op, FLAG_DAMNED))
2735 {
2736 if (who->contr)
2737 { 2765 {
2738 new_draw_info (NDI_UNIQUE, 0, who, "Oops, it feels deadly cold!"); 2766 who->failmsg ("Oops, it feels deadly cold!");
2739 SET_FLAG (tmp2, FLAG_KNOWN_CURSED); 2767 SET_FLAG (tmp2, FLAG_KNOWN_CURSED);
2740 } 2768 }
2741 }
2742
2743 if (who->contr)
2744 esrv_send_item (who, tmp2);
2745 } 2769 }
2746 2770
2747 return 1; /* otherwise, an attempt to drop causes problems */ 2771 return 1; /* otherwise, an attempt to drop causes problems */
2748 2772
2749 case BOW: 2773 case BOW:
… …
2756 { 2780 {
2757 pl->ranged_ob = 0; 2781 pl->ranged_ob = 0;
2758 who->change_weapon (pl->combat_ob); 2782 who->change_weapon (pl->combat_ob);
2759 } 2783 }
2760 2784
2761 new_draw_info_format (NDI_UNIQUE, 0, who, "You unready %s.", query_name (op)); 2785 who->statusmsg (format ("You unready %s.", query_name (op)));
2762 } 2786 }
2763 else 2787 else
2764 { 2788 {
2765 who->change_skill (0); 2789 who->change_skill (0);
2766 2790
… …
2772 2796
2773 break; 2797 break;
2774 2798
2775 case BUILDER: 2799 case BUILDER:
2776 if (who->contr) 2800 if (who->contr)
2777 new_draw_info_format (NDI_UNIQUE, 0, who, "You unready %s.", query_name (op)); 2801 who->statusmsg (format ("You unready %s.", query_name (op)));
2778 break; 2802 break;
2779 2803
2780 default: 2804 default:
2781 new_draw_info_format (NDI_UNIQUE, 0, who, "You unapply %s.", query_name (op)); 2805 who->statusmsg (format ("You unapply %s.", query_name (op)));
2782 break; 2806 break;
2783 } 2807 }
2784 2808
2809 if (aflags & AP_NO_MERGE || !merge_ob (op, 0))
2810 if (object *pl = op->visible_to ())
2811 esrv_send_item (pl, op);
2812
2785 who->update_stats (); 2813 who->update_stats ();
2786
2787 if (!(aflags & AP_NO_MERGE))
2788 {
2789 object *tmp = merge_ob (op, 0);
2790
2791 if (who->contr)
2792 {
2793 if (tmp)
2794 { /* it was merged */
2795 esrv_del_item (who->contr, op->count);
2796 op = tmp;
2797 }
2798
2799 esrv_send_item (who, op);
2800 }
2801 }
2802 2814
2803 return 0; 2815 return 0;
2804} 2816}
2805 2817
2806/** 2818/**
… …
2853 for (object *tmp = who->inv; tmp; tmp = tmp->below) 2865 for (object *tmp = who->inv; tmp; tmp = tmp->below)
2854 if (QUERY_FLAG (tmp, FLAG_APPLIED) && tmp->is_range ()) 2866 if (QUERY_FLAG (tmp, FLAG_APPLIED) && tmp->is_range ())
2855 if ((aflags & AP_IGNORE_CURSE) || (aflags & AP_PRINT) || (!QUERY_FLAG (tmp, FLAG_CURSED) && !QUERY_FLAG (tmp, FLAG_DAMNED))) 2867 if ((aflags & AP_IGNORE_CURSE) || (aflags & AP_PRINT) || (!QUERY_FLAG (tmp, FLAG_CURSED) && !QUERY_FLAG (tmp, FLAG_DAMNED)))
2856 { 2868 {
2857 if (aflags & AP_PRINT) 2869 if (aflags & AP_PRINT)
2858 new_draw_info (NDI_UNIQUE, 0, who, query_name (tmp)); 2870 who->failmsg (query_name (tmp));
2859 else 2871 else
2860 unapply_special (who, tmp, aflags); 2872 unapply_special (who, tmp, aflags);
2861 } 2873 }
2862 else 2874 else
2863 { 2875 {
2864 /* In this case, we want to try and remove a cursed item. 2876 /* In this case, we want to try and remove a cursed item.
2865 * While we know it won't work, we want unapply_special to 2877 * While we know it won't work, we want unapply_special to
2866 * at least generate the message. 2878 * at least generate the message.
2867 */ 2879 */
2868 new_draw_info_format (NDI_UNIQUE, 0, who, 2880 who->failmsg (format ("No matter how hard you try, you just can't remove the %s." CANNOT_REMOVE_CURSED, query_name (tmp)));
2869 "No matter how hard you try, you just can't remove the %s." CANNOT_REMOVE_CURSED,
2870 query_name (tmp));
2871 return 1; 2881 return 1;
2872 } 2882 }
2873 2883
2874 for (int i = 0; i < NUM_BODY_LOCATIONS; i++) 2884 for (int i = 0; i < NUM_BODY_LOCATIONS; i++)
2875 { 2885 {
… …
2898 2908
2899 /* If we are just printing, we don't care about cursed status */ 2909 /* If we are just printing, we don't care about cursed status */
2900 if ((aflags & AP_IGNORE_CURSE) || (aflags & AP_PRINT) || (!(QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED)))) 2910 if ((aflags & AP_IGNORE_CURSE) || (aflags & AP_PRINT) || (!(QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED))))
2901 { 2911 {
2902 if (aflags & AP_PRINT) 2912 if (aflags & AP_PRINT)
2903 new_draw_info (NDI_UNIQUE, 0, who, query_name (tmp)); 2913 who->failmsg (query_name (tmp));
2904 else 2914 else
2905 unapply_special (who, tmp, aflags); 2915 unapply_special (who, tmp, aflags);
2906 } 2916 }
2907 else 2917 else
2908 { 2918 {
2909 /* Cursed item that we can't unequip - tell the player. 2919 /* Cursed item that we can't unequip - tell the player.
2910 * Note this could be annoying if this is just one of a few, 2920 * Note this could be annoying if this is just one of a few,
2911 * so it may not be critical (eg, putting on a ring and you have 2921 * so it may not be critical (eg, putting on a ring and you have
2912 * one cursed ring.) 2922 * one cursed ring.)
2913 */ 2923 */
2914 new_draw_info_format (NDI_UNIQUE, 0, who, "The %s just won't come off." CANNOT_REMOVE_CURSED, query_name (tmp)); 2924 who->failmsg (format ("The %s just won't come off." CANNOT_REMOVE_CURSED, query_name (tmp)));
2915 } 2925 }
2916 2926
2917 last = tmp->below; 2927 last = tmp->below;
2918 } 2928 }
2919 /* if we got here, this slot is freed up - otherwise, if it wasn't freed up, the 2929 /* if we got here, this slot is freed up - otherwise, if it wasn't freed up, the
… …
3100 if (basic_flag == AP_APPLY) 3110 if (basic_flag == AP_APPLY)
3101 return 0; 3111 return 0;
3102 3112
3103 if (!(aflags & AP_IGNORE_CURSE) && (QUERY_FLAG (op, FLAG_CURSED) || QUERY_FLAG (op, FLAG_DAMNED))) 3113 if (!(aflags & AP_IGNORE_CURSE) && (QUERY_FLAG (op, FLAG_CURSED) || QUERY_FLAG (op, FLAG_DAMNED)))
3104 { 3114 {
3105 new_draw_info_format (NDI_UNIQUE, 0, who, 3115 who->failmsg (format ("No matter how hard you try, you just can't remove %s." CANNOT_REMOVE_CURSED, query_name (op)));
3106 "No matter how hard you try, you just can't remove %s." CANNOT_REMOVE_CURSED,
3107 query_name (op));
3108 return 1; 3116 return 1;
3109 } 3117 }
3110 3118
3111 return unapply_special (who, op, aflags); 3119 return unapply_special (who, op, aflags);
3112 } 3120 }
3113
3114 if (basic_flag == AP_UNAPPLY) 3121 else if (basic_flag == AP_UNAPPLY)
3115 return 0; 3122 return 0;
3116 3123
3117 // if the item is combat/ranged, wield the relevant slot first 3124 // if the item is combat/ranged, wield the relevant slot first
3118 // to resolve conflicts. 3125 // to resolve conflicts.
3119 if (player *pl = who->contr) 3126 if (player *pl = who->contr)
… …
3128 /* Can't just apply this object. Lets see what not and what to do */ 3135 /* Can't just apply this object. Lets see what not and what to do */
3129 if (int i = can_apply_object (who, op)) 3136 if (int i = can_apply_object (who, op))
3130 { 3137 {
3131 if (i & CAN_APPLY_NEVER) 3138 if (i & CAN_APPLY_NEVER)
3132 { 3139 {
3133 new_draw_info_format (NDI_UNIQUE, 0, who, 3140 who->failmsg (format ("You don't have the body to use a %s. H<You can never apply this item.>", query_name (op)));
3134 "You don't have the body to use a %s. H<You can never apply this item.>",
3135 query_name (op));
3136 return 1; 3141 return 1;
3137 } 3142 }
3138 else if (i & CAN_APPLY_RESTRICTION) 3143 else if (i & CAN_APPLY_RESTRICTION)
3139 { 3144 {
3140 new_draw_info_format (NDI_UNIQUE, 0, who, 3145 who->failmsg (format (
3141 "You have a prohibition against using a %s. " 3146 "You have a prohibition against using a %s. "
3142 "H<Your belief, profession or class prevents you from applying this item.>", 3147 "H<Your belief, profession or class prevents you from applying this item.>",
3143 query_name (op)); 3148 query_name (op)
3149 ));
3144 return 1; 3150 return 1;
3145 } 3151 }
3146 3152
3147 if (who->type != PLAYER) 3153 if (who->type != PLAYER)
3148 { 3154 {
… …
3152 } 3158 }
3153 else 3159 else
3154 { 3160 {
3155 if (who->contr->unapply == unapply_never || (i & CAN_APPLY_UNAPPLY_CHOICE && who->contr->unapply == unapply_nochoice)) 3161 if (who->contr->unapply == unapply_never || (i & CAN_APPLY_UNAPPLY_CHOICE && who->contr->unapply == unapply_nochoice))
3156 { 3162 {
3157 new_draw_info (NDI_UNIQUE, 0, who, "You need to unapply some of the following item(s) or change your applymode:"); 3163 who->failmsg ("You need to unapply some of the following item(s) or change your applymode:");
3158 unapply_for_ob (who, op, AP_PRINT); 3164 unapply_for_ob (who, op, AP_PRINT);
3159 return 1; 3165 return 1;
3160 } 3166 }
3161 else if (who->contr->unapply == unapply_always || !(i & CAN_APPLY_UNAPPLY_CHOICE)) 3167 else if (who->contr->unapply == unapply_always || !(i & CAN_APPLY_UNAPPLY_CHOICE))
3162 if (unapply_for_ob (who, op, aflags)) 3168 if (unapply_for_ob (who, op, aflags))
… …
3168 { 3174 {
3169 skop = find_skill_by_name (who, op->skill); 3175 skop = find_skill_by_name (who, op->skill);
3170 3176
3171 if (!skop) 3177 if (!skop)
3172 { 3178 {
3173 new_draw_info_format (NDI_UNIQUE, 0, who, "You need the %s skill to use this item!", &op->skill); 3179 who->failmsg (format ("You need the %s skill to use this item!", &op->skill));
3174 return 1; 3180 return 1;
3175 } 3181 }
3176 else 3182 else
3177 /* While experience will be credited properly, we want to change the 3183 /* While experience will be credited properly, we want to change the
3178 * skill so that the dam and wc get updated 3184 * skill so that the dam and wc get updated
… …
3182 3188
3183 if (who->type == PLAYER 3189 if (who->type == PLAYER
3184 && op->item_power 3190 && op->item_power
3185 && op->item_power + who->contr->item_power > settings.item_power_factor * who->level) 3191 && op->item_power + who->contr->item_power > settings.item_power_factor * who->level)
3186 { 3192 {
3187 new_draw_info (NDI_UNIQUE, 0, who,
3188 "Equipping that combined with other items would consume your soul!" LACK_ITEM_POWER); 3193 who->failmsg ("Equipping that combined with other items would consume your soul!" LACK_ITEM_POWER);
3189 return 1; 3194 return 1;
3190 } 3195 }
3191 3196
3192 /* Ok. We are now at the state where we can apply the new object. 3197 /* Ok. We are now at the state where we can apply the new object.
3193 * Note that we don't have the checks for can_use_... 3198 * Note that we don't have the checks for can_use_...
3194 * below - that is already taken care of by can_apply_object. 3199 * below - that is already taken care of by can_apply_object.
3195 */ 3200 */
3196 if (op->nrof > 1) 3201 tmp = op->nrof > 1 ? op->split (op->nrof - 1) : 0;
3197 tmp = get_split_ob (op, op->nrof - 1);
3198 else
3199 tmp = 0;
3200 3202
3201 if (INVOKE_OBJECT (BE_READY, op, ARG_OBJECT (who)) || INVOKE_OBJECT (READY, who, ARG_OBJECT (op))) 3203 if (INVOKE_OBJECT (BE_READY, op, ARG_OBJECT (who)) || INVOKE_OBJECT (READY, who, ARG_OBJECT (op)))
3202 return RESULT_INT (0); 3204 return RESULT_INT (0);
3203 3205
3204 switch (op->type) 3206 switch (op->type)
3205 { 3207 {
3206 case WEAPON: 3208 case WEAPON:
3207 if (!check_weapon_power (who, op->last_eat)) 3209 if (!check_weapon_power (who, op->last_eat))
3208 { 3210 {
3209 new_draw_info (NDI_UNIQUE, 0, who, "This weapon is too powerful for you to use. " 3211 op->failmsg ("This weapon is too powerful for you to use. It would consume your soul!" LACK_ITEM_POWER);
3210 "It would consume your soul!." LACK_ITEM_POWER);
3211 3212
3212 if (tmp) 3213 if (tmp)
3213 insert_ob_in_ob (tmp, who); 3214 insert_ob_in_ob (tmp, who);
3214 3215
3215 return 1; 3216 return 1;
… …
3219 // i.e. "R" can use Ragnarok's sword. 3220 // i.e. "R" can use Ragnarok's sword.
3220 if (op->level && (strncmp (op->name, who->name, strlen (who->name)))) 3221 if (op->level && (strncmp (op->name, who->name, strlen (who->name))))
3221 { 3222 {
3222 /* if the weapon does not have the name as the character, can't use it. */ 3223 /* if the weapon does not have the name as the character, can't use it. */
3223 /* (Ragnarok's sword attempted to be used by Foo: won't work) */ 3224 /* (Ragnarok's sword attempted to be used by Foo: won't work) */
3224 new_draw_info (NDI_UNIQUE, 0, who, 3225 who->failmsg ("The weapon does not recognize you as its owner. H<Its name indicates that it belongs to somebody else.>");
3225 "The weapon does not recognize you as its owner. "
3226 "H<Its name indicates that it belongs to somebody else.>");
3227 3226
3228 if (tmp) 3227 if (tmp)
3229 insert_ob_in_ob (tmp, who); 3228 insert_ob_in_ob (tmp, who);
3230 3229
3231 return 1; 3230 return 1;
3232 } 3231 }
3233 3232
3234 if (!skop) 3233 if (!skop)
3235 { 3234 {
3236 new_draw_info_format (NDI_UNIQUE, 0, who, "The %s is broken, please report this to the dungeon master!", query_name (op));//TODO 3235 who->failmsg (format ("The %s is broken, please report this to the dungeon master!", query_name (op)));//TODO
3237 return 1; 3236 return 1;
3238 } 3237 }
3239 3238
3240 SET_FLAG (op, FLAG_APPLIED); 3239 SET_FLAG (op, FLAG_APPLIED);
3241 who->change_skill (skop); 3240 who->change_skill (skop);
3242 3241
3243 if (who->contr) 3242 if (who->contr)
3244 who->change_weapon (who->contr->combat_ob = op); 3243 who->change_weapon (who->contr->combat_ob = op);
3245 3244
3246 new_draw_info_format (NDI_UNIQUE, 0, who, "You wield %s.", query_name (op)); 3245 who->statusmsg (format ("You wield %s.", query_name (op)));
3247 3246
3248 SET_FLAG (who, FLAG_READY_WEAPON); 3247 SET_FLAG (who, FLAG_READY_WEAPON);
3249 change_abil (who, op); 3248 change_abil (who, op);
3250 break; 3249 break;
3251 3250
… …
3258 case BRACERS: 3257 case BRACERS:
3259 case CLOAK: 3258 case CLOAK:
3260 case RING: 3259 case RING:
3261 case AMULET: 3260 case AMULET:
3262 SET_FLAG (op, FLAG_APPLIED); 3261 SET_FLAG (op, FLAG_APPLIED);
3263 new_draw_info_format (NDI_UNIQUE, 0, who, "You wear %s.", query_name (op)); 3262 who->statusmsg (format ("You wear %s.", query_name (op)));
3264 change_abil (who, op); 3263 change_abil (who, op);
3265 break; 3264 break;
3266 3265
3267 case LAMP: 3266 case LAMP:
3268 if (op->stats.food < 1) 3267 if (op->stats.food < 1)
3269 { 3268 {
3270 new_draw_info_format (NDI_UNIQUE, 0, who, 3269 who->failmsg (format (
3271 "Your %s is out of fuel! " 3270 "Your %s is out of fuel! "
3272 "H<Lamps and similar items need fuel. They cannot be refilled.>", &op->name); 3271 "H<Lamps and similar items need fuel. They cannot be refilled.>",
3272 &op->name
3273 ));
3273 return 1; 3274 return 1;
3274 } 3275 }
3275 3276
3276 new_draw_info_format (NDI_UNIQUE, 0, who, "You turn on your %s.", &op->name); 3277 who->statusmsg (format ("You turn on your %s.", &op->name));
3278
3277 tmp2 = arch_to_object (op->other_arch); 3279 tmp2 = arch_to_object (op->other_arch);
3278 tmp2->stats.food = op->stats.food; 3280 tmp2->stats.food = op->stats.food;
3279 SET_FLAG (tmp2, FLAG_APPLIED); 3281 SET_FLAG (tmp2, FLAG_APPLIED);
3280 3282
3281 if (QUERY_FLAG (op, FLAG_INV_LOCKED)) 3283 if (QUERY_FLAG (op, FLAG_INV_LOCKED))
3282 SET_FLAG (tmp2, FLAG_INV_LOCKED); 3284 SET_FLAG (tmp2, FLAG_INV_LOCKED);
3283 3285
3284 insert_ob_in_ob (tmp2, who); 3286 who->insert (tmp2);
3285 3287
3286 /* Remove the old lantern */ 3288 /* Remove the old lantern */
3287 if (who->type == PLAYER)
3288 esrv_del_item (who->contr, op->count);
3289
3290 op->destroy (); 3289 op->destroy ();
3291 3290
3292 /* insert the portion that was split off */ 3291 /* insert the portion that was split off */
3293 if (tmp) 3292 if (tmp)
3294 { 3293 who->insert (tmp);
3295 insert_ob_in_ob (tmp, who);
3296 if (who->type == PLAYER)
3297 esrv_send_item (who, tmp);
3298 }
3299 3294
3300 who->update_stats (); 3295 who->update_stats ();
3301 3296
3302 if (QUERY_FLAG (op, FLAG_CURSED) || QUERY_FLAG (op, FLAG_DAMNED)) 3297 if (QUERY_FLAG (op, FLAG_CURSED) || QUERY_FLAG (op, FLAG_DAMNED))
3303 if (who->type == PLAYER) 3298 if (who->type == PLAYER)
3304 { 3299 {
3305 new_draw_info (NDI_UNIQUE, 0, who, 3300 who->failmsg ("Oops, it feels deadly cold! H<Maybe it wasn't such a bright (hrhrhr) idea to apply this cursed or damned item.>");
3306 "Oops, it feels deadly cold! "
3307 "H<Maybe it wasn't such a bright idea to apply this cursed/damned item.>");
3308 SET_FLAG (tmp2, FLAG_KNOWN_CURSED); 3301 SET_FLAG (tmp2, FLAG_KNOWN_CURSED);
3309 } 3302 }
3310
3311 if (who->type == PLAYER)
3312 esrv_send_item (who, tmp2);
3313 3303
3314 return 0; 3304 return 0;
3315 3305
3316 case SKILL_TOOL: 3306 case SKILL_TOOL:
3317 // applying a skill tool also readies the skill 3307 // applying a skill tool also readies the skill
… …
3340 who->change_weapon (pl->combat_ob = item); 3330 who->change_weapon (pl->combat_ob = item);
3341 goto found_weapon; 3331 goto found_weapon;
3342 } 3332 }
3343 } 3333 }
3344 3334
3345 new_draw_info_format (NDI_UNIQUE, 0, who, 3335 who->failmsg (format (
3346 "You need to apply a '%s' melee weapon before readying this skill. " 3336 "You need to apply a '%s' melee weapon before readying this skill. "
3347 "H<Some skills need an item, in this case a melee weapon, to function.>", 3337 "H<Some skills need an item, in this case a melee weapon, to function.>",
3348 &op->skill); 3338 &op->skill
3339 ));
3349 return 1; 3340 return 1;
3350 3341
3351 found_weapon:; 3342 found_weapon:;
3352 } 3343 }
3353 else 3344 else
… …
3363 //TODO: bows should/must all have skill missile weapon right now 3354 //TODO: bows should/must all have skill missile weapon right now
3364 who->change_weapon (pl->ranged_ob = item); 3355 who->change_weapon (pl->ranged_ob = item);
3365 goto found_bow; 3356 goto found_bow;
3366 } 3357 }
3367 3358
3368 new_draw_info (NDI_UNIQUE, 0, who, 3359 who->failmsg (
3369 "You need to apply a missile weapon before readying this skill. " 3360 "You need to apply a missile weapon before readying this skill. "
3370 "H<Some skills need an item, in this case a missile weapon, to function.>"); 3361 "H<Some skills need an item, in this case a missile weapon, to function.>"
3362 );
3371 return 1; 3363 return 1;
3372 3364
3373 found_bow:; 3365 found_bow:;
3374 } 3366 }
3375 else 3367 else
3376 who->change_weapon (pl->ranged_ob = op); 3368 who->change_weapon (pl->ranged_ob = op);
3377 } 3369 }
3378 3370
3379 if (!op->invisible) 3371 if (!op->invisible)
3380 { 3372 {
3381 new_draw_info_format (NDI_UNIQUE, 0, who, "You ready %s.", query_name (op)); 3373 who->statusmsg (format (
3382 new_draw_info_format (NDI_UNIQUE, 0, who, "You can now use the skill: %s.", &op->skill); 3374 "You ready %s."
3375 "You can now use the skill: %s.",
3376 query_name (op),
3377 &op->skill
3378 ));
3383 } 3379 }
3384 else 3380 else
3385 new_draw_info_format (NDI_UNIQUE, 0, who, "Readied skill: %s.", op->skill ? &op->skill : &op->name); 3381 who->statusmsg (format ("Readied skill: %s.", op->skill ? &op->skill : &op->name));
3386 } 3382 }
3387 else 3383 else
3388 { 3384 {
3389 SET_FLAG (op, FLAG_APPLIED); 3385 SET_FLAG (op, FLAG_APPLIED);
3390 change_abil (who, op); 3386 change_abil (who, op);
… …
3395 break; 3391 break;
3396 3392
3397 case BOW: 3393 case BOW:
3398 if (!check_weapon_power (who, op->last_eat)) 3394 if (!check_weapon_power (who, op->last_eat))
3399 { 3395 {
3400 new_draw_info (NDI_UNIQUE, 0, who,
3401 "That weapon is too powerful for you to use. It would consume your soul!" LACK_ITEM_POWER); 3396 who->failmsg ("That weapon is too powerful for you to use. It would consume your soul!" LACK_ITEM_POWER);
3402 3397
3403 if (tmp) 3398 if (tmp)
3404 insert_ob_in_ob (tmp, who); 3399 insert_ob_in_ob (tmp, who);
3405 3400
3406 return 1; 3401 return 1;
3407 } 3402 }
3408 3403
3409 if (op->level && (strncmp (op->name, who->name, strlen (who->name)))) 3404 if (op->level && (strncmp (op->name, who->name, strlen (who->name))))
3410 { 3405 {
3411 new_draw_info (NDI_UNIQUE, 0, who,
3412 "The weapon does not recognize you as its owner. " 3406 who->failmsg ("The weapon does not recognize you as its owner. "
3413 "H<Its name indicates that it belongs to somebody else.>"); 3407 "H<Its name indicates that it belongs to somebody else.>");
3414 if (tmp) 3408 if (tmp)
3415 insert_ob_in_ob (tmp, who); 3409 insert_ob_in_ob (tmp, who);
3416 3410
3417 return 1; 3411 return 1;
… …
3423 case HORN: 3417 case HORN:
3424 /* check for skill, alter player status */ 3418 /* check for skill, alter player status */
3425 3419
3426 if (!skop) 3420 if (!skop)
3427 { 3421 {
3428 new_draw_info_format (NDI_UNIQUE, 0, who, "The %s is broken, please report this to the dungeon master!", query_name (op));//TODO 3422 who->failmsg (format ("The %s is broken, please report this to the dungeon master!", query_name (op)));//TODO
3429 return 1; 3423 return 1;
3430 } 3424 }
3431 3425
3432 SET_FLAG (op, FLAG_APPLIED); 3426 SET_FLAG (op, FLAG_APPLIED);
3433 who->change_skill (skop); 3427 who->change_skill (skop);
3434 3428
3435 if (who->contr) 3429 if (who->contr)
3436 { 3430 {
3437 who->contr->ranged_ob = op; 3431 who->contr->ranged_ob = op;
3438 3432
3439 new_draw_info_format (NDI_UNIQUE, 0, who, "You ready %s.", query_name (op)); 3433 who->statusmsg (format ("You ready %s.", query_name (op)));
3440 3434
3441 if (op->type == BOW) 3435 if (op->type == BOW)
3442 { 3436 {
3443 who->current_weapon = op; 3437 who->current_weapon = op;
3444 change_abil (who, op); 3438 change_abil (who, op);
3445 new_draw_info_format (NDI_UNIQUE, 0, who,
3446 "You will now fire %s with %s.", op->race ? &op->race : "nothing", query_name (op)); 3439 who->statusmsg (format ("You will now fire %s with %s.", op->race ? &op->race : "nothing", query_name (op)));
3447 } 3440 }
3448 } 3441 }
3449 else 3442 else
3450 { 3443 {
3451 if (op->type == BOW) 3444 if (op->type == BOW)
… …
3461 { 3454 {
3462 //TODO: wtf does this do? shouldn't this be managed automatically (slots?) 3455 //TODO: wtf does this do? shouldn't this be managed automatically (slots?)
3463 if (who->contr->ranged_ob && who->contr->ranged_ob->type == BUILDER) 3456 if (who->contr->ranged_ob && who->contr->ranged_ob->type == BUILDER)
3464 unapply_special (who, who->contr->ranged_ob, 0); 3457 unapply_special (who, who->contr->ranged_ob, 0);
3465 3458
3466 new_draw_info_format (NDI_UNIQUE, 0, who, "You ready your %s.", query_name (op)); 3459 who->statusmsg (format ("You ready your %s.", query_name (op)));
3467 3460
3468 who->contr->ranged_ob = op; 3461 who->contr->ranged_ob = op;
3469 } 3462 }
3470 break; 3463 break;
3471 3464
3472 default: 3465 default:
3473 new_draw_info_format (NDI_UNIQUE, 0, who, "You apply %s.", query_name (op)); 3466 who->statusmsg (format ("You apply %s.", query_name (op)));
3474 } /* end of switch op->type */ 3467 }
3475 3468
3476 SET_FLAG (op, FLAG_APPLIED); 3469 SET_FLAG (op, FLAG_APPLIED);
3477 3470
3478 if (tmp) 3471 if (tmp)
3479 tmp = insert_ob_in_ob (tmp, who); 3472 who->insert (tmp);
3480 3473
3481 who->update_stats (); 3474 who->update_stats ();
3482 3475
3483 /* We exclude spell casting objects. The fire code will set the 3476 /* We exclude spell casting objects. The fire code will set the
3484 * been applied flag when they are used - until that point, 3477 * been applied flag when they are used - until that point,
… …
3488 SET_FLAG (op, FLAG_BEEN_APPLIED); 3481 SET_FLAG (op, FLAG_BEEN_APPLIED);
3489 3482
3490 if (QUERY_FLAG (op, FLAG_CURSED) || QUERY_FLAG (op, FLAG_DAMNED)) 3483 if (QUERY_FLAG (op, FLAG_CURSED) || QUERY_FLAG (op, FLAG_DAMNED))
3491 if (who->type == PLAYER) 3484 if (who->type == PLAYER)
3492 { 3485 {
3493 new_draw_info (NDI_UNIQUE, 0, who, 3486 who->failmsg (
3494 "Oops, it feels deadly cold! " 3487 "Oops, it feels deadly cold! "
3495 "H<Maybe it wasn't such a bright idea to apply this cursed/damned item.>"); 3488 "H<Maybe it wasn't such a bright idea to apply this cursed or damned item.>"
3489 );
3496 SET_FLAG (op, FLAG_KNOWN_CURSED); 3490 SET_FLAG (op, FLAG_KNOWN_CURSED);
3497 } 3491 }
3498 3492
3499 if (who->type == PLAYER) 3493 if (object *pl = op->visible_to ())
3500 {
3501 /* if multiple objects were applied, update both slots */
3502 if (tmp)
3503 esrv_send_item (who, tmp);
3504
3505 esrv_send_item (who, op); 3494 esrv_send_item (pl, op);
3506 }
3507 3495
3508 return 0; 3496 return 0;
3509} 3497}
3510 3498
3511int 3499int
… …
3525int 3513int
3526auto_apply (object *op) 3514auto_apply (object *op)
3527{ 3515{
3528 object *tmp = NULL, *tmp2; 3516 object *tmp = NULL, *tmp2;
3529 int i; 3517 int i;
3518
3519 CLEAR_FLAG (op, FLAG_AUTO_APPLY);
3530 3520
3531 switch (op->type) 3521 switch (op->type)
3532 { 3522 {
3533 case SHOP_FLOOR: 3523 case SHOP_FLOOR:
3534 if (!op->has_random_items ()) 3524 if (!op->has_random_items ())
… …
3536 3526
3537 do 3527 do
3538 { 3528 {
3539 i = 10; /* let's give it 10 tries */ 3529 i = 10; /* let's give it 10 tries */
3540 while ((tmp = generate_treasure (op->randomitems, 3530 while ((tmp = generate_treasure (op->randomitems,
3541 op->stats.exp ? (int) op->stats.exp : MAX (op->map->difficulty, 5))) == NULL && --i); 3531 op->stats.exp
3532 ? (int) op->stats.exp
3533 : max (op->map->difficulty, 5)))
3534 == NULL && --i);
3535
3542 if (tmp == NULL) 3536 if (tmp == NULL)
3543 return 0; 3537 return 0;
3538
3544 if (QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED)) 3539 if (QUERY_FLAG (tmp, FLAG_CURSED) || QUERY_FLAG (tmp, FLAG_DAMNED))
3545 { 3540 {
3546 tmp->destroy (); 3541 tmp->destroy ();
3547 tmp = NULL; 3542 tmp = NULL;
3548 } 3543 }
… …
3551 3546
3552 tmp->x = op->x; 3547 tmp->x = op->x;
3553 tmp->y = op->y; 3548 tmp->y = op->y;
3554 SET_FLAG (tmp, FLAG_UNPAID); 3549 SET_FLAG (tmp, FLAG_UNPAID);
3555 insert_ob_in_map (tmp, op->map, NULL, 0); 3550 insert_ob_in_map (tmp, op->map, NULL, 0);
3556 CLEAR_FLAG (op, FLAG_AUTO_APPLY);
3557 identify (tmp); 3551 identify (tmp);
3558 break; 3552 break;
3559 3553
3560 case TREASURE: 3554 case TREASURE:
3561 if (QUERY_FLAG (op, FLAG_IS_A_TEMPLATE)) 3555 if (QUERY_FLAG (op, FLAG_IS_A_TEMPLATE))
… …
3568 /* If we generated an object and put it in this object inventory, 3562 /* If we generated an object and put it in this object inventory,
3569 * move it to the parent object as the current object is about 3563 * move it to the parent object as the current object is about
3570 * to disappear. An example of this item is the random_* stuff 3564 * to disappear. An example of this item is the random_* stuff
3571 * that is put inside other objects. 3565 * that is put inside other objects.
3572 */ 3566 */
3573 for (tmp = op->inv; tmp; tmp = tmp2)
3574 {
3575 tmp2 = tmp->below;
3576 tmp->remove ();
3577
3578 if (op->env) 3567 if (op->env)
3579 insert_ob_in_ob (tmp, op->env); 3568 while (op->inv)
3580 else 3569 op->env->insert (op->inv);
3581 tmp->destroy ();
3582 }
3583 3570
3584 op->destroy (); 3571 op->destroy ();
3585 break; 3572 break;
3586 } 3573 }
3587 return tmp ? 1 : 0; 3574
3575 return !!tmp;
3588} 3576}
3589 3577
3590/** 3578/**
3591 * fix_auto_apply goes through the entire map every time a map 3579 * fix_auto_apply goes through the entire map every time a map
3592 * is loaded or swapped in and performs special actions for 3580 * is loaded or swapped in and performs special actions for
… …
3606 3594
3607 if (tmp->inv) 3595 if (tmp->inv)
3608 { 3596 {
3609 object *invtmp, *invnext; 3597 object *invtmp, *invnext;
3610 3598
3611 for (invtmp = tmp->inv; invtmp != NULL; invtmp = invnext) 3599 for (invtmp = tmp->inv; invtmp; invtmp = invnext)
3612 { 3600 {
3613 invnext = invtmp->below; 3601 invnext = invtmp->below;
3614 3602
3615 if (QUERY_FLAG (invtmp, FLAG_AUTO_APPLY)) 3603 if (QUERY_FLAG (invtmp, FLAG_AUTO_APPLY))
3616 auto_apply (invtmp); 3604 auto_apply (invtmp);
3617 else if (invtmp->type == TREASURE && invtmp->has_random_items ()) 3605 else if (invtmp->type == TREASURE && invtmp->has_random_items ())
3618 { 3606 {
3619 while ((invtmp->stats.hp--) > 0) 3607 while (invtmp->stats.hp-- > 0)
3620 create_treasure (invtmp->randomitems, invtmp, 0, difficulty, 0); 3608 create_treasure (invtmp->randomitems, invtmp, 0, difficulty, 0);
3621 3609
3622 invtmp->randomitems = NULL; 3610 invtmp->randomitems = NULL;
3623 } 3611 }
3624 else if (invtmp && invtmp->arch 3612 else if (invtmp && invtmp->arch
… …
3736 /* check for hp, sp change */ 3724 /* check for hp, sp change */
3737 if (food->stats.hp != 0) 3725 if (food->stats.hp != 0)
3738 { 3726 {
3739 if (QUERY_FLAG (food, FLAG_CURSED)) 3727 if (QUERY_FLAG (food, FLAG_CURSED))
3740 { 3728 {
3741 assign (who->contr->killer, food->name); 3729 who->contr->killer = food;
3742 hit_player (who, food->stats.hp, food, AT_POISON, 1); 3730 hit_player (who, food->stats.hp, food, AT_POISON, 1);
3743 new_draw_info (NDI_UNIQUE, 0, who, "Eck!...that was poisonous!"); 3731 who->failmsg ("Eck!...that was poisonous!");
3744 } 3732 }
3745 else 3733 else
3746 { 3734 {
3747 if (food->stats.hp > 0) 3735 if (food->stats.hp > 0)
3748 new_draw_info (NDI_UNIQUE, 0, who, "You begin to feel better."); 3736 who->statusmsg ("You begin to feel better.");
3749 else 3737 else
3750 new_draw_info (NDI_UNIQUE, 0, who, "Eck!...that was poisonous!"); 3738 who->failmsg ("Eck!...that was poisonous!");
3739
3751 who->stats.hp += food->stats.hp; 3740 who->stats.hp += food->stats.hp;
3752 } 3741 }
3753 } 3742 }
3743
3754 if (food->stats.sp != 0) 3744 if (food->stats.sp != 0)
3755 { 3745 {
3756 if (QUERY_FLAG (food, FLAG_CURSED)) 3746 if (QUERY_FLAG (food, FLAG_CURSED))
3757 { 3747 {
3758 new_draw_info (NDI_UNIQUE, 0, who, "You are drained of mana!"); 3748 who->failmsg ("You are drained of mana!");
3759 who->stats.sp -= food->stats.sp; 3749 who->stats.sp -= food->stats.sp;
3760 if (who->stats.sp < 0) 3750 if (who->stats.sp < 0)
3761 who->stats.sp = 0; 3751 who->stats.sp = 0;
3762 } 3752 }
3763 else 3753 else
3764 { 3754 {
3765 new_draw_info (NDI_UNIQUE, 0, who, "You feel a rush of magical energy!"); 3755 who->statusmsg ("You feel a rush of magical energy!");
3766 who->stats.sp += food->stats.sp; 3756 who->stats.sp += food->stats.sp;
3767 /* place limit on max sp from food? */ 3757 /* place limit on max sp from food? */
3768 } 3758 }
3769 } 3759 }
3760
3770 who->update_stats (); 3761 who->update_stats ();
3771} 3762}
3772 3763
3773/** 3764/**
3774 * Designed primarily to light torches/lanterns/etc. 3765 * Designed primarily to light torches/lanterns/etc.
… …
3784 item = find_marked_object (who); 3775 item = find_marked_object (who);
3785 if (item) 3776 if (item)
3786 { 3777 {
3787 if (lighter->last_eat && lighter->stats.food) 3778 if (lighter->last_eat && lighter->stats.food)
3788 { /* lighter gets used up */ 3779 { /* lighter gets used up */
3789 /* Split multiple lighters if they're being used up. Otherwise *
3790 * one charge from each would be used up. --DAMN */
3791 if (lighter->nrof > 1)
3792 {
3793 object *oneLighter = lighter->clone (); 3780 object *oneLighter = lighter->split ();
3794
3795 lighter->nrof -= 1;
3796 oneLighter->nrof = 1;
3797 oneLighter->stats.food--; 3781 oneLighter->stats.food--;
3798 esrv_send_item (who, lighter); 3782 who->insert (oneLighter);
3799 oneLighter = insert_ob_in_ob (oneLighter, who);
3800 esrv_send_item (who, oneLighter);
3801 }
3802 else
3803 lighter->stats.food--;
3804 } 3783 }
3805 else if (lighter->last_eat) 3784 else if (lighter->last_eat)
3785 {
3806 { /* no charges left in lighter */ 3786 /* no charges left in lighter */
3807 new_draw_info_format (NDI_UNIQUE, 0, who, "You attempt to light the %s with a used up %s.", &item->name, &lighter->name); 3787 who->failmsg (format ("You attempt to light the %s with a used up %s.", &item->name, &lighter->name));
3808 return; 3788 return;
3809 } 3789 }
3810 3790
3811 /* Perhaps we should split what we are trying to light on fire? 3791 /* Perhaps we should split what we are trying to light on fire?
3812 * I can't see many times when you would want to light multiple 3792 * I can't see many times when you would want to light multiple
… …
3818 3798
3819 save_throw_object (item, AT_FIRE, who); 3799 save_throw_object (item, AT_FIRE, who);
3820 3800
3821 if (item->destroyed ()) 3801 if (item->destroyed ())
3822 { 3802 {
3823 new_draw_info_format (NDI_UNIQUE, 0, who, "You light the %s with the %s.", &item->name, &lighter->name); 3803 who->statusmsg (format ("You light the %s with the %s.", &item->name, &lighter->name));
3824 /* Need to update the player so that the players glow radius 3804 /* Need to update the player so that the players glow radius
3825 * gets changed. 3805 * gets changed.
3826 */ 3806 */
3827 if (is_player_env) 3807 if (is_player_env)
3828 who->update_stats (); 3808 who->update_stats ();
3829 } 3809 }
3830 else 3810 else
3831 new_draw_info_format (NDI_UNIQUE, 0, who, "You attempt to light the %s with the %s and fail.", &item->name, &lighter->name); 3811 who->failmsg (format ("You attempt to light the %s with the %s and fail.", &item->name, &lighter->name));
3832 } 3812 }
3833 else /* nothing to light */ 3813 else
3834 new_draw_info (NDI_UNIQUE, 0, who, "You need to mark a lightable object."); 3814 who->failmsg ("You need to mark a lightable object.");
3835
3836} 3815}
3837 3816
3838/** 3817/**
3839 * op made some mistake with a scroll, this takes care of punishment. 3818 * op made some mistake with a scroll, this takes care of punishment.
3840 * scroll_failure()- hacked directly from spell_failure 3819 * scroll_failure()- hacked directly from spell_failure
… …
3847 3826
3848 if (failure <= -1 && failure > -15) 3827 if (failure <= -1 && failure > -15)
3849 { /* wonder */ 3828 { /* wonder */
3850 object *tmp; 3829 object *tmp;
3851 3830
3852 new_draw_info (NDI_UNIQUE, 0, op, "Your spell warps!"); 3831 op->failmsg ("Your spell warps!");
3853 tmp = get_archetype (SPELL_WONDER); 3832 tmp = get_archetype (SPELL_WONDER);
3854 cast_wonder (op, op, 0, tmp); 3833 cast_wonder (op, op, 0, tmp);
3855 tmp->destroy (); 3834 tmp->destroy ();
3856 } 3835 }
3857 else if (failure <= -15 && failure > -35) 3836 else if (failure <= -15 && failure > -35)
3858 { /* drain mana */ 3837 { /* drain mana */
3859 new_draw_info (NDI_UNIQUE, 0, op, "Your mana is drained!."); 3838 op->failmsg ("Your mana is drained!");
3860 op->stats.sp -= random_roll (0, power - 1, op, PREFER_LOW); 3839 op->stats.sp -= random_roll (0, power - 1, op, PREFER_LOW);
3861 if (op->stats.sp < 0) 3840 if (op->stats.sp < 0)
3862 op->stats.sp = 0; 3841 op->stats.sp = 0;
3863 } 3842 }
3864 else if (settings.spell_failure_effects == TRUE) 3843 else if (settings.spell_failure_effects == TRUE)
3865 { 3844 {
3866 if (failure <= -35 && failure > -60) 3845 if (failure <= -35 && failure > -60)
3867 { /* confusion */ 3846 { /* confusion */
3868 new_draw_info (NDI_UNIQUE, 0, op, "The magic recoils on you!"); 3847 op->failmsg ("The magic recoils on you!");
3869 confuse_player (op, op, power); 3848 confuse_player (op, op, power);
3870 } 3849 }
3871 else if (failure <= -60 && failure > -70) 3850 else if (failure <= -60 && failure > -70)
3872 { /* paralysis */ 3851 { /* paralysis */
3873 new_draw_info (NDI_UNIQUE, 0, op, "The magic recoils and paralyzes " "you!"); 3852 op->failmsg ("The magic recoils and paralyzes you!");
3874 paralyze_player (op, op, power); 3853 paralyze_player (op, op, power);
3875 } 3854 }
3876 else if (failure <= -70 && failure > -80) 3855 else if (failure <= -70 && failure > -80)
3877 { /* blind */ 3856 { /* blind */
3878 new_draw_info (NDI_UNIQUE, 0, op, "The magic recoils on you!"); 3857 op->failmsg ("The magic recoils on you!");
3879 blind_player (op, op, power); 3858 blind_player (op, op, power);
3880 } 3859 }
3881 else if (failure <= -80) 3860 else if (failure <= -80)
3882 { /* blast the immediate area */ 3861 { /* blast the immediate area */
3883 object *tmp = get_archetype (LOOSE_MANA); 3862 object *tmp = get_archetype (LOOSE_MANA);
3884 cast_magic_storm (op, tmp, power); 3863 cast_magic_storm (op, tmp, power);
3885 new_draw_info (NDI_UNIQUE, 0, op, "You unleash uncontrolled mana!"); 3864 op->failmsg ("You unleash uncontrolled mana!");
3886 tmp->destroy (); 3865 tmp->destroy ();
3887 } 3866 }
3888 } 3867 }
3889} 3868}
3890 3869
… …
3955 if (!strcmp (walk->name, "NOCLASSFACECHANGE")) 3934 if (!strcmp (walk->name, "NOCLASSFACECHANGE"))
3956 flag_change_face = 0; 3935 flag_change_face = 0;
3957 3936
3958 if (flag_change_face) 3937 if (flag_change_face)
3959 { 3938 {
3960 pl->animation_id = GET_ANIM_ID (change);
3961 pl->face = change->face; 3939 pl->face = change->face;
3962 3940 pl->animation_id = change->animation_id;
3963 if (QUERY_FLAG (change, FLAG_ANIMATE)) 3941 pl->flag [FLAG_ANIMATE] = change->flag [FLAG_ANIMATE];
3964 SET_FLAG (pl, FLAG_ANIMATE);
3965 else
3966 CLEAR_FLAG (pl, FLAG_ANIMATE);
3967 } 3942 }
3968 3943
3969 /* check the special case of can't use weapons */ 3944 /* check the special case of can't use weapons */
3970 /*if(QUERY_FLAG(change,FLAG_USE_WEAPON)) CLEAR_FLAG(pl,FLAG_USE_WEAPON); */ 3945 /*if(QUERY_FLAG(change,FLAG_USE_WEAPON)) CLEAR_FLAG(pl,FLAG_USE_WEAPON); */
3971 if (!strcmp (change->name, "monk")) 3946 if (!strcmp (change->name, "monk"))
… …
4002 3977
4003 marked = find_marked_object (pl); 3978 marked = find_marked_object (pl);
4004 3979
4005 if (!marked) 3980 if (!marked)
4006 { 3981 {
4007 new_draw_info_format (NDI_UNIQUE, 0, pl, "Use the %s with what item?", query_name (transformer)); 3982 pl->failmsg (format ("Use the %s with what item?", query_name (transformer)));
4008 return; 3983 return;
4009 } 3984 }
4010 3985
4011 if (!marked->slaying) 3986 if (!marked->slaying)
4012 { 3987 {
4013 new_draw_info_format (NDI_UNIQUE, 0, pl, "You can't use the %s with your %s!", query_name (transformer), query_name (marked)); 3988 pl->failmsg (format ("You can't use the %s with your %s!", query_name (transformer), query_name (marked)));
4014 return; 3989 return;
4015 } 3990 }
4016 3991
4017 /* check whether they are compatible or not */ 3992 /* check whether they are compatible or not */
4018 find = strstr (marked->slaying, transformer->arch->archname); 3993 find = strstr (marked->slaying, transformer->arch->archname);
4019 if (!find || (*(find + strlen (transformer->arch->archname)) != ':')) 3994 if (!find || (*(find + strlen (transformer->arch->archname)) != ':'))
4020 { 3995 {
4021 new_draw_info_format (NDI_UNIQUE, 0, pl, "You can't use the %s with your %s!", query_name (transformer), query_name (marked)); 3996 pl->failmsg (format ("You can't use the %s with your %s!", query_name (transformer), query_name (marked)));
4022 return; 3997 return;
4023 } 3998 }
4024 3999
4025 find += strlen (transformer->arch->archname) + 1; 4000 find += strlen (transformer->arch->archname) + 1;
4026 /* Item can be used, now find how many and what it yields */ 4001 /* Item can be used, now find how many and what it yields */
… …
4036 else 4011 else
4037 yield = 1; 4012 yield = 1;
4038 4013
4039 while (isdigit (*find)) 4014 while (isdigit (*find))
4040 find++; 4015 find++;
4016
4041 while (*find == ' ') 4017 while (*find == ' ')
4042 find++; 4018 find++;
4043 4019
4044 memset (got, 0, MAX_BUF); 4020 memset (got, 0, MAX_BUF);
4045 4021
… …
4056 4032
4057 /* Now create new item, remove used ones when required. */ 4033 /* Now create new item, remove used ones when required. */
4058 new_item = get_archetype (got); 4034 new_item = get_archetype (got);
4059 if (!new_item) 4035 if (!new_item)
4060 { 4036 {
4061 new_draw_info_format (NDI_UNIQUE, 0, pl, "This %s is strange, better to not use it.", query_base_name (marked, 0)); 4037 pl->failmsg (format ("This %s is strange, better to not use it.", query_base_name (marked, 0)));
4062 return; 4038 return;
4063 } 4039 }
4064 4040
4065 new_item->nrof = yield; 4041 new_item->nrof = yield;
4042
4066 new_draw_info_format (NDI_UNIQUE, 0, pl, "You %s the %s.", &transformer->slaying, query_base_name (marked, 0)); 4043 pl->statusmsg (format ("You %s the %s.", &transformer->slaying, query_base_name (marked, 0)));
4067 insert_ob_in_ob (new_item, pl); 4044
4068 esrv_send_inventory (pl, pl); 4045 pl->insert (new_item);
4069 /* Eat up one item */ 4046 /* Eat up one item */
4070 decrease_ob_nr (marked, 1); 4047 marked->decrease ();
4071 4048
4072 /* Eat one transformer if needed */ 4049 /* Eat one transformer if needed */
4073 if (transformer->stats.food) 4050 if (transformer->stats.food)
4074 if (--transformer->stats.food == 0) 4051 if (--transformer->stats.food == 0)
4075 decrease_ob_nr (transformer, 1); 4052 transformer->decrease ();
4076} 4053}
4054

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines