Re: [Musicpd-dev-team] New command line parser

2013-11-26 Thread Denis Krjuchkov
25.11.2013 23:19, Matthias Petschick пишет: > Hello, > > this patch (db238cc23f3bb5c9768d30ae24b99e67a5795203) introduces a > segfault upon client connect when mpd is compiled with libwrap > enabled. The reason is the removal of g_option_context_parse, which > calls g_set_prgname "as a side-effect"

Re: [Musicpd-dev-team] New command line parser

2013-11-25 Thread Matthias Petschick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hello, this patch (db238cc23f3bb5c9768d30ae24b99e67a5795203) introduces a segfault upon client connect when mpd is compiled with libwrap enabled. The reason is the removal of g_option_context_parse, which calls g_set_prgname "as a side-effect". Some l

Re: [Musicpd-dev-team] New command line parser

2013-11-24 Thread Max Kellermann
On 2013/11/24 14:08, Denis Krjuchkov wrote: > 24.11.2013 18:38, Max Kellermann ??: > >I was wondering whether we should use boost for that. Now you > >implemented a new parser. Why that? > > > > Why not? Probably for the complex cases it's better to use a > library* but MPD currently us

Re: [Musicpd-dev-team] New command line parser

2013-11-24 Thread Denis Krjuchkov
24.11.2013 18:38, Max Kellermann пишет: > I was wondering whether we should use boost for that. Now you > implemented a new parser. Why that? > Why not? Probably for the complex cases it's better to use a library* but MPD currently uses very simple flag options that are easily parsed without a

Re: [Musicpd-dev-team] New command line parser

2013-11-24 Thread Max Kellermann
On 2013/11/24 12:34, Denis Krjuchkov wrote: > this time I'm reworking command line parsing to avoid using g_option > stuff. I was wondering whether we should use boost for that. Now you implemented a new parser. Why that?

[Musicpd-dev-team] New command line parser

2013-11-24 Thread Denis Krjuchkov
Hi, this time I'm reworking command line parsing to avoid using g_option stuff. Commit: http://git.musicpd.org/cgit/dk/mpd.git/commit/?h=new-cmdline-parser&id=db238cc23f3bb5c9768d30ae24b99e67a5795203 Pull from new-cmdline-parser branch of git://git.musicpd.org/dk/mpd.git -- Denis ---