[Freeciv-Dev] (PR#8491) Re: Freeciv commit: kauf: A new connect dialog for the client. This allo...

["Vasco Alexandre da Silva Costa" <vasc@xxxxxxxxxxxxxx>]

<URL: http://rt.freeciv.org/Ticket/Display.html?id=8491 >

> [i-freeciv-lists@xxxxxxxxxxxxx - Mon Apr 12 14:02:18 2004]:
> 
> On Mon, Apr 12, 2004 at 06:41:13AM -0700, Mike Kaufman wrote:
> > > IMHO this should look like:
> > >
> > >   SINT32 int_value;
> > >   SINT32 int_value_default;
> > >   SINT32 int_value_min;
> > >   SINT32 int_value_max;
> > >
> > >   BOOL bool_value;
> > >   BOOL bool_value_default;
> > >
> > >   STRING string_value[MAX_LEN_PACKET];
> > >   STRING string_value_default[MAX_LEN_PACKET];

I also think this is better. Or alternatively structs within structs.
i.e.:

struct {
  struct {
    SINT32 value;
    SINT32 default;
    SINT32 min;
    SINT32 max;
  } int;

  struct {
    BOOL value;
    BOOL default;
  } bool;
}

But I doubt the RPC IDL allows it. The current names also got me
confused when I first looked at them.