[Home] [Downloads] [Search] [Help/forum]

Gammon Software Solutions forum

See www.mushclient.com/spam for dealing with forum spam. Please read the MUSHclient FAQ!

[Folder]  Entire forum
-> [Folder]  SMAUG
. -> [Folder]  SMAUG coding
. . -> [Subject]  issues with pager
Home  |  Users  |  Search  |  FAQ
Username:
Register forum user name
Password:
Forgotten password?
(New message)
Subject: issues with pager
Name:
Your forum user name.
Register forum user name
Password:
Your forum password.
Forgotten password?
Message:
Message to be posted (in English, please).
Forum codes:
Check this if your message uses 'forum codes' or templates (auto-detected for new posts).
Forum codes Templates

Save this message ...

Subject review (reverse sequence)

Posted by David Haley   USA  (3,881 posts)  [Biography] bio   Moderator
Date Sun 16 Jul 2006 09:45 PM (UTC)  [ quote  ]
Message
Hey, no problem. :) This is the kind of annoyance you see in SMAUG a lot. Things, that really shouldn't, can have far-reaching consequences and seriously muck things up in ways that are hard to foresee. When I replaced the SMAUG network code with my own I had a fair number of headaches, a lot relating to the pager. But, it's good that you fixed it!
David Haley aka Ksilyan
Head Programmer,
Legends of the Darkstone

http://david.the-haleys.org
[Go to top] top
Posted by Tzaro   USA  (41 posts)  [Biography] bio
Date Sun 16 Jul 2006 10:22 AM (UTC)  [ quote  ]
Message
yep, I found the problem. The issue wasn't with anything changed in color.c at all. The issue was someone had changed nearly all of our calls from send_to_char to send_to_pager. Apparently, there had been an issue with send_to_char that was causing jarbled data to be returned to the character when mass-amounts of data would be sent.

The problem with send_to_char was with flush_buffer. It was putting a 'pause' in the data sent to the char for every 512 bytes of data sent.... but the player could choose to have more than that sent to them. I changed it to reflect how much data the player wanted to have sent and changed all the send_to_pager calls back to send_to_char and everything was resolved.

Ksilyan, if you ever happen to see me strolling by, feel free to thwap me for wasting your time. I'll be sure to check the function calls first instead of just assuming they are how they're supposed to be from now on.
Implementer of Lost Prophecy,
Tzaro
[Go to top] top
Posted by David Haley   USA  (3,881 posts)  [Biography] bio   Moderator
Date Thu 13 Jul 2006 02:47 PM (UTC)  [ quote  ]
Message
Hmm... I can't see offhand why that would be happening from the code you're showing. Can you maybe show what you changed, like a before/after of the comm functions?

What seems to be happening is that the MUD outputs the pager's contents before the other buffer (the normal, main one). Normally, when the pager is on, everything sent to the normal buffer (using send_to_char) is supposed to be redirected to the pager's buffer. You could check your send_to_char_color function and make sure that's what's happening.
David Haley aka Ksilyan
Head Programmer,
Legends of the Darkstone

http://david.the-haleys.org
[Go to top] top
Posted by Tzaro   USA  (41 posts)  [Biography] bio
Date Thu 13 Jul 2006 01:50 PM (UTC)  [ quote  ]

Amended on Thu 13 Jul 2006 01:52 PM (UTC) by Tzaro

Message
This is what's displayed when I exam a guard with pager "on".

<used as light>     a torch
<worn on finger>    (Magical) (Glowing) a ring of the city guard
<worn on body>      a crested breastplate
<worn on head>      an iron helm
<worn on legs>      mail leggings
<worn on feet>      black boots
<worn on hands>     iron gauntlets
<worn on arms>      chain sleeves
<wielded>           a longsword
You see nothing special about him.
The guard is in perfect health.
the guard is using:

Mobile #21037 'guard man' is a level 16 human warrior.

You peek at his inventory:
     Nothing.

-----------------------------------------------------------

This is pager "off":

You see nothing special about him.
The guard is in perfect health.

the guard is using:
<used as light>     a torch
<worn on finger>    (Magical) (Glowing) a ring of the city guard
<worn on body>      a crested breastplate
<worn on head>      an iron helm
<worn on legs>      mail leggings
<worn on feet>      black boots
<worn on hands>     iron gauntlets
<worn on arms>      chain sleeves
<wielded>           a longsword

Mobile #21037 'guard man' is a level 16 human warrior.

You peek at his inventory:
     Nothing.

Sorry I didn't explain this better the first time though :X

Tz
Implementer of Lost Prophecy,
Tzaro
[Go to top] top
Posted by David Haley   USA  (3,881 posts)  [Biography] bio   Moderator
Date Wed 12 Jul 2006 03:25 PM (UTC)  [ quote  ]
Message
Just to make sure I understand the problem: are you saying that the text is arriving out of order, of sorts, with each chunk (sent to pager or directly to char) arriving separately?
David Haley aka Ksilyan
Head Programmer,
Legends of the Darkstone

http://david.the-haleys.org
[Go to top] top
Posted by Tzaro   USA  (41 posts)  [Biography] bio
Date Wed 12 Jul 2006 10:34 AM (UTC)  [ quote  ]
Message
Incase it brings up question... my colorcode function reads:

int colorcode( const char *col, char *code, CHAR_DATA * ch, bool override )

-Tz
Implementer of Lost Prophecy,
Tzaro
[Go to top] top
Posted by Tzaro   USA  (41 posts)  [Biography] bio
Date Wed 12 Jul 2006 10:32 AM (UTC)  [ quote  ]
Message
Has anyone ever run across something like this:

if PCFLAG_PAGERON returns true, anything that uses "send_to_pager()" is displayed first, followed by everything that uses "send_to_char()".

if PCFLAG_PAGERON returns false, anything that uses "send_to_char()" is displayed first, followed by everything that uses "send_to_pager()".

I'm baffled. The only thing 'unique' about my version of color.c is that I condensed send_to_pager_color and colorize together. below is a copy of what the code looks like in send_to_pager_color()... least the part of interest (I think).

      if( IS_NPC( ch ) || !IS_SET( ch->pcdata->flags, PCFLAG_PAGERON ) )
      {
         send_to_char_color( txt, d->character );
         return;
      }

      while( ( colstr = strpbrk( prevstr, "&{}" ) ) != NULL )
      {
         if( colstr > prevstr )
            write_to_pager( ch->desc, prevstr, ( colstr - prevstr ) );

         ln = colorcode( colstr, colbuf, ch, FALSE );
         if( ln < 0 )
         {
            prevstr = colstr + 1;
            break;
         }
         else if( ln > 0 )
            write_to_pager( ch->desc, colbuf, ln );
         prevstr = colstr + 2;
      }

      if( *prevstr )
         write_to_pager( ch->desc, prevstr, 0 );
   }
   return;
}

Any clues?

-Tz
Implementer of Lost Prophecy,
Tzaro
[Go to top] top

The dates and times for posts above are shown in Universal Co-ordinated Time (UTC).

To show them in your local time you can join the forum, and then set the 'time correction' field in your profile to the number of hours difference between your location and UTC time.

1,512 views.

[Reply to this subject]  Reply to this subject   [New subject]  Start a new subject   [Refresh] Refresh page

Go to topic:           Search the forum

[Go to top] top

[Home]

Written by Nick Gammon - 5K

Comments to: Gammon Software support
[RH click to get RSS URL] Forum RSS feed ( http://www.gammon.com.au/rss/forum.xml )

[Best viewed with any browser - 2K]    [Internet Contents Rating Association (ICRA) - 2K]    [Web site powered by FutureQuest.Net]