[Freeciv-Dev] Re: Cleaning up gotohand.c (PR#1068)

[Andrew Sutton <ansutton@xxxxxxx>]

> > > -   /* NOT c = 1 (Syela) [why not? - Thue] */
> > > -   move_cost = (ptile->move_cost[dir] ? SINGLE_MOVE : 0);
> > > +   /* NOT move_cost = MOVE_COST_ROAD (Syela) [why not? - Thue]
> > > +    * possibly because if an igter unit reuses warmap of a city
> > > +    * (see generate_warmap), it won't be THAT much different
> > > +    * from the correct warmap.  For consistency we should have
> > > +    * SINGLE_MOVE for disembarking igter unit too -- GB */
> >
> > No. Please don't write novels in comments. Especially if they start
> > with "possibly".
>
> Why not? Really???
> I think it's ok...

i'd agree with daniel. comments don't make code worse, or less readable. 
although, it would be nice if the comments were a little more understandable 
than "NOT move_cost = ....".

andy