Complete.Org: Mailing Lists: Archives: freeciv-dev: December 2001:
[Freeciv-Dev] Re: [PATCH] base_real_map_distance (PR#1049) 		Home

[Freeciv-Dev] Re: [PATCH] base_real_map_distance (PR#1049)

[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: rf13@xxxxxxxxxxxxxxxxxxxxxx, Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>, Raahul Kumar <raahul_da_man@xxxxxxxxx>
Cc: Petr Baudis <pasky@xxxxxxxxxxx>, jdorje@xxxxxxxxxxxxxxxxxxxxx, freeciv-dev@xxxxxxxxxxx, bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: [PATCH] base_real_map_distance (PR#1049)
From: Andrew Sutton <ansutton@xxxxxxx>
Date: Wed, 5 Dec 2001 11:39:11 -0500 
On Wednesday 05 December 2001 11:18 am, Raimar Falke wrote:

>  <some number>
>  This isn't about initialization!
>
> /* A */
>   x = punit->x, y = punit->y;
>
> /* B */
>   x = punit->x;
>   y = punit->y;

you're kidding, right? somebody would actually use A? it's not very readable 
and actually has a semantic meaning other than just doing a couple of things 
in one line.

int x = (1, 2); /* legal but weird */

my vote is for B. i prefer readability over concise code - when they do the 
same thing.

andy
[Prev in Thread] Current Thread [Next in Thread]