[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 11:48:20AM +0200, Lukas Slebodnik wrote: > Thank you for review. > > Feel free to change/add comments before pushing patches. I just added a simple comment that the structure field is unused at the moment. All patches are pushed to ding-libs master: *

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Lukas Slebodnik
On (22/06/16 11:43), Jakub Hrozek wrote: >On Wed, Jun 22, 2016 at 10:52:34AM +0200, Lukas Slebodnik wrote: >> On (22/06/16 10:39), Jakub Hrozek wrote: >> >On Wed, Jun 22, 2016 at 09:28:26AM +0200, Lukas Slebodnik wrote: >> >> ehlo, >> >> >> >> The first 4 patches are slightly modified version >>

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 10:52:34AM +0200, Lukas Slebodnik wrote: > On (22/06/16 10:39), Jakub Hrozek wrote: > >On Wed, Jun 22, 2016 at 09:28:26AM +0200, Lukas Slebodnik wrote: > >> ehlo, > >> > >> The first 4 patches are slightly modified version > >> of Michal's patches. Mostly coding style

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Lukas Slebodnik
On (22/06/16 10:39), Jakub Hrozek wrote: >On Wed, Jun 22, 2016 at 09:28:26AM +0200, Lukas Slebodnik wrote: >> ehlo, >> >> The first 4 patches are slightly modified version >> of Michal's patches. Mostly coding style issues, fixed reports from static >> analyzers + fixed small issues which would

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 09:28:26AM +0200, Lukas Slebodnik wrote: > ehlo, > > The first 4 patches are slightly modified version > of Michal's patches. Mostly coding style issues, fixed reports from static > analyzers + fixed small issues which would be fixed as part of review process. > > Last