cfg_setopt()

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

cfg_setopt()

Daniel.Pocock



Hi,

Is there a reason why cfg_setopt is not in the public API?

I want to do something like this in my application, manually adding a
new instance of `my_multi_section' after parsing has completed:

   cfg_opt = cfg_getopt(config_file, "my_multi_section");
   cfg_val_tmp = cfg_setopt(config_file, cfg_opt, 0);
   cfg_tmp = cfg_val_tmp->section;
   cfg_setstr(cfg_tmp, "my_value", "hello world");

my_multi_section is defined:

   CFG_SEC("my_multi_section", my_multi_section_opts, CFGF_MULTI)

I've patched confuse.c and confuse.h so that cfg_setopt is in the API,
and it seems to work.

Regards,

Daniel
_______________________________________________

This e-mail may contain information that is confidential, privileged or otherwise protected from disclosure. If you are not an intended recipient of this e-mail, do not duplicate or redistribute it by any means. Please delete it and any attachments and notify the sender that you have received it in error. Unless specifically indicated, this e-mail is not an offer to buy or sell or a solicitation to buy or sell any securities, investment products or other financial product or service, an official confirmation of any transaction, or an official statement of Barclays. Any views or opinions presented are solely those of the author and do not necessarily represent those of Barclays. This e-mail is subject to terms available at the following link: www.barcap.com/emaildisclaimer. By messaging with Barclays you consent to the foregoing.  Barclays Capital is the investment banking division of Barclays Bank PLC, a company registered in England (number 1026167) with its registered office at 1 Churchill Place, London, E14 5HP.  This email may relate to or be sent from other members of the Barclays Group.
_______________________________________________


_______________________________________________
Confuse-devel mailing list
[hidden email]
http://lists.nongnu.org/mailman/listinfo/confuse-devel
Reply | Threaded
Open this post in threaded view
|

RE: cfg_setopt() [patch]

Daniel.Pocock


I got no reply to the earlier message I sent on this topic (below)

Therefore, I am posting the patch, attached
 

> -----Original Message-----
> From:
> confuse-devel-bounces+daniel.pocock=barclayscapital.com@nongnu
> .org
> [mailto:confuse-devel-bounces+daniel.pocock=barclayscapital.co
> [hidden email]] On Behalf Of Pocock, Daniel: IT (LDN)
> Sent: 13 August 2008 11:20
> To: [hidden email]
> Subject: [Confuse-devel] cfg_setopt()
>
>
>
>
> Hi,
>
> Is there a reason why cfg_setopt is not in the public API?
>
> I want to do something like this in my application, manually
> adding a new instance of `my_multi_section' after parsing has
> completed:
>
>    cfg_opt = cfg_getopt(config_file, "my_multi_section");
>    cfg_val_tmp = cfg_setopt(config_file, cfg_opt, 0);
>    cfg_tmp = cfg_val_tmp->section;
>    cfg_setstr(cfg_tmp, "my_value", "hello world");
>
> my_multi_section is defined:
>
>    CFG_SEC("my_multi_section", my_multi_section_opts, CFGF_MULTI)
>
> I've patched confuse.c and confuse.h so that cfg_setopt is in
> the API, and it seems to work.
>
> Regards,
>
> Daniel
> _______________________________________________
>
> This e-mail may contain information that is confidential,
> privileged or otherwise protected from disclosure. If you are
> not an intended recipient of this e-mail, do not duplicate or
> redistribute it by any means. Please delete it and any
> attachments and notify the sender that you have received it
> in error. Unless specifically indicated, this e-mail is not
> an offer to buy or sell or a solicitation to buy or sell any
> securities, investment products or other financial product or
> service, an official confirmation of any transaction, or an
> official statement of Barclays. Any views or opinions
> presented are solely those of the author and do not
> necessarily represent those of Barclays. This e-mail is
> subject to terms available at the following link:
> www.barcap.com/emaildisclaimer. By messaging with Barclays
> you consent to the foregoing.  Barclays Capital is the
> investment banking division of Barclays Bank PLC, a company
> registered in England (number 1026167) with its registered
> office at 1 Churchill Place, London, E14 5HP.  This email may
> relate to or be sent from other members of the Barclays Group.
> _______________________________________________
>
>
> _______________________________________________
> Confuse-devel mailing list
> [hidden email]
> http://lists.nongnu.org/mailman/listinfo/confuse-devel
>
_______________________________________________

