[Freeciv-Dev] Re: Massive Patch!

[Raahul Kumar <raahul_da_man@xxxxxxxxx>]

--- Chris Richards <chrisr@xxxxxxxxxxxxxxxx> wrote:
> This is a massive patch which does some clean up in autoattack.c.  It
> removes a redundant variable and conditional.  Beware, it isn't for
> the weak of heart.
> 
> /cjr
> 
> > Index: server/autoattack.c
> ===================================================================
> RCS file: /home/freeciv/CVS/freeciv/server/autoattack.c,v
> retrieving revision 1.29
> diff -u -w -b -r1.29 autoattack.c
> --- server/autoattack.c       2001/10/30 10:34:23     1.29
> +++ server/autoattack.c       2001/12/28 20:32:58
> @@ -149,14 +149,13 @@
>  {
>    int id = punit->id;
>    struct unit *enemy;
> -  int debug = 0;
>  
>    enemy=search_best_target(pplayer,pcity,punit);
>  
>    /* nothing found */
>    if(enemy == NULL) return;
>    
> -  if (debug) freelog(LOG_DEBUG, "launching attack");
> +  freelog(LOG_DEBUG, "launching attack");
>    
>    notify_player_ex(pplayer, enemy->x, enemy->y, E_NOEVENT,
>                  _("Game: Auto-Attack: %s's %s attacking %s's %s"),
> 


Looks good. It's nice to see the big patches start to roll in.

__________________________________________________
Do You Yahoo!?
Send your FREE holiday greetings online!
http://greetings.yahoo.com