Follow-up Comment #8, bug #18764 (project freeciv):

Spotted in passing: this bail-out check in real_menus_update() looks wrong to
me:


  if (NULL == ui_manager && !can_client_change_view()) {
    return;
  }


I'd have thought it should be ||, not &&. Any other opinions?
This came in in patch #1348 (between 2.2 and 2.3).
However, changing this doesn't make the problem go away.

    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?18764>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


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

Reply via email to