Login
User Name:

Password:



Register
Forgot your password?
Vote for Us!
 Couple bugs
Today, 5:42 pm
By Remcon
Bug in disarm( )
Nov 12, 2017, 6:54 pm
By GatewaySysop
Bug in will_fall( )
Oct 23, 2017, 1:35 am
By GatewaySysop
Bug in do_zap( ), do_brandish( )
Oct 18, 2017, 1:52 pm
By GatewaySysop
Bug in get_exp_worth( )
Oct 10, 2017, 1:26 am
By GatewaySysop
LOP 1.45
Author: Remcon
Submitted by: Remcon
LOP Heroes Edition
Author: Vladaar
Submitted by: Vladaar
Heroes sound extras
Author: Vladaar
Submitted by: Vladaar
6Dragons 4.3
Author: Vladaar
Submitted by: Vladaar
Memwatch
Author: Johan Lindh
Submitted by: Vladaar
Users Online
CommonCrawl, Yahoo!, Sogou, Yandex

Members: 0
Guests: 16
Stats
Files
Topics
Posts
Members
Newest Member
477
3,705
19,232
608
LAntorcha
Today's Birthdays
There are no member birthdays today.
Related Links
» SmaugMuds.org » Codebases » SWR FUSS » move_char bug with positions
Forum Rules | Mark all | Recent Posts

move_char bug with positions
< Newer Topic :: Older Topic >

Pages:<< prev 1, 2 next >>
Post is unread #21 Aug 2, 2010, 3:21 pm
Go to the top of the page
Go to the bottom of the page

Kayle
Off the Edge of the Map
GroupAdministrators
Posts1,195
JoinedMar 21, 2006

Conner said:

Sorry, Kayle, I'm pretty much in the same boat over here. If I put it in, I've still got to try to test it somehow by my lonesome too. My mud's not exactly been crowded to capacity since my move to the country that mandated switching to this damn laggy satellite feed. :(

Yeah, neither of mine are open, and with work, I lack the time I used to have to thoroughly test things before posting them. I do have some active staff members though, I probably could have gotten them to test it. But Meh, that's hindsight for you.


Sharmair said:

Stuffs.

Thanks Sharmair.

It felt wrong to me as well to toss all those checks into move_char when the actual problem lies with auto-exits themselves not the actual movement of the character. I hadn't stopped to think about the fall through from the check being where I had it though, so that was another good catch on your part. I guess that's good enough confirmation for me that using a check_pos() call on the auto-exits is enough to fix this issue. So I'll see about getting an actual fix posted soon.
       
Post is unread #22 Aug 2, 2010, 6:01 pm
Go to the top of the page
Go to the bottom of the page

Conner
Sorcerer
GroupMembers
Posts870
JoinedMay 8, 2005

Aint hindsight wonderous? ;)

So, does this address things like being pulled through an exit by room settings (See also help PULLTYPES) as well as flee, being dragged, etc now?
       
Post is unread #23 Aug 2, 2010, 6:08 pm
Go to the top of the page
Go to the bottom of the page

Kayle
Off the Edge of the Map
GroupAdministrators
Posts1,195
JoinedMar 21, 2006

My/Sharmair's fix only affects auto-exits.

Applying the original fix will disrupt flee. I dunno about pulltypes, I would assume it would disrupt those as well though.

That's why I was uneasy about applying the fix to move_char, too many things use move_char that fixing it directly in there will break things other than fixing just the issue, if you apply a targeted fix, for instance, a check_pos() call in the interpret function in the section for auto-exits, you fix the issue, without breaking any other things.
       
Post is unread #24 Aug 2, 2010, 6:30 pm
Go to the top of the page
Go to the bottom of the page

Samson
Black Hand
GroupAdministrators
Posts3,639
JoinedJan 1, 2002

Second (or 3rd now?) Kayle's logic. move_char() is a pretty big catch-all function for an awful lot of things. Fleeing being one, pulltypes another, and I'm sure there's probably other stuff that hasn't been noticed yet. Basically it would disrupt anything where the assumption is the move will happen regardless.

So fixing the problem in the auto-exit code looks like the better option.
       
Post is unread #25 Aug 3, 2010, 10:37 am
Go to the top of the page
Go to the bottom of the page

Conner
Sorcerer
GroupMembers
Posts870
JoinedMay 8, 2005

So, does that mean we're adopting one of these proposed fixes as official then?
       
Post is unread #26 Aug 3, 2010, 5:38 pm
Go to the top of the page
Go to the bottom of the page

Kayle
Off the Edge of the Map
GroupAdministrators
Posts1,195
JoinedMar 21, 2006

Yeah, I'll get it posted and released hopefully tomorrow after work.
       
Post is unread #27 Aug 4, 2010, 9:39 pm
Go to the top of the page
Go to the bottom of the page

Conner
Sorcerer
GroupMembers
Posts870
JoinedMay 8, 2005

Sounds good. :)
       
Pages:<< prev 1, 2 next >>