Page 1 of 1

[FIXED] /Worlds populated

Posted: December 27th, 2011, 8:12 pm
by Jonty800
I noticed that /worlds populated lists all worlds that are loaded, but surely this will mean that if the main world is empty, it will still be listed? I noticed this and changed it to: if (world.Players.Count > 0)

I also noticed quite a while back that in /help mark it says "/Mark X Y H".

H is like.... soooooooo last branch xD

Re: /Worlds populated

Posted: December 28th, 2011, 11:09 am
by fragmer
If you use...

Code: Select all

.Where( w => w.Count > 0 )
...then you will be revealing worlds that have hidden players online. To be safe, I recommend doing...

Code: Select all

.Where( w => w.Players.Any( player.CanSee ) )
fCraft already uses this method for "/worlds populated" since 0.611 (r1277+).

I fixed "/Help Mark" in 0.700 (r1340), thank you for catching it.

Re: /Worlds populated

Posted: December 28th, 2011, 11:15 pm
by Jonty800
Oh, facepalm, I forgot about hidden players thanks!

Is there a way I can see a long list of all your commits? Cia only shows me the last 20 or something.

Re: /Worlds populated

Posted: December 29th, 2011, 12:51 pm
by fragmer
Use an SVN client to see the complete revision log.