On Tue, Aug 18, 2015 at 05:15:02PM +0200, Pavel Březina wrote:
On 08/18/2015 01:35 PM, Jakub Hrozek wrote:
>I wonder if it would be cleaner if there was a function that would
>accept a mem_ctx and line "syntax" and would create a
>sss_colondb_read_field* describing a line. Then sss_colondb_readline()
>would remove the talloc children first, then fill new data.
>
>Or even make the line syntax part of open?
>
>The usage could look like this:
> db = sss_colondb_open(mem_ctx, SSS_COLONDB_READ, "u:s:s:u");
> whle (sss_colodb_readline(db, &line)) {
> }
>
>I think it would be nicer to have the line syntax part of open, because
>the line syntax can't change, it's a property of the file.
This is nice. Shall I create a ticket for it? I'd rather get those patches
pushed and move to fqdn fix first, since this may take longer time to get
done right.
Sure, create a ticket, self-assign it and put it into 1.13.2.