Login
User Name:

Password:



Register
Forgot your password?
Vote for Us!
Couple bugs
Dec 12, 2017, 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, Yandex, Yahoo!, Google

Members: 0
Guests: 13
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 » SmaugFUSS » Bug: time_update does not che...
Forum Rules | Mark all | Recent Posts

Bug: time_update does not check sector type
< Newer Topic :: Older Topic >

Pages:<< prev 1 next >>
Post is unread #1 Feb 18, 2012, 11:53 pm
Go to the top of the page
Go to the bottom of the page

GatewaySysop
Conjurer
GroupMembers
Posts367
JoinedMar 7, 2005

So was play testing and caught messages indoors / underwater and elsewhere about the time of day updates and started wondering if I broke something. Turns out this is a bug in the stock code, which despite the new weather system, is still retained to my knowledge.

In update.c, in function time_update, you'll find this:

if( d->connected == CON_PLAYING && IS_OUTSIDE( d->character ) && IS_AWAKE( d->character ) )


That should probably be something like this, which is how the original weather_update( ) used to work, so I think the inconsistency was just an oversight:

if( d->connected == CON_PLAYING && IS_OUTSIDE( d->character ) 
&& !NO_WEATHER_SECT( d->character->in_room->sector_type )
&& IS_AWAKE( d->character ) )


That should take care of it.
       
Post is unread #2 Feb 19, 2012, 6:14 am
Go to the top of the page
Go to the bottom of the page

Aurin
Magician
GroupMembers
Posts189
JoinedSep 5, 2010

Thanks! Was wondering where to make this update, though it was low on my bug list for fixing. heh
       
Post is unread #3 Feb 19, 2012, 1:03 pm
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,858
JoinedJul 26, 2005

and yet another nice catch :)
       
Pages:<< prev 1 next >>