[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]  Format with FUSS
Home  |  Users  |  Search  |  FAQ
Username:
Register forum user name
Password:
Forgotten password?
(New message)
Subject: Format with FUSS
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)

Pages: 1 2  

Posted by Gatewaysysop2   USA  (142 posts)  [Biography] bio
Date Wed 30 Nov 2005 04:40 AM (UTC)  [ quote  ]
Message
Just as a follow-up, what Samson suggested worked perfectly.

Kudos for the tip. You the man, as always. ;)

"The world of men is dreaming, it has gone mad in its sleep, and a snake is strangling it, but it can't wake up." -D.H. Lawrence
[Go to top] top
Posted by Gatewaysysop2   USA  (142 posts)  [Biography] bio
Date Tue 29 Nov 2005 04:52 AM (UTC)  [ quote  ]
Message
will give it a shot, thanks for the suggestion. :)

"The world of men is dreaming, it has gone mad in its sleep, and a snake is strangling it, but it can't wake up." -D.H. Lawrence
[Go to top] top
Posted by Samson   USA  (683 posts)  [Biography] bio
Date Mon 28 Nov 2005 02:38 AM (UTC)  [ quote  ]
Message 

   max_buf_lines = 24;

   if( ch->substate == SUB_MPROG_EDIT || ch->substate == SUB_HELP_EDIT )
      max_buf_lines = 48;


In stock code, that's the variable that causes all this: max_buf_lines. Notice they have 2 different values for it. I'm not entirely sure why Sadiq had that variable array setup in his code. I don't think it really needs to be there.

So instead of:

char temp_buf[MAX_STRING_LENGTH + max_buf_lines];


Perhaps just:

char temp_buf[MAX_STRING_LENGTH];


Cause in the grand scheme of things, all he's really done is let another 48 chars go into temp_buf which doesn't seem very useful.

This line will need changing too:

strncpy( temp_buf + p, edit->line[x], MAX_STRING_LENGTH + max_buf_lines - p );
SmaugMuds.org: http://www.smaugmuds.org - The Smaug MUDs Community Center

"The past was erased, the erasure was forgotten, the lie became truth." -- George Orwell, 1984
[Go to top] top
Posted by Gatewaysysop2   USA  (142 posts)  [Biography] bio
Date Sun 27 Nov 2005 11:50 PM (UTC)  [ quote  ]
Message
Has anyone come up with a work around on this particular snippet? Looks like a handy feature to have, but since my code otherwise gets passed cleanly by the -pedantic flag, I'd rather not remove it from the makefile just for the sake of having a format option in the line editor. Would be neat to have otherwise though. :-/
"The world of men is dreaming, it has gone mad in its sleep, and a snake is strangling it, but it can't wake up." -D.H. Lawrence
[Go to top] top
Posted by Samson   USA  (683 posts)  [Biography] bio
Date Sun 27 Nov 2005 04:05 AM (UTC)  [ quote  ]
Message
The error doesn't come up when compiling C++ but it is a legitimate issue in C since C does not technically allow variable sized arrays. What amazes me is that it takes using the -pedantic switch just to catch this.
SmaugMuds.org: http://www.smaugmuds.org - The Smaug MUDs Community Center

"The past was erased, the erasure was forgotten, the lie became truth." -- George Orwell, 1984
[Go to top] top
Posted by Gatewaysysop2   USA  (142 posts)  [Biography] bio
Date Sun 27 Nov 2005 01:08 AM (UTC)  [ quote  ]
Message
Anyone else notice that the -pedantic flag doesn't like this snippet? It complains about variable sized arrays if memory serves. Has anyone come up with a work around for this or do they just ignore the error or not use -pedantic?

"The world of men is dreaming, it has gone mad in its sleep, and a snake is strangling it, but it can't wake up." -D.H. Lawrence
[Go to top] top
Posted by Mopop   (115 posts)  [Biography] bio
Date Fri 25 Nov 2005 11:16 PM (UTC)  [ quote  ]
Message
Gosh, How silly of me. I didnt know how it worked *blush* Thanks Nick and everyone else for your help!!! *crosses format off list*
[Go to top] top
Posted by Nick Gammon   Australia  (18,770 posts)  [Biography] bio   Forum Administrator
Date Fri 25 Nov 2005 11:12 PM (UTC)  [ quote  ]
Message
A line full of S isn't a good test, there are no word breaks to format at. It worked for me, see this:


> 
/l
------------------
 1> You stand inside the northern gate of Darkhaven. 
 2> To the south lies the
 3> intersection of Vertic Avenue and Justice Road. 
------------------
> 
/f
Reformating...
Reformating done.
> 
/l
------------------
 1> You stand inside the northern gate of Darkhaven.  To the south lies the
 2> intersection of Vertic Avenue and Justice Road. 
------------------
>


Before the formatting I had 3 lines, the middle one was too short. After formatting it looked better.
- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top
Posted by Mopop   (115 posts)  [Biography] bio
Date Fri 25 Nov 2005 10:06 PM (UTC)  [ quote  ]
Message
Nice, It compiled clean and I tested it out. It doesnt even format

