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

> assert() should be depreciated in newer version. Prefer the
> log_assert_ret() macro or RETURN_IF_FAIL() in S2_2. 

I thought about this but did not know that to safely return? map_colatitude()
should return the colatitude. Should I use 0, MAX_COLATITUDE or a value
inbetween? Perhaps 'MAX_COLATITUDE / 2' as for all temperate maps could be OK
- see the attached patch.

(file #8001)
    _______________________________________________________

Additional Item Attachment:

File name:
version2-trunk-S2_2-add-check-for-ptile-NULL-within-map_colatitude.patch
Size:0 KB


    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  Nachricht geschickt von/durch Gna!
  http://gna.org/


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

Reply via email to