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. Some

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

2013-11-24 Thread Max Kellermann
On 2013/11/24 12:34, Denis Krjuchkov de...@crazydev.net 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?

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 any

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

2013-11-24 Thread Max Kellermann
On 2013/11/24 14:08, Denis Krjuchkov de...@crazydev.net 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