Follow-up Comment #10, bug #13869 (project freeciv):

Hm, this is crazy, I didn't look at this patch, but it seems now that it
doesn't work at all.  I didn't understand this was to replace the validation
function, I thought it was about the action callback.  I am strongly
against.

It is normal to pass a string to a string validation function, a boolean to a
boolean validation function and an integer to a validation function!  This
patch just check if the current value is normal, not the value we want to
set.

I guess this would have to be reverted.


    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  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