ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/deliantra/server/common/loader.l
(Generate patch)

Comparing deliantra/server/common/loader.l (file contents):
Revision 1.1 by root, Fri Feb 3 07:11:36 2006 UTC vs.
Revision 1.8 by root, Sat Aug 26 08:44:04 2006 UTC

1%{ 1%{
2/* 2/*
3 * static char *rcsid_object_c = 3 * static char *rcsid_object_c =
4 * "$Id: loader.l,v 1.1 2006/02/03 07:11:36 root Exp $"; 4 * "$Id: loader.l,v 1.8 2006/08/26 08:44:04 root Exp $";
5 */ 5 */
6 6
7/* 7/*
8 CrossFire, A Multiplayer game for X-windows 8 CrossFire, A Multiplayer game for X-windows
9 9
… …
40#define YY_DECL int lex_load(object *op, int map_flags) 40#define YY_DECL int lex_load(object *op, int map_flags)
41 41
42static char *yval(); 42static char *yval();
43 43
44static int lex_error; 44static int lex_error;
45static char msgbuf[HUGE_BUF]; 45static char msgbuf[65536];
46static char lorebuf[HUGE_BUF]; 46static char lorebuf[65536];
47
48/* Maps the MOVE_* values to names */
49static const char *const move_name[] = {"walk", "fly_low", "fly_high", "swim", "boat",
50 NULL};
47 51
48/* This table is only necessary to convert objects that existed before the 52/* This table is only necessary to convert objects that existed before the
49 * spell object conversion to the new object. It was not practical 53 * spell object conversion to the new object. It was not practical
50 * to go through every mapping looking for every potion, rod, wand, etc 54 * to go through every mapping looking for every potion, rod, wand, etc
51 * that had a sp set and update to the new value. So this maps the 55 * that had a sp set and update to the new value. So this maps the
… …
53 * If you are adding a new spell, you should not modify this - you 57 * If you are adding a new spell, you should not modify this - you
54 * new spell won't have been used, and thus won't have any legacy object. 58 * new spell won't have been used, and thus won't have any legacy object.
55 * NULL entries in this table are valid - to denote objects that should 59 * NULL entries in this table are valid - to denote objects that should
56 * not be updated for whatever reason. 60 * not be updated for whatever reason.
57 */ 61 */
58char *spell_mapping[] = { 62const char *spell_mapping[] = {
59"spell_magic_bullet", /* 0 */ 63"spell_magic_bullet", /* 0 */
60"spell_small_fireball", /* 1 */ 64"spell_small_fireball", /* 1 */
61"spell_medium_fireball", /* 2 */ 65"spell_medium_fireball", /* 2 */
62"spell_large_fireball", /* 3 */ 66"spell_large_fireball", /* 3 */
63"spell_burning_hands", /* 4 */ 67"spell_burning_hands", /* 4 */
… …
321 } 325 }
322 } 326 }
323 LOG(llevError,"set_body_info called with bogus params: %s\n", params); 327 LOG(llevError,"set_body_info called with bogus params: %s\n", params);
324} 328}
325 329
330// return a suitable strign describign an objetc in enough detail to find it
331// used only in check_loaded_object: TODO remove static, move it elsewhere and
332// use it in more log messages.
333static char *op_debug_info_ (object *op, char *info)
334{
335 char info2[256 * 3];
336 char *p = info;
337
338 p += snprintf (p, 256, "%d=\"%s%s%s\"",
339 op->count,
340 op->name ? op->name : "(anon)",
341 op->title ? " " : "",
342 op->title ? op->title : "");
343
344 if (op->env)
345 p += snprintf (p, 256, "(in %s)", op_debug_info_ (op->env, info2));
346
347 if (op->map)
348 p += snprintf (p, 256, "(on %s@%d+%d)", op->map->path, op->x, op->y);
349
350 return info;
351}
352
353static char *op_debug_info (object *op)
354{
355 static char info[256 * 3];
356
357 return op_debug_info_ (op, info);
358}
326 359
327/* This function checks the object after it has been loaded (when we 360/* This function checks the object after it has been loaded (when we
328 * get the 'end' in the input stream). This function can be used to 361 * get the 'end' in the input stream). This function can be used to
329 * deal with legacy objects where fields may have changed. It can also be used 362 * deal with legacy objects where fields may have changed. It can also be used
330 * to check for objects to make sure there are no common errors. 363 * to check for objects to make sure there are no common errors.
… …
359 * really just to catch any errors - program will still run, but 392 * really just to catch any errors - program will still run, but
360 * not in the ideal fashion. 393 * not in the ideal fashion.
361 */ 394 */
362 if ((op->type == WEAPON || op->type==BOW) && arch_init) { 395 if ((op->type == WEAPON || op->type==BOW) && arch_init) {
363 if (!op->skill) { 396 if (!op->skill) {
364 LOG(llevError,"Weapon %s lacks a skill.\n", op->name); 397 LOG(llevError,"Weapon %s lacks a skill.\n", op_debug_info (op));
365 } else if ((!strcmp(op->skill,"one handed weapons") && op->body_info[1] != -1) || 398 } else if ((!strcmp(op->skill,"one handed weapons") && op->body_info[1] != -1) ||
366 (!strcmp(op->skill,"two handed weapons") && op->body_info[1] != -2)) { 399 (!strcmp(op->skill,"two handed weapons") && op->body_info[1] != -2)) {
367 LOG(llevError,"weapon %s arm usage does not match skill: %d, %s\n", 400 LOG(llevError,"weapon %s arm usage does not match skill: %d, %s\n",
368 op->name, op->body_info[1], op->skill); 401 op_debug_info (op), op->body_info[1], op->skill);
369 } 402 }
370 } 403 }
371 404
372 /* We changed last_heal to gen_sp_armour, which is what it 405 /* We changed last_heal to gen_sp_armour, which is what it
373 * really does for many objects. Need to catch any in maps 406 * really does for many objects. Need to catch any in maps
… …
378 (op->type == SHIELD) || (op->type == RING) || 411 (op->type == SHIELD) || (op->type == RING) ||
379 (op->type == BOOTS) || (op->type == GLOVES) || 412 (op->type == BOOTS) || (op->type == GLOVES) ||
380 (op->type == AMULET ) || (op->type == GIRDLE) || 413 (op->type == AMULET ) || (op->type == GIRDLE) ||
381 (op->type == BRACERS ) || (op->type == CLOAK)) { 414 (op->type == BRACERS ) || (op->type == CLOAK)) {
382 if (op->last_heal) { 415 if (op->last_heal) {
383 LOG(llevDebug,"Object %s still has last_heal set, not gen_sp_armour\n", 416 LOG(llevDebug,"Object %s still has last_heal set, not gen_sp_armour\n", op_debug_info (op));
384 op->name?op->name:"NULL");
385 op->gen_sp_armour = op->last_heal; 417 op->gen_sp_armour = op->last_heal;
386 op->last_heal = 0; 418 op->last_heal = 0;
387 } 419 }
388 if (editor) ip =0; 420 if (editor) ip =0;
389 else ip = calc_item_power(op, 0); 421 else ip = calc_item_power(op, 0);
390 /* Legacy objects from before item power was in the game */ 422 /* Legacy objects from before item power was in the game */
391 if (!op->item_power && ip) { 423 if (!op->item_power && ip) {
392 if (ip > 3) { 424 if (ip > 3) {
393 LOG(llevDebug,"Object %s had no item power, using %d\n", 425 LOG(llevDebug,"Object %s had no item power, using %d\n", op_debug_info (op), ip);
394 op->name?op->name:"NULL", ip);
395 } 426 }
396 op->item_power = ip; 427 op->item_power = ip;
397 } 428 }
398 /* Check for possibly bogus values. Has to meet both these criteria - 429 /* Check for possibly bogus values. Has to meet both these criteria -
399 * something that has item_power 1 is probably just fine if our calculated 430 * something that has item_power 1 is probably just fine if our calculated
… …
401 * similarly, it item_power is 0, the first check will always pass, 432 * similarly, it item_power is 0, the first check will always pass,
402 * but not the second one. 433 * but not the second one.
403 */ 434 */
404 if (ip > 2 *op->item_power && ip > (op->item_power + 3)) { 435 if (ip > 2 *op->item_power && ip > (op->item_power + 3)) {
405 LOG(llevDebug,"Object %s seems to have too low item power? %d > %d\n", 436 LOG(llevDebug,"Object %s seems to have too low item power? %d > %d\n",
406 op->name?op->name:"NULL", ip, op->item_power); 437 op_debug_info (op), ip, op->item_power);
407 } 438 }
408 439
409 } 440 }
410 /* Old spellcasting object - need to load in the appropiate object */ 441 /* Old spellcasting object - need to load in the appropiate object */
411 if ((op->type == ROD || op->type == WAND || op->type == SCROLL || op->type == HORN 442 if ((op->type == ROD || op->type == WAND || op->type == SCROLL || op->type == HORN
… …
434 } 465 }
435 466
436 if (QUERY_FLAG(op, FLAG_MONSTER)) { 467 if (QUERY_FLAG(op, FLAG_MONSTER)) {
437 if (op->stats.hp > op->stats.maxhp) 468 if (op->stats.hp > op->stats.maxhp)
438 LOG(llevDebug,"Monster %s has hp set higher than maxhp (%d>%d)\n", 469 LOG(llevDebug,"Monster %s has hp set higher than maxhp (%d>%d)\n",
439 op->name, 470 op_debug_info (op),
440 op->stats.hp, op->stats.maxhp); 471 op->stats.hp, op->stats.maxhp);
441 472
442 /* The archs just need to be updated for this */ 473 /* The archs just need to be updated for this */
443 if (op->move_type ==0) op->move_type = MOVE_WALK; 474 if (op->move_type ==0) op->move_type = MOVE_WALK;
444 } 475 }
… …
535 *end='\0'; 566 *end='\0';
536 } 567 }
537 set_ob_key_value(op, key, value, TRUE); 568 set_ob_key_value(op, key, value, TRUE);
538} 569}
539 570
571static void set_move(MoveType *mt, char *params) {
572 char *str;
573 int i, negate;
574
575 if (isdigit(*params)) {
576 *mt = atoi(params);
577 } else {
578 *mt=0;
579 for (str=strtok(params, " "); str; str=strtok(NULL, " ")) {
580 negate=0;
581 if (!strcasecmp(str, "all"))
582 *mt |= MOVE_ALL;
583 else {
584 if (*str=='-') {
585 negate = 1;
586 str++;
587 }
588 for (i=0; move_name[i] != NULL; i++) {
589 if (!strcasecmp(move_name[i], str)) {
590 if (negate) {
591 *mt &= ~(1<<i);
592 } else {
593 *mt |= (1<<i);
594 }
595 break;
596 }
597 }
598 if (move_name[i] == NULL) {
599 /* fly is a special case - covers both fly_low and
600 * fly_high - since it doesn't match to a specific
601 * single bit, have to special case it.
602 */
603 if (!strcasecmp(str,"flying")) {
604 if (negate) {
605 *mt &= ~MOVE_FLYING;
606 } else {
607 *mt |= MOVE_FLYING;
608 }
609 } else {
610 LOG(llevDebug, "common/loader.l: set_move - unknown move string '%s'\n", str);
611 }
612 }
613 } /* Else not all move types */
614 } /* for strtok */
615 } /* Else not a numeric value */
616}
540 617
541%} 618%}
542 619
543 620
544 621
… …
566 643
567%} 644%}
568 645
569^msg{WS}$ { BEGIN( MESSAGE ); msgbuf[0]='\0'; } 646^msg{WS}$ { BEGIN( MESSAGE ); msgbuf[0]='\0'; }
570<MESSAGE>^endmsg{WS}$ { BEGIN( INITIAL ); 647<MESSAGE>^endmsg{WS}$ { BEGIN( INITIAL );
571 op->msg=add_string(msgbuf);
572 /* Just print a warning so we can be reasonably safe 648 /* Just print a warning so we can be reasonably safe
573 * about not overflowing the buffer. 649 * about not overflowing the buffer.
574 */ 650 */
575 if (strlen(op->msg) > (HUGE_BUF/2)) 651 if (strlen(msgbuf) >= HUGE_BUF)
652 {
576 LOG(llevDebug, "\n\tWarning message length > %d (max allowed=%d): %d\n>%.80s<\n", 653 LOG(llevDebug, "\n\tError message length >= %d: %d\n>%.80s<\n",
577 HUGE_BUF/2, HUGE_BUF, strlen(op->msg),op->msg); 654 HUGE_BUF, strlen(op->msg),op->msg);
655 op->msg = add_string ("ERROR, please report: string too long, winged.\n");
656 }
657 else
658 op->msg = add_string (msgbuf);
578 } 659 }
579<MESSAGE>.* {strcat(msgbuf, yytext); strcat(msgbuf,"\n"); } 660<MESSAGE>.* {strcat(msgbuf, yytext); strcat(msgbuf,"\n"); }
580 661
581^lore{WS}$ { BEGIN( LORE ); lorebuf[0]='\0'; } 662^lore{WS}$ { BEGIN( LORE ); lorebuf[0]='\0'; }
582<LORE>^endlore{WS}$ { BEGIN( INITIAL ); 663<LORE>^endlore{WS}$ { BEGIN( INITIAL );
… …
612^name_pl{S} { char *yv=yval(); 693^name_pl{S} { char *yv=yval();
613 694
614 if (*yv=='\0') LOG(llevError,"Name without val\n"); 695 if (*yv=='\0') LOG(llevError,"Name without val\n");
615 else FREE_AND_COPY(op->name_pl, yv); 696 else FREE_AND_COPY(op->name_pl, yv);
616 } 697 }
698^attach{S} { char *yv=yval();
699 if (*yv)
700 op->attach = add_string (yv);
701 }
617^skill{S} FREE_AND_COPY(op->skill,yval()); 702^skill{S} FREE_AND_COPY(op->skill,yval());
618^custom_name{S} { char *yv=yval(); 703^custom_name{S} { char *yv=yval();
619 704
620 if (*yv=='\0') LOG(llevError,"Custom name without val\n"); 705 if (*yv=='\0') LOG(llevError,"Custom name without val\n");
621 else FREE_AND_COPY(op->custom_name, yv); 706 else FREE_AND_COPY(op->custom_name, yv);
… …
769^fly_off{S} { if (IVAL) op->move_off |= MOVE_FLY_LOW; else op->move_off &= ~MOVE_FLY_LOW; } 854^fly_off{S} { if (IVAL) op->move_off |= MOVE_FLY_LOW; else op->move_off &= ~MOVE_FLY_LOW; }
770^flying{S} { if (IVAL) op->move_type |= MOVE_FLY_LOW; else op->move_type &= ~MOVE_FLY_LOW; } 855^flying{S} { if (IVAL) op->move_type |= MOVE_FLY_LOW; else op->move_type &= ~MOVE_FLY_LOW; }
771 856
772%{ /* These are the new values */ 857%{ /* These are the new values */
773%} 858%}
774^move_block{S} op->move_block = IVAL; 859^move_block{S} set_move(&op->move_block, yval());
775^move_type{S} op->move_type = IVAL; 860^move_allow{S} set_move(&op->move_allow, yval());
776^move_on{S} op->move_on = IVAL; 861^move_type{S} set_move(&op->move_type, yval());
777^move_off{S} op->move_off = IVAL; 862^move_on{S} set_move(&op->move_on, yval());
778^move_slow{S} op->move_slow = IVAL; 863^move_off{S} set_move(&op->move_off, yval());
864^move_slow{S} set_move(&op->move_slow, yval());
779^move_slow_penalty{S} op->move_slow_penalty = FVAL; 865^move_slow_penalty{S} op->move_slow_penalty = FVAL;
780 866
781 867
782^monster{S} SET_OR_CLEAR_FLAG(op, FLAG_MONSTER, IVAL); 868^monster{S} SET_OR_CLEAR_FLAG(op, FLAG_MONSTER, IVAL);
783^neutral{S} SET_OR_CLEAR_FLAG(op, FLAG_NEUTRAL, IVAL); 869^neutral{S} SET_OR_CLEAR_FLAG(op, FLAG_NEUTRAL, IVAL);
784^no_attack{S} SET_OR_CLEAR_FLAG(op, FLAG_NO_ATTACK, IVAL); 870^no_attack{S} SET_OR_CLEAR_FLAG(op, FLAG_NO_ATTACK, IVAL);
785^no_damage{S} SET_OR_CLEAR_FLAG(op, FLAG_NO_DAMAGE, IVAL); 871^no_damage{S} SET_OR_CLEAR_FLAG(op, FLAG_NO_DAMAGE, IVAL);
786^friendly{S} { if (IVAL) { 872^friendly{S} { if (IVAL) {
787 SET_FLAG(op, FLAG_FRIENDLY); 873 SET_FLAG(op, FLAG_FRIENDLY);
788 if (op->type != PLAYER) { 874 if (op->type != PLAYER)
789 LOG(llevDebug," Adding friendly object %s.\n",op->name);
790 add_friendly_object(op); 875 add_friendly_object (op);
791 }
792 } 876 }
793 else CLEAR_FLAG(op, FLAG_FRIENDLY); 877 else CLEAR_FLAG(op, FLAG_FRIENDLY);
794 } 878 }
795^generator{S} SET_OR_CLEAR_FLAG(op, FLAG_GENERATOR, IVAL); 879^generator{S} SET_OR_CLEAR_FLAG(op, FLAG_GENERATOR, IVAL);
796^use_content_on_gen{S} SET_OR_CLEAR_FLAG (op,FLAG_CONTENT_ON_GEN, IVAL); 880^use_content_on_gen{S} SET_OR_CLEAR_FLAG (op,FLAG_CONTENT_ON_GEN, IVAL);
… …
904^resist_disease{S} SET_RESIST(op, ATNR_DISEASE, IVAL); 988^resist_disease{S} SET_RESIST(op, ATNR_DISEASE, IVAL);
905 989
906 /* Old style resistances */ 990 /* Old style resistances */
907^immune{S} set_protection(op, IVAL, RESIST_IMMUNE); 991^immune{S} set_protection(op, IVAL, RESIST_IMMUNE);
908^protected{S} set_protection(op, IVAL, RESIST_PROT); 992^protected{S} set_protection(op, IVAL, RESIST_PROT);
909^vulnerable{S} set_protection(op, IVAL, RESIST_VULN); 993^vulnerable{S} set_protection(op, IVAL, (uint16) RESIST_VULN);
910 994
911 /* old values - keep them around for now, but they should be removed at some point */ 995 /* old values - keep them around for now, but they should be removed at some point */
912^has_ready_rod{S} SET_OR_CLEAR_FLAG(op, FLAG_READY_RANGE, IVAL); 996^has_ready_rod{S} SET_OR_CLEAR_FLAG(op, FLAG_READY_RANGE, IVAL);
913^has_ready_horn{S} SET_OR_CLEAR_FLAG(op, FLAG_READY_RANGE, IVAL); 997^has_ready_horn{S} SET_OR_CLEAR_FLAG(op, FLAG_READY_RANGE, IVAL);
914^has_ready_wand{S} SET_OR_CLEAR_FLAG(op, FLAG_READY_RANGE, IVAL); 998^has_ready_wand{S} SET_OR_CLEAR_FLAG(op, FLAG_READY_RANGE, IVAL);
… …
953^can_impale{S} { /* That these are for the new combat code */ } 1037^can_impale{S} { /* That these are for the new combat code */ }
954^can_cut{S} { /* just ignore for now */ } 1038^can_cut{S} { /* just ignore for now */ }
955^can_dam_armour{S} { } 1039^can_dam_armour{S} { }
956^weapontype{S} op->weapontype = IVAL; 1040^weapontype{S} op->weapontype = IVAL;
957^tooltype{S} op->tooltype = IVAL; 1041^tooltype{S} op->tooltype = IVAL;
958^casting_time{S} op->casting_time = FVAL; 1042^casting_time{S} op->casting_time = (sint16) FVAL;
959^elevation{S} op->elevation = IVAL; 1043^elevation{S} op->elevation = IVAL;
960^smoothlevel{S} op->smoothlevel = IVAL; 1044^smoothlevel{S} op->smoothlevel = IVAL;
961^client_type{S} op->client_type = IVAL; 1045^client_type{S} op->client_type = IVAL;
962^body_{A} set_body_info(op, yytext); 1046^body_{A} set_body_info(op, yytext);
963^duration{S} op->duration = IVAL; 1047^duration{S} op->duration = IVAL;
… …
1386/* This is a list of pointers that correspond to the FLAG_.. values. 1470/* This is a list of pointers that correspond to the FLAG_.. values.
1387 * This is a simple 1:1 mapping - if FLAG_FRIENDLY is 15, then 1471 * This is a simple 1:1 mapping - if FLAG_FRIENDLY is 15, then
1388 * the 15'th element of this array should match that name. 1472 * the 15'th element of this array should match that name.
1389 * If an entry is NULL, that is a flag not to loaded/saved. 1473 * If an entry is NULL, that is a flag not to loaded/saved.
1390 */ 1474 */
1391static char *flag_names[NUM_FLAGS+1] = { 1475static const char *const flag_names[NUM_FLAGS+1] = {
1392"alive", "wiz", NULL, NULL, "was_wiz", "applied", "unpaid", 1476"alive", "wiz", NULL, NULL, "was_wiz", "applied", "unpaid",
1393"can_use_shield", "no_pick", NULL /* walk_on*/, NULL /* no_pass */, /* 10 */ 1477"can_use_shield", "no_pick", NULL /* walk_on*/, NULL /* no_pass */, /* 10 */
1394"is_animated", NULL /* slow_move */, 1478"is_animated", NULL /* slow_move */,
1395NULL /* flying */, "monster", "friendly", "generator", 1479NULL /* flying */, "monster", "friendly", "generator",
1396"is_thrown", "auto_apply", "treasure", "player sold", /* 20 */ 1480"is_thrown", "auto_apply", "treasure", "player sold", /* 20 */
… …
1420"activate_on_release", "is_water", "use_content_on_gen", NULL, "is_buildable", /* 110 */ 1504"activate_on_release", "is_water", "use_content_on_gen", NULL, "is_buildable", /* 110 */
1421NULL 1505NULL
1422}; 1506};
1423 1507
1424 1508
1425void save_double(char *buf,char *name,double v)
1426{
1427 char tbuf[200];
1428
1429 sprintf(tbuf,"%s %f\n",name,v);
1430 strcat(buf,tbuf);
1431}
1432
1433/* 1509/*
1434 * Initialises the array of variable-names. Needed before any 1510 * Initialises the array of variable-names. Needed before any
1435 * objects can be loaded. Called by init_library(). 1511 * objects can be loaded. Called by init_library().
1436 */ 1512 */
1437 1513
1438void init_vars() { 1514void init_vars() {
1439} 1515}
1440 1516
1441/*For get_ob_diff speed reason*/ 1517/*For get_ob_diff speed reason*/
1442typedef struct { 1518typedef struct {
1443 char* name; 1519 const char *name;
1444 int length; 1520 int length;
1445}genericname; 1521}genericname;
1446static genericname evtnames[13]= 1522static genericname evtnames[13]=
1447{ 1523{
1448 {"event_none ",11}, 1524 {"event_none ",11},
… …
1492 {"event_trigger_options ",22}, 1568 {"event_trigger_options ",22},
1493 {"event_close_options ",20}, 1569 {"event_close_options ",20},
1494 {"event_timer_options ",20} 1570 {"event_timer_options ",20}
1495}; 1571};
1496 1572
1573/* This returns a string of the integer movement type */
1574static char* get_string_move_type(MoveType mt)
1575{
1576 static char retbuf[MAX_BUF], retbuf_all[MAX_BUF];
1577 int i, all_count=0, count;
1578
1579 strcpy(retbuf,"");
1580 strcpy(retbuf_all," all");
1581
1582 /* Quick check, and probably fairly common */
1583 if (mt == MOVE_ALL) return retbuf_all+1;
1584 if (mt == 0) {
1585 strcpy(retbuf,"0");
1586 return retbuf;
1587 }
1588
1589 /* We basically slide the bits down. Why look at MOVE_ALL?
1590 * because we may want to return a string like 'all -swim',
1591 * and if we just looked at mt, we couldn't get that.
1592 */
1593 for (i=MOVE_ALL, count=0; i!=0; i >>= 1, count++) {
1594 if (mt & (1<<count)) {
1595 strcat(retbuf, " ");
1596 strcat(retbuf, move_name[count]);
1597 } else {
1598 strcat(retbuf_all, " -");
1599 strcat(retbuf_all, move_name[count]);
1600 all_count++;
1601 }
1602 }
1603 /* Basically, if there is a single negation, return it, eg
1604 * 'all -swim'. But more than that, just return the
1605 * enumerated values. It doesn't make sense to return
1606 * 'all -walk -fly_low' - it is shorter to return 'fly_high swim'
1607 */
1608 if (all_count <=1) return retbuf_all+1;
1609 else return retbuf+1;
1610}
1611
1612
1497/* 1613/*
1498 * Returns a pointer to a static string which contains all variables 1614 * Returns a pointer to a static string which contains all variables
1499 * which are different in the two given objects. op is the what object 1615 * which are different in the two given objects. op is the what object
1500 * the different values will be taken from. This function is 1616 * the different values will be taken from. This function is
1501 * typically used to dump objects (op2=empty object), or to save objects 1617 * typically used to dump objects (op2=empty object), or to save objects
… …
1535 * NEVER touch buf between PREPARE_FASTCAT(buf) and 1651 * NEVER touch buf between PREPARE_FASTCAT(buf) and
1536 * FINISH_FASTCAT(buf) 1652 * FINISH_FASTCAT(buf)
1537 */ 1653 */
1538 static int already_run = 0; 1654 static int already_run = 0;
1539 static int flag_lens[NUM_FLAGS]; 1655 static int flag_lens[NUM_FLAGS];
1540 static char buf2[HUGE_BUF]; 1656 char buf2[4096]; // / was HUFE_BUF, which was hugely incorrect, as is this value, but much less so
1541 static char buf[HUGE_BUF]; 1657 static char buf[128*1024]; // < the assumption is that every object always fits. fixing this bug, however
1542 char* fastbuf; 1658 char* fastbuf; // \ requires a large rewrite of the code, so its left to the next total rewrite.
1543 int tmp; 1659 int tmp;
1544 int i; 1660 int i;
1545#if 0
1546/*Memory polluting code. Should help detect problems, very slow*/
1547 for (i=0;i<HUGE_BUF;i++){
1548 buf[i]='a'+(unsigned short)(i%25);
1549 }
1550#endif
1551 event *etmp; 1661 event *etmp;
1552 event *etmp2; 1662 event *etmp2;
1553 key_value * my_field; 1663 key_value * my_field;
1554 key_value * arch_field; 1664 key_value * arch_field;
1555 1665
… …
1684 if(op->stats.maxgrace!=op2->stats.maxgrace) 1794 if(op->stats.maxgrace!=op2->stats.maxgrace)
1685 FAST_SAVE_LONG(fastbuf,"maxgrace ",op->stats.maxgrace,9); 1795 FAST_SAVE_LONG(fastbuf,"maxgrace ",op->stats.maxgrace,9);
1686 1796
1687 if(op->stats.exp!=op2->stats.exp) { 1797 if(op->stats.exp!=op2->stats.exp) {
1688#ifndef WIN32 1798#ifndef WIN32
1689 sprintf(buf2,"%lld\n", op->stats.exp); 1799 sprintf(buf2,"%lld", op->stats.exp);
1690#else 1800#else
1691 sprintf(buf2,"%I64d\n", op->stats.exp); 1801 sprintf(buf2,"%I64d", op->stats.exp);
1692#endif 1802#endif
1693 ADD_STRINGLINE_ENTRY(fastbuf,"exp ", buf2, 4); 1803 ADD_STRINGLINE_ENTRY(fastbuf,"exp ", buf2, 4);
1694 } 1804 }
1695 1805
1696 if(op->perm_exp!=op2->perm_exp) { 1806 if(op->perm_exp!=op2->perm_exp) {
1697#ifndef WIN32 1807#ifndef WIN32
1698 sprintf(buf2,"%lld\n", op->perm_exp); 1808 sprintf(buf2,"%lld", op->perm_exp);
1699#else 1809#else
1700 sprintf(buf2,"%I64d\n", op->perm_exp); 1810 sprintf(buf2,"%I64d", op->perm_exp);
1701#endif 1811#endif
1702 ADD_STRINGLINE_ENTRY(fastbuf,"perm_exp ", buf2, 9); 1812 ADD_STRINGLINE_ENTRY(fastbuf,"perm_exp ", buf2, 9);
1703 } 1813 }
1704 1814
1705 if(op->expmul!=op2->expmul) 1815 if(op->expmul!=op2->expmul)
… …
1843 1953
1844 if (op->gen_sp_armour != op2->gen_sp_armour) { 1954 if (op->gen_sp_armour != op2->gen_sp_armour) {
1845 FAST_SAVE_LONG(fastbuf,"gen_sp_armour ",op->gen_sp_armour,14); 1955 FAST_SAVE_LONG(fastbuf,"gen_sp_armour ",op->gen_sp_armour,14);
1846 } 1956 }
1847 1957
1958 /* I've kept the old int move type saving code commented out.
1959 * In an ideal world, we'd know if we want to do a quick
1960 * save (say to a temp map, where we don't care about strings),
1961 * or a slower save/dm dump, where printing out strings is handy.
1962 */
1848 if (op->move_type != op2->move_type) { 1963 if (op->move_type != op2->move_type) {
1849 FAST_SAVE_LONG(fastbuf,"move_type ",op->move_type,10); 1964 /*FAST_SAVE_LONG(fastbuf,"move_type ",op->move_type,10)*/
1965 ADD_STRINGLINE_ENTRY(fastbuf,"move_type ",
1966 get_string_move_type(op->move_type),
1967 10);
1850 } 1968 }
1851
1852 if (op->move_block != op2->move_block) { 1969 if (op->move_block != op2->move_block) {
1853 FAST_SAVE_LONG(fastbuf,"move_block ",op->move_block,11); 1970 /*FAST_SAVE_LONG(fastbuf,"move_block ",op->move_block,11)*/
1971 ADD_STRINGLINE_ENTRY(fastbuf,"move_block ",
1972 get_string_move_type(op->move_block),
1973 11);
1854 } 1974 }
1855 1975 if (op->move_allow != op2->move_allow) {
1976 /*FAST_SAVE_LONG(fastbuf,"move_allow ",op->move_allow,11);*/
1977 ADD_STRINGLINE_ENTRY(fastbuf,"move_allow ",
1978 get_string_move_type(op->move_allow),
1979 11);
1980 }
1856 if (op->move_on != op2->move_on) { 1981 if (op->move_on != op2->move_on) {
1857 FAST_SAVE_LONG(fastbuf,"move_on ",op->move_on,8); 1982 /*FAST_SAVE_LONG(fastbuf,"move_on ",op->move_on,8);*/
1983 ADD_STRINGLINE_ENTRY(fastbuf,"move_on ",
1984 get_string_move_type(op->move_on),
1985 8);
1858 } 1986 }
1859
1860 if (op->move_off != op2->move_off) { 1987 if (op->move_off != op2->move_off) {
1861 FAST_SAVE_LONG(fastbuf,"move_off ",op->move_off,9); 1988 /*FAST_SAVE_LONG(fastbuf,"move_off ",op->move_off,9);*/
1989 ADD_STRINGLINE_ENTRY(fastbuf,"move_off ",
1990 get_string_move_type(op->move_off),
1991 9);
1862 } 1992 }
1863 if (op->move_slow != op2->move_slow) { 1993 if (op->move_slow != op2->move_slow) {
1864 FAST_SAVE_LONG(fastbuf,"move_slow ",op->move_slow,10); 1994 /*FAST_SAVE_LONG(fastbuf,"move_slow ",op->move_slow,10);*/
1995 ADD_STRINGLINE_ENTRY(fastbuf,"move_slow ",
1996 get_string_move_type(op->move_slow),
1997 10);
1865 } 1998 }
1866 1999
1867 if (op->move_slow_penalty != op2->move_slow_penalty) { 2000 if (op->move_slow_penalty != op2->move_slow_penalty) {
1868 FAST_SAVE_LONG(fastbuf,"move_slow_penalty ",op->move_slow_penalty,18); 2001 FAST_SAVE_LONG(fastbuf,"move_slow_penalty ",(long) op->move_slow_penalty,18);
1869 } 2002 }
1870 2003
1871 if (!COMPARE_FLAGS(op,op2)) { 2004 if (!COMPARE_FLAGS(op,op2)) {
1872 for (tmp=0; tmp <= NUM_FLAGS; tmp++) { 2005 for (tmp=0; tmp <= NUM_FLAGS; tmp++) {
1873 if (flag_names[tmp] && (QUERY_FLAG(op, tmp) != QUERY_FLAG(op2, tmp))) { 2006 if (flag_names[tmp] && (QUERY_FLAG(op, tmp) != QUERY_FLAG(op2, tmp))) {

Diff Legend

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