Complete.Org: Mailing Lists: Archives: freeciv-dev: August 2001:
[Freeciv-Dev] [Fwd: Re: New city dialog - stage 12]
Home

[Freeciv-Dev] [Fwd: Re: New city dialog - stage 12]

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Cc: Freeciv List <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] [Fwd: Re: New city dialog - stage 12]
From: Daniel Sjölie <deepone@xxxxxxxxxx>
Date: Wed, 22 Aug 2001 22:14:22 +0200

Ok, since it seems Raimar missed this one... :)
I tried this patch just now on a fresh checkout and is seemed to work
very well... Had to run 'automake; autoconf' though...

/Daniel

----- Forwarded message from Thue <thue@xxxxxxx> -----

From: Thue <thue@xxxxxxx>
To: freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: New city dialog - stage 12
Date: Thu, 16 Aug 2001 22:31:02 +0200

On Wednesday 15 August 2001 18:02, Thue wrote:
> On Sunday 06 May 2001 12:13, Tuomas Airaksinen wrote:
> >  Hi,
> >
> >  New version of citydialog patch is out.
> >
> >  Changes:
> >
> > * Bug fix: you cannot have two city dialog with same cities (it was
> >   possible with arrow buttons (next/prev city))
> > * Mike Kaufman's cleanup patch merged:
> >     lots of indentation
> >     removed some empty space
> >     removed the gtk_widget_ref() and gtk_object_set_data_full()
> >     added some clist_freeze/thaw
> >     added sentry/fortify button and callback
> >     fixed diplomat/spy stuff.
> >     used gtk_notebook_append_page()
> >     removed some extraneous h/vboxes that were doing anything
> >     some commenting in the notebook
> >
> >  Patch:
> >  http://tuma.stc.cx/files/freeciv-new_citydialog-12.diff.gz
>
> I updated the patch to CVS and fixed some bugs. Get it at
> ftp://ftp.freeciv.org/freeciv/incoming/new_city_dialog.diff.bz2
>
> -Thue

This is a fairly big patch, and I have not been able to go it over line 
by line. But it has had a good deal of review by others on the list, 
and it
1) Seems to be structurally good.
2) Seems to work.

So I would recomment it was applied.
Since the remaining freeciv people and not I are going to live with it, 
it would seem fitting if one of them applied it.

-Thue

----- End forwarded message -----

-- 
Now take a deep breath, smile and don't take life so seriously... :)


[Prev in Thread] Current Thread [Next in Thread]
  • [Freeciv-Dev] [Fwd: Re: New city dialog - stage 12], Daniel Sjölie <=