On 3 Jan 2012, at 23:44, Stuart Buchanan wrote:

> 1) I've included location information for the servers, so they
> can be used sensibly by users to determine the correct MP server
> to connect to. So our instructions for new users can hopefully be simply
> to open the dialog, select a server close to the user, and press Connect.
> Another win for usability - great work James et al. !

Excellent.

> 
> 2) I've remove the port setting. This defaults to 5000, and isn't something
> that a user connecting to the standard MP servers is going to change.
> If someone is connecting to a non-standard server, they'll need to do so
> from the command line anyway

Yep, seems reasonable to me - as long as we retain the functionality under the 
hood, it's one less thing for users to get wrong in the GUI :)

> 3) The only way I found to disconnect from the MP environment was to set
> the txhost to "0.0.0.0" (from a code read of multiplayermgr.cxx). Is
> there a better
> way to disconnect? As far as I can see, there's no way to do so gracefully at
> present - looks like an "else" branch is missing from the if() test of
> the txhost
> string.

Right now setting an empty host is the way to disconnect it. A separate 
'connect' boolean input to the code could be arranged, which triggers 
connection & disconnection when set - but obviously this also needs to keep 
compatibility with any command line use. It would also be trivial to add 
explicit 'mp-connect' and 'mp-disconnect' commands, instead of expressing 
everything as property state changes.

James

------------------------------------------------------------------------------
Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex
infrastructure or vast IT resources to deliver seamless, secure access to
virtual desktops. With this all-in-one solution, easily deploy virtual 
desktops for less than the cost of PCs and save 60% on VDI infrastructure 
costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox
_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel

Reply via email to