Complete.Org: Mailing Lists: Archives: freeciv-dev: August 2001:
[Freeciv-Dev] [BUGFIX] 't' command active unit bugfix
Home

[Freeciv-Dev] [BUGFIX] 't' command active unit bugfix

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Freeciv Developers List <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] [BUGFIX] 't' command active unit bugfix
From: Stepan Roh <stepan@xxxxxxxx>
Date: Thu, 30 Aug 2001 00:46:52 +0200 (CEST)

Attached is a diff against current CVS (generated with cvs diff -u) which
fixes bug related to 't' command and blinking active unit. It's simple,
not very clean and only for GTK client (though could be easily ported to
other clients). The problem with 't' command is that it's stateless. I
think it should be changed. Maybe it could be combined with "Map grid"
command which puts red border around each used tile and white border
around each tile, which can be used. There could be tile info (those three
colored boxes with food/shields/trade) in those red bordered tiles like in
't' command (but without shading).

This patch doesn't solve the problem when one or more of shaded tiles get
refreshed (and loses 't' info), but rest of tiles stays unchanged. This is
ugly.

I also spotted some more problems with blinking active units: they
overwrite post-generated (?) map info (like city names, production, etc.).
This should be changed too, IMHO.

Have a nice day.

Stepan Roh

Attachment: t_command_active_unit_bugfix.diff
Description: diff


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