Complete.Org: Mailing Lists: Archives: freeciv-dev: December 2001:
[Freeciv-Dev] Re: [PATCH] convert save_options() to secfile (PR#1097)
Home

[Freeciv-Dev] Re: [PATCH] convert save_options() to secfile (PR#1097)

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Mike Kaufman <mkaufman@xxxxxxxxxxxxxx>
Cc: freeciv-dev@xxxxxxxxxxx, bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: [PATCH] convert save_options() to secfile (PR#1097)
From: Daniel Sjölie <deepone@xxxxxxxxxx>
Date: Fri, 7 Dec 2001 08:57:47 +0100

On 2001-12-06 17:20:40, Mike Kaufman wrote:
> here's another patch...
> 
> currently the client uses fprintf() to put options into .civclientrc
> this patch changes this to secfile_insert_* etc, and does away with 
> open_option_file() to boot.
> 
> pros:         consistency (somewhat lame, but alright)
>       reduces code (a bit)
>       allows one to easily save global worklists to file (good)
> 
> this last one is mainly what I'm aiming for. I personally think that
> it's silly to have to create new global worklists every time I play.
> It's almost not worth doing. Saving worklists to file would allow me to
> do it once and then forget about it. More on this when I have code to
> show.
> 
> oh yeah, there's a con with this patch:
> 
> con:
>       current code puts a nice comment beside the message options.
>       secfile_insert can't do this (if it can somebody show me how)
>       so there's a kludge to make a comment. It gets the job done,
>       but its not pretty. I don't think that the patch should be 
>       rejected for this however.

I like it...
Perhaps there is a better way to solve the problem though?
How about adding a "secfile_insert_comment"?? Or something similar...

/Daniel

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


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