Update of bug #19385 (project freeciv):
Summary: inconsequent return-type => unit_is_in_focus()
returns pointer value as bool
_______________________________________________________
Follow-up Comment #1:
Hm. Debatable.
I think this sort of thing -- not insisting bools only have values 0 or 1 --
is a common C idiom in some codebases; in such a context it's unwise to test
for ==TRUE, and care is needed when assigning to bitfields.
I'm not sure if we have an explicit policy on this in Freeciv.
Is VC complaining about it? What's the warning?
I'd be slightly surprised if this were the only instance of this sort of thing
in the Freeciv codebase.
(I see you've renamed the file to .cpp -- are you compiling it as C++ rather
than C? I don't know if idiom or rules differ there.)
_______________________________________________________
Reply to this item at:
<http://gna.org/bugs/?19385>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
