URL:
<http://gna.org/patch/?6842>
Summary: Do not include netintf.h to mapimg.c
Project: Freeciv
Submitted by: cazfi
Submitted on: Fri 22 Jan 2016 10:50:27 PM EET
Category: general
Priority: 5 - Normal
Status: Ready For Test
Privacy: Public
Assigned to: None
Originator Email:
Open/Closed: Open
Discussion Lock: Any
Planned Release: 2.5.3, 2.6.0, 3.0.0
_______________________________________________________
Details:
Include of netintf.h has been in mapimg.c from the beginning -> it has not
been added when it has been required. Mapimg should have nothing to do with
networking, and as it compiles fine without, attached patch removes the
include.
If there's one "blindly" added include there, it's quite possible that some of
the other includes are unnecessary too. Feel free to investigate that, but I'm
now interested about removal of this more troublesome include contributing to
the networking code mess.
_______________________________________________________
File Attachments:
-------------------------------------------------------
Date: Fri 22 Jan 2016 10:50:27 PM EET Name: MapimgNetHdrNot.patch Size: 369B
By: cazfi
<http://gna.org/patch/download.php?file_id=26424>
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?6842>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