Begin entering your text now (/? = help /s = save /c = clear /l = list)
-----------------------------------------------------------------------
>
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
(Long line trimmed)
> >
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
(Long line trimmed)
> >
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
(Long line trimmed)
> >
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
(Long line trimmed)
> >
/f
Reformating...
Reformating done.
>
/s
Done.

<32700hp 102m 110mv> <#7>
l
Floating in a void
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
SSS
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
SSS
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
SSS
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
SSS
Exits: none.

Would anyone happen to know why? I hate having all these format problems -.- I just want to get it off my to do list lol.
[Go to top] top
Posted by Nick Gammon   Australia  (18,770 posts)  [Biography] bio   Forum Administrator
Date Fri 25 Nov 2005 09:01 PM (UTC)  [ quote  ]

Amended on Fri 25 Nov 2005 09:02 PM (UTC) by Nick Gammon

Message
The variable "x" was already used earlier up. Rename the snippet's x to x1 to work around this. That is, the major part of it could be:


      if ( !str_cmp( cmd+1, "f" ) )
	{
	    char   temp_buf[MAX_STRING_LENGTH+max_buf_lines];
	    int    x1, ep, old_p, end_mark; 
	    int    p = 0;
	    
          pager_printf( ch, "Reformating...\n\r");

	    for ( x1 = 0; x1 < edit->numlines; x1++ )
	    {
	       strcpy ( temp_buf+p , edit->line[x1] );
	       p += strlen( edit->line[x1] );
	       temp_buf[p] = ' ';
	       p++;
	    }
	    
	    temp_buf[p] = '\0';
	    end_mark = p;
	    p = 75;
	    old_p =0;
	    edit->on_line =0;
	    edit->numlines =0;
	    
	    while ( old_p < end_mark )
	    {
	        while ( temp_buf[p] != ' ' && p > old_p )
	          p--;
	          
	        if ( p == old_p )
	          p +=75;
	        
	        if ( p > end_mark )
	          p = end_mark;
	        
	        ep =0;
	        for ( x1= old_p ; x1 < p ; x1++ )  
	        {
	           edit->line[edit->on_line][ep] = temp_buf[x1];
                   ep++;
                }
                edit->line[edit->on_line][ep] = '\0';

	        edit->on_line++;
	        edit->numlines++;
	        
	        old_p = p+1 ;
	        p += 75;
	        
	    }
          pager_printf( ch, "Reformating done.\n\r> ");
	    return;
	}

- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top
Posted by Mopop   (115 posts)  [Biography] bio
Date Fri 25 Nov 2005 08:06 PM (UTC)  [ quote  ]
Message
cc1: warnings being treated as errors
build.c: In function `edit_buffer':
build.c:5265: warning: declaration of `x' shadows a previous local
build.c:5150: warning: shadowed declaration is here

I added everything here are the bugs, I tried making some sense of it and failed. If you are gonna explain it to me, explain it in idiot terms as ive only been coding for bout 12 days =)
[Go to top] top
Posted by Samson   USA  (683 posts)  [Biography] bio
Date Fri 25 Nov 2005 04:25 PM (UTC)  [ quote  ]
Message
You may find this one a bit less problematic - all it does is patch the existing editor to include a /f option.

http://www.afkmud.com/scripts/download.php?file=Sadiq/Format.txt

Do let me know if there are any problems with it not behaving properly, it's been awhile since this one was last looked at since it's one of Sadiq's older ones.
SmaugMuds.org: http://www.smaugmuds.org - The Smaug MUDs Community Center

"The past was erased, the erasure was forgotten, the lie became truth." -- George Orwell, 1984
[Go to top] top
Posted by Mopop   (115 posts)  [Biography] bio
Date Wed 23 Nov 2005 10:30 PM (UTC)  [ quote  ]
Message
I scraped it, it made me too pissed. I just need a good /f command for SmaugFUSS I cant really edit all the stuff because im not sure of the differences of 1.4 smaug and FUSS :(. i wish I just knew how to hard code it myself =P
[Go to top] top
Posted by Nick Gammon   Australia  (18,770 posts)  [Biography] bio   Forum Administrator
Date Wed 23 Nov 2005 07:46 PM (UTC)  [ quote  ]
Message
"shadows" and "redeclares" sounds like things are either in the wrong place, or put there twice.
- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top
Posted by Conner   USA  (381 posts)  [Biography] bio
Date Wed 23 Nov 2005 12:21 AM (UTC)  [ quote  ]
Message
We established already that the strange distcc messages are something about Wolfpaw's clustered server auto-back-up at compile system in place...
-=Conner=-
--
Come test your mettle in the Land of Legends at telnet://tcdbbs.zapto.org:4000
or, for a little family oriented medieval fun, come join us at The Castle's Dungeon BBS at telnet://tcdbbs.zapto.org
or, if you just want information about either, check our web page at http://tcdbbs.zapto.org
[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.

5,577 views.

This is page 1, subject is 2 pages long: 1 2  [Next page]

[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]