SmaugMuds.org - Viewing Topic: Bug in format_obj_to

Bug in format_obj_to_char( )

< Newer Topic :: Older Topic >

Pages:<< prev 1 next >>

#1 Jan 12, 2017, 1:27 am

GatewaySysop
Conjurer

GroupMembers
Posts396
JoinedMar 7, 2005

Another one that I found recently. Not sure if someone was just really confused when they did this or what, but the more I looked, the less sense it made. :blink:

Original code is found in act_info.c, format_obj_to_char( )

   if( ch->Class == CLASS_PALADIN
       && ( IS_OBJ_STAT( obj, ITEM_ANTI_EVIL ) && !IS_OBJ_STAT( obj, ITEM_ANTI_NEUTRAL )
            && !IS_OBJ_STAT( obj, ITEM_ANTI_GOOD ) ) )
      mudstrlcat( buf, "(Flaming Red) ", MAX_STRING_LENGTH );
   if( ch->Class == CLASS_PALADIN
       && ( !IS_OBJ_STAT( obj, ITEM_ANTI_EVIL ) && IS_OBJ_STAT( obj, ITEM_ANTI_NEUTRAL )
            && !IS_OBJ_STAT( obj, ITEM_ANTI_GOOD ) ) )
      mudstrlcat( buf, "(Flaming Grey) ", MAX_STRING_LENGTH );
   if( ch->Class == CLASS_PALADIN
       && ( !IS_OBJ_STAT( obj, ITEM_ANTI_EVIL ) && !IS_OBJ_STAT( obj, ITEM_ANTI_NEUTRAL )
            && IS_OBJ_STAT( obj, ITEM_ANTI_GOOD ) ) )
      mudstrlcat( buf, "(Flaming White) ", MAX_STRING_LENGTH );

   if( ch->Class == CLASS_PALADIN
       && ( IS_OBJ_STAT( obj, ITEM_ANTI_EVIL ) && IS_OBJ_STAT( obj, ITEM_ANTI_NEUTRAL )
            && !IS_OBJ_STAT( obj, ITEM_ANTI_GOOD ) ) )
      mudstrlcat( buf, "(Smouldering Red-Grey) ", MAX_STRING_LENGTH );
   if( ch->Class == CLASS_PALADIN
       && ( IS_OBJ_STAT( obj, ITEM_ANTI_EVIL ) && !IS_OBJ_STAT( obj, ITEM_ANTI_NEUTRAL )
            && IS_OBJ_STAT( obj, ITEM_ANTI_GOOD ) ) )
      mudstrlcat( buf, "(Smouldering Red-White) ", MAX_STRING_LENGTH );
   if( ch->Class == CLASS_PALADIN
       && ( !IS_OBJ_STAT( obj, ITEM_ANTI_EVIL ) && IS_OBJ_STAT( obj, ITEM_ANTI_NEUTRAL )
            && IS_OBJ_STAT( obj, ITEM_ANTI_GOOD ) ) )
      mudstrlcat( buf, "(Smouldering Grey-White) ", MAX_STRING_LENGTH );

Am I crazy, or should that not be written like this instead? :huh:

    if ( ch->Class==CLASS_PALADIN && ( IS_OBJ_STAT(obj, ITEM_ANTI_EVIL) && !IS_OBJ_STAT(obj, ITEM_ANTI_NEUTRAL) && !IS_OBJ_STAT(obj, ITEM_ANTI_GOOD))   )
          mudstrlcat( buf, "(Smouldering Grey-White) ", MAX_STRING_LENGTH  );
    if ( ch->Class==CLASS_PALADIN && ( !IS_OBJ_STAT(obj, ITEM_ANTI_EVIL) && IS_OBJ_STAT(obj, ITEM_ANTI_NEUTRAL) && !IS_OBJ_STAT(obj, ITEM_ANTI_GOOD))   )
          mudstrlcat( buf, "(Smouldering Red-White) ", MAX_STRING_LENGTH  );
    if ( ch->Class==CLASS_PALADIN && (!IS_OBJ_STAT(obj, ITEM_ANTI_EVIL) && !IS_OBJ_STAT(obj, ITEM_ANTI_NEUTRAL) && IS_OBJ_STAT(obj, ITEM_ANTI_GOOD))   )
          mudstrlcat( buf, "(Smouldering Red-Grey) ", MAX_STRING_LENGTH  );
    if ( ch->Class==CLASS_PALADIN && ( IS_OBJ_STAT(obj, ITEM_ANTI_EVIL) && IS_OBJ_STAT(obj, ITEM_ANTI_NEUTRAL) && !IS_OBJ_STAT(obj, ITEM_ANTI_GOOD))   )
          mudstrlcat( buf, "(Burning White) ", MAX_STRING_LENGTH  );
    if ( ch->Class==CLASS_PALADIN && ( IS_OBJ_STAT(obj, ITEM_ANTI_EVIL) && !IS_OBJ_STAT(obj, ITEM_ANTI_NEUTRAL) && IS_OBJ_STAT(obj, ITEM_ANTI_GOOD))   )
          mudstrlcat( buf, "(Burning Grey) ", MAX_STRING_LENGTH  );
    if ( ch->Class==CLASS_PALADIN && ( !IS_OBJ_STAT(obj, ITEM_ANTI_EVIL) && IS_OBJ_STAT(obj, ITEM_ANTI_NEUTRAL) && IS_OBJ_STAT(obj, ITEM_ANTI_GOOD))   )
          mudstrlcat( buf, "(Burning Red) ", MAX_STRING_LENGTH  );

#2 Jan 12, 2017, 3:02 pm

Sadiq
Fledgling

GroupMembers
Posts24
JoinedJan 7, 2010

I would make (and have made) this change. The rest of the SMAUG code traditionally uses the Red=Evil, Grey=Neutral, White=Good format, so this makes sense. Implemented. :wink:

#3 Jan 12, 2017, 4:38 pm

GatewaySysop
Conjurer

GroupMembers
Posts396
JoinedMar 7, 2005

Sadiq said:

I would make (and have made) this change. The rest of the SMAUG code traditionally uses the Red=Evil, Grey=Neutral, White=Good format, so this makes sense. Implemented. :wink:

I knew I wasn't crazy! Thanks for the confirmation. :cyclops:

#4 Jan 13, 2017, 10:01 am

Remcon
Geomancer

GroupAdministrators
Posts1,874
JoinedJul 26, 2005

thanks while i had changed the whole look of that stuff because of how confusing it looked i hadn't paid the stuff it showed any attention, modified it