This e-mail may contain information that is confidential, privileged or otherwise protected from disclosure. If you are not an intended recipient of this e-mail, do not duplicate or redistribute it by any means. Please delete it and any attachments and notify the sender that you have received it in error. Unless specifically indicated, this e-mail is not an offer to buy or sell or a solicitation to buy or sell any securities, investment products or other financial product or service, an official confirmation of any transaction, or an official statement of Barclays. Any views or opinions presented are solely those of the author and do not necessarily represent those of Barclays. This e-mail is subject to terms available at the following link: www.barcap.com/emaildisclaimer. By messaging with Barclays you consent to the foregoing.  Barclays Capital is the investment banking division of Barclays Bank PLC, a company registered in England (number 1026167) with its registered office at 1 Churchill Place, London, E14 5HP.  This email may relate to or be sent from other members of the Barclays Group.
_______________________________________________

_______________________________________________
Confuse-devel mailing list
[hidden email]
http://lists.nongnu.org/mailman/listinfo/confuse-devel

confuse-setopt.diff (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: cfg_setopt() [patch]

Martin Hedenfalk-4
6 okt 2008 kl. 13.23 skrev <[hidden email]> <[hidden email]
 >:

>
>
> I got no reply to the earlier message I sent on this topic (below)
>
> Therefore, I am posting the patch, attached

Hi,

I'm catching up with the mailing list and actually just commited your  
suggestion to CVS the other day. I'll add your doxygen documentation  
too, thanks.

        -martin



_______________________________________________
Confuse-devel mailing list
[hidden email]
http://lists.nongnu.org/mailman/listinfo/confuse-devel
Reply | Threaded
Open this post in threaded view
|

RE: cfg_setopt() [patch]

Daniel.Pocock

> I'm catching up with the mailing list and actually just
> commited your suggestion to CVS the other day. I'll add your
> doxygen documentation too, thanks.


Thanks Martin, I'm glad you find it useful.

I notice EPEL only includes v2.5:

  ftp://mirror.switch.ch/mirror/epel/5/x86_64/repoview/libconfuse.html

I'm guessing that adapting their spec file for 2.6 or 2.7 would not be
too hard, although I've been holding back until a 2.7 release comes
along (hopefully incorporating my patch) - have you got any date in mind
for such a release?

Regards,

Daniel
_______________________________________________

This e-mail may contain information that is confidential, privileged or otherwise protected from disclosure. If you are not an intended recipient of this e-mail, do not duplicate or redistribute it by any means. Please delete it and any attachments and notify the sender that you have received it in error. Unless specifically indicated, this e-mail is not an offer to buy or sell or a solicitation to buy or sell any securities, investment products or other financial product or service, an official confirmation of any transaction, or an official statement of Barclays. Any views or opinions presented are solely those of the author and do not necessarily represent those of Barclays. This e-mail is subject to terms available at the following link: www.barcap.com/emaildisclaimer. By messaging with Barclays you consent to the foregoing.  Barclays Capital is the investment banking division of Barclays Bank PLC, a company registered in England (number 1026167) with its registered office at 1 Churchill Place, London, E14 5HP.  This email may relate to or be sent from other members of the Barclays Group.
_______________________________________________


_______________________________________________
Confuse-devel mailing list
[hidden email]
http://lists.nongnu.org/mailman/listinfo/confuse-devel
Reply | Threaded
Open this post in threaded view
|

Re: cfg_setopt() [patch]

Martin Hedenfalk-4
7 okt 2008 kl. 13.48 skrev <[hidden email]> <[hidden email]
 >:

>
>> I'm catching up with the mailing list and actually just
>> commited your suggestion to CVS the other day. I'll add your
>> doxygen documentation too, thanks.
>
>
> Thanks Martin, I'm glad you find it useful.
>
> I notice EPEL only includes v2.5:
>
>  ftp://mirror.switch.ch/mirror/epel/5/x86_64/repoview/libconfuse.html
>
> I'm guessing that adapting their spec file for 2.6 or 2.7 would not be
> too hard, although I've been holding back until a 2.7 release comes
> along (hopefully incorporating my patch) - have you got any date in  
> mind
> for such a release?

I'd like to fix the lexer match bug (reported by Sylvain Bertrand on  
this list in july) before 2.7. So hopefully quite soon.

        -martin


>
> Regards,
>
> Daniel



_______________________________________________
Confuse-devel mailing list
[hidden email]
http://lists.nongnu.org/mailman/listinfo/confuse-devel