On Wed, 25 Jan 2017 10:10:18 +0100
"Enrico Weigelt, metux IT consult" <[email protected]> wrote:

No.  Please, enough of these.  The code below is commented out
temporarily to make it easier to test multiplayer while I extract more DOM
from the login code.

>  
>          case IN_GAME:
> -            /*
> -            // Disable this check if you need to debug a multiplayer client.
> -            if 
> (FreeColDebugger.isInDebugMode(FreeColDebugger.DebugMode.MENUS)) {
> -                getGUI().showErrorMessage(StringTemplate
> -                    .template("client.debugConnect"));
> -                return false;
> -            }
> -            */
>              // Find the players, choose one.
>              List<String> names = fcc.getVacantPlayerNames();
>              if (names.isEmpty()) return false;

You are so keen to mess with stuff that you do not know the history of
or reasons for... and I am faced with a huge pile of patches to review.

We have a resource problem here.  FreeCol is a small project with few
contributors.  We need to focus on high priority tasks, not
counterproductive stuff like the above.

Cheers,
Mike Pope
 

Attachment: pgpR6Q4j5GQT_.pgp
Description: OpenPGP digital signature

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Freecol-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/freecol-developers

Reply via email to