Follow-up Comment #2, bug #24935 (project freeciv):

The proposed fix isn't thread-safe, because the underlying Qt classes
(QStringList, QLabel) are not. You need a synchronization primitive (a mutex
or some kind of atomic variable). Thread-safety is easier to get right when
using Qt signals together with separation of concerns.

Also, the variable this_thread isn't destroyed properly.


    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  Message posté via/par Gna!
  http://gna.org/


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

Reply via email to