[Freeciv-Dev] (PR#3783) support for pixel_border bigger that 1

["Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>]

[jdorje - Mon Mar 24 18:11:49 2003]:

> On Mon, 2003-03-24 at 12:58, Rafa³ Bursig wrote:
> 
> > Some time ago I try 128x64 tileset theme. Simple way to get such 
> > grapchics is resize exist tilesets x 2 but problem is that pixel 
> > borders are resize too. I haven to edit those graphics or add support 
> > for that bigger pixel border. ( I prefered secound )
> > 
> > method is easy :
> > is_pixel_border = 0 - no pixel border
> > is_pixel_border = 1 - pixel border exist ( 1 pixel )
> > is_pixel_border = 2 - pixel border exist ( 2 pixels )
> 
> Changing this value from a boolean to an integer is quite reasonable. 
> But we also need to rename it if we do this, while keeping
> backward-compatibility at the same time.  This becomes a bit ugly, but I
> think the loading code could be something like
> 
>   if (secfile_lookup_bool_default(file, FALSE,
>                                   "%s.is_pixel_border", ...) {
>     pixel_border = 1;
>   } else {
>     pixel_border = secfile_lookup_int_default(file, 0,
>                      "%s.pixel_border", ...);
>   }
> 
> Which would keep backward-compatibility while changing the variable name
> to match the new meaning.

And the patch.

jason

npixel_border-2.diff
Description: npixel_border-2.diff