<URL: http://bugs.freeciv.org/Ticket/Display.html?id=40303 >

Marko Lindqvist wrote:
> <URL: http://bugs.freeciv.org/Ticket/Display.html?id=40303 >
> 
>  $subject
> 
>  To be committed soon.

    if (game.player_ptr) {
      char *text;

-    if (game.player_ptr->is_ready) {
+    if (game.player_ptr == NULL) {
+      text = _("_Ready");
+    } else if (game.player_ptr->is_ready) {
        text = _("Not _ready");
      } else {
        int num_unready = 0;

This patch makes no sense.

How can game.player_ptr be NULL when 3 lines above there's a check to 
see if it's non-NULL?

And why is the text "Not ready" when the is_ready field is set?

-jason



_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to