[Confuse-devel] Fwd: Small patches for const/shared

[Martin Hedenfalk]

> From: Jens Rehsack <address@hidden>
> To: address@hidden
> Subject: Fwd: Small patches for const/shared
> 
> Hi,
> 
> I think it's not a good idea to have a closed devel-mailing
> list. I do not intend to increase my engagement in OSS
> to one more library and collect mailing lists. I'm already
> subscribed to about 20 lists, I can't read them all when
> I add more.

Of course, you're absolutely right. It's rather silly to require subscription 
for this mailing list.

But it was closed out of frustration over the amount of spam it received. 
Unless Savannah has started using greylisting, however, I'm afraid it will stay 
that way.

I'll forward you email to the list. Thank you.

        -martin


> Jens
> 
> ---------- Forwarded message ----------
> From:  <address@hidden>
> Date: 2010/10/7
> Subject: Small patches for const/shared
> To: address@hidden
> 
> 
> You are not allowed to post to this mailing list, and your message has
> been automatically rejected.  If you think that your messages are
> being rejected in error, contact the mailing list owner at
> address@hidden
> 
> 
> 
> ---------- Weitergeleitete Nachricht ----------
> From: Jens Rehsack <address@hidden>
> To: address@hidden
> Date: Thu, 7 Oct 2010 08:36:39 +0200
> Subject: Small patches for const/shared
> Hi all,
> 
> first: thanks for libConfuse. It saves us a lot of work.
> 
> Nevertheless I have two small patches (the const-patch probably could
> be extended to more functions).
> Furthermore, it doesn't solve the:
> 
> config.cpp:46: warning: deprecated conversion from string constant to ‘char*’
> config.cpp:46: warning: deprecated conversion from string constant to ‘char*’
> config.cpp:54: warning: deprecated conversion from string constant to ‘char*’
> config.cpp:54: warning: deprecated conversion from string constant to ‘char*’
> ...
> 
> It wasn't enought/didn't work to make all cfg_opt_t instances const like
> const cfg_opt_t foot_opts[] = {...};
> When I have some time after finishing the PoC, I'll take a deeper look.
> 
> The second patch allows users to enable libconfuse' shared library target.
> I don't see any reason to prevent it at all. Linking compatibility can be 
> solved
> with appropriate library version numbers
> (http://www.lrde.epita.fr/~adl/autotools.html).
> 
> Best regards,
> Jens

const.patch
Description: Binary data

allow-shared.patch
Description: Binary data

>