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

[Mike Kaufman <mkaufman@xxxxxxxxxxxxxx>]

On Tue, Jan 15, 2002 at 11:44:21PM +0100, Reinier Post wrote:
> On Thu, Dec 06, 2001 at 05:20:40PM -0800, 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.
> 
> What about backward compatibility?
> Apart from that it's great.

What it doesn't do, (as has been pointed out) is add comments is a nice 
manner, though you can do it. And since comments are stripped out
anyway, it's backwards compatible.

instead of:

message_where_00=3  # Low Funds

you get:

# 0=== Low Funds ===0
message_where_00=3


If you've tried CMA 3.0 or 3.1, you've used this patch.

I do not yet have the kahunas necessary to add a
secfile_insert_comment() to the registry. I would welcome anyone to do
so. It'd be nice to be able to place a comment _after_ a setting instead
of it's own line...

-mike