On 09/12/2016 10:01 AM, Lukas Slebodnik wrote:
> You have a point here (and I regret adding the ENOENT retval in
> >but the difference is that ldb_search wrapper changes /functionality/, this
> >just adds logging. So the only thing we would miss if we forget to use
> >the wrapper is the extra debugging. And in that case we would have to
> >build a new package and commit the messages to master, but that's no
> >different from missing debug messages in general.
Inconsistencies are bad. it does not matter wheter it's about ENOENT
if you are afraid of inconsistency in using wrappers at all places in
code I think I could prepare test which say you that you should write
sss_ldb_* instead od ldb_*.