Login
User Name:

Password:



Register
Forgot your password?
Vote for Us!
tintin++ ogg sound player script for linux
Author: Robert Smith
Submitted by: Vladaar
6Dragons ogg Soundpack
Author: Vladaar
Submitted by: Vladaar
6Dragons 4.4
Author: Vladaar
Submitted by: Vladaar
LoP 1.46
Author: Remcon
Submitted by: Remcon
LOP 1.45
Author: Remcon
Submitted by: Remcon
Users Online
CommonCrawl, Yandex, Google, DotBot, Sogou

Members: 0
Guests: 16
Stats
Files
Topics
Posts
Members
Newest Member
481
3,739
19,386
621
KellieBusb
Today's Birthdays
There are no member birthdays today.
Related Links
» SmaugMuds.org » Codebases » SmaugFUSS » Bug in disarm( )
Forum Rules | Mark all | Recent Posts

Bug in disarm( )
< Newer Topic :: Older Topic >

Pages:<< prev 1 next >>
Post is unread #1 Jan 12, 2017, 1:15 am   Last edited Jan 12, 2017, 1:17 am by GatewaySysop
Go to the top of the page
Go to the bottom of the page

GatewaySysop
Conjurer
GroupMembers
Posts392
JoinedMar 7, 2005

Yeah so another thing I stumbled on while doing some play testing. Wanted to create a character with a bladed stump for a weapon, so as to allow stabbing / backstabbing type damage and the like, obviously not wanting it removable. So I set the item_noremove flag but, to my surprise, the mobile could still be disarmed (no pun intended) in combat. :facepalm:

Seems to me that if you can't remove the item from yourself, you probably shouldn't be able to be disarmed of it either. :huh:

A little peek reveals the problem, that there is no accounting for the item_noremove object flag, in the disarm( ) function.


In skills.c, you'll find disarm( ), likely just above do_disarm( ), and in particular this piece:

    if ( check_grip( ch, victim ) )
    {
    	learn_from_failure( ch, gsn_disarm );
    	return;
    }


Probably ought to make a slight modification like so:
    if ( check_grip( ch, victim ) || IS_OBJ_STAT( obj, ITEM_NOREMOVE ) )
    {
    	learn_from_failure( ch, gsn_disarm );
    	return;
    }


And that is that. :imp:


       
Post is unread #2 Jan 12, 2017, 2:47 pm
Go to the top of the page
Go to the bottom of the page

Sadiq
Fledgling
GroupMembers
Posts24
JoinedJan 7, 2010

Nice catch! :smile:
       
Post is unread #3 Jan 13, 2017, 9:45 am
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,874
JoinedJul 26, 2005

i remember as a player we did that on noremove weapons so we could remove them and drop them lol
       
Pages:<< prev 1 next >>