Patch for status command

Developer discussion of experimental fixes, changes, and improvements.

Moderators: Nexuiz Moderators, Moderators

Patch for status command

Postby terencehill » Sun Nov 23, 2008 11:12 pm

I've fixed status command. Now it's nicer to see, more readable and it displays the complete names (no more truncated tags).

The patch for this command is needed also to display more chars in names with lots of new color tags that I did. A name like ^xf00t^x0f0e^xf00r^x0f0ence ^4hill without this patch is shown with only 3 chars (a part the horrible truncated tag...): ter^xf0

Download link: http://sites.google.com/site/terencehill/Nexuiz/status.patch

And don't forget, this patch works only in your own server.

Image
terencehill
Alien
 
Posts: 176
Joined: Thu Jul 10, 2008 10:33 pm
Location: Italy

Postby divVerent » Tue Nov 25, 2008 7:01 am

I really am not sure whether this can be applied, or whether the output of the "status" command must stay the same to be compatible with other quake engines (some engines issue that command in the background, and parse its output).

However, there is a new command "cmd teamstatus" which shows a more detailed scoreboard than status. Maybe there could be a similar command that doesn't focus on the scores, but on ping and packetloss, and is also some "cmd". It could be aliased to "st" then...
1. Open Notepad
2. Paste: ÿþMSMSMS
3. Save
4. Open the file in Notepad again

You can vary the number of "MS", so you can clearly see it's MS which is causing it.
divVerent
Site admin and keyboard killer
 
Posts: 3809
Joined: Thu Mar 02, 2006 4:46 pm
Location: BRLOGENSHFEGLE

Postby Alien » Tue Nov 25, 2008 7:37 am

Doesn't dp have own networking protocol?
Alien
Forum addon
 
Posts: 1212
Joined: Tue Apr 22, 2008 7:12 am

Postby divVerent » Tue Nov 25, 2008 9:17 am

Yes, but:

1. it also parses "status" replies (for darkplaces_iplog.txt)

2. it does support acting as a Quake server, and people actually do that and connect with other Quake engines to DP. Your status patch currently breaks that.
1. Open Notepad
2. Paste: ÿþMSMSMS
3. Save
4. Open the file in Notepad again

You can vary the number of "MS", so you can clearly see it's MS which is causing it.
divVerent
Site admin and keyboard killer
 
Posts: 3809
Joined: Thu Mar 02, 2006 4:46 pm
Location: BRLOGENSHFEGLE

Postby Alien » Tue Nov 25, 2008 10:54 am

Maybe it would be possible to differentiate output based on the protocol:
old status if protocol == NQ or QW and new status if protocol == DP. Problems would arise if other engines support DP protocol too and use status to parse info. Same goes for qstat.
Alien
Forum addon
 
Posts: 1212
Joined: Tue Apr 22, 2008 7:12 am

Postby Asraniel » Tue Nov 25, 2008 11:01 am

probably just creating a new command will be way easier
Asraniel
Alien
 
Posts: 112
Joined: Tue Feb 28, 2006 9:15 pm

Postby terencehill » Tue Nov 25, 2008 4:59 pm

The problem in the current status command is that is actually unreadable.

You use it when u want to know or the ip of a player, or his number, to kick him or simply to send him a private message.

Therefore the command "cmd teamstatus" is unuseful for this goal.

But I've understood, this patch can't be applied, so I guess the things will stay as they are.
terencehill
Alien
 
Posts: 176
Joined: Thu Jul 10, 2008 10:33 pm
Location: Italy

Postby [-z-] » Tue Nov 25, 2008 5:04 pm

terencehill wrote:But I've understood, this patch can't be applied, so I guess the things will stay as they are.


But there's no reason another command can't be aliased, like 'superstatus'. The concern is just with breaking current systems.
[-z-]
Site Admin and Nexuiz Ninja
 
Posts: 1794
Joined: Mon Nov 13, 2006 12:20 am
Location: Florida

Postby terencehill » Wed Dec 10, 2008 7:49 pm

BY prompt of esteel, I've done a patch to the status command to displays in other 2 differents ways the informations on players playing in the current server, as the default way is very chaotic and unreadable. Now it's nicer to see, more readable and it displays the complete names (no more truncated tags).
So now the default command is still available, but if you put after status the number 1 or 2 you have 2 other different layouts.

* status 1 shows an extended layout with many infos,
* status 2 shows a semplified layout with the more importants infos.

Using these 2 new commands, the IPs of players are not stored in the file darkplaces_iplog.txt.

status 0 works in the same way as status.

Screenshot: http://sites.google.com/site/terencehill/Nexuiz/status_v2.jpg

Using parameters other than 0, 1 or 2, or using more parameters, it prints nothing.

The status command is server side so the patch can be tried only in your own server.

Download the patch here: http://sites.google.com/site/terencehill/Nexuiz/status_v2.patch.
terencehill
Alien
 
Posts: 176
Joined: Thu Jul 10, 2008 10:33 pm
Location: Italy

Postby Alien » Wed Dec 10, 2008 9:11 pm

Finally, no reason not to implement it. Congrats and thanks.
Alien
Forum addon
 
Posts: 1212
Joined: Tue Apr 22, 2008 7:12 am

Next

Return to Nexuiz - Development

Who is online

Users browsing this forum: No registered users and 1 guest

cron