[SSSD] DEBUG macro refactoring v6
Jakub Hrozek
jhrozek at redhat.com
Wed Feb 12 21:44:51 UTC 2014
On Wed, Feb 12, 2014 at 10:40:49PM +0100, Jakub Hrozek wrote:
> On Wed, Feb 12, 2014 at 05:04:39PM +0100, Jakub Hrozek wrote:
> > On Wed, Feb 12, 2014 at 10:38:45AM -0500, Stephen Gallagher wrote:
> > > -----BEGIN PGP SIGNED MESSAGE-----
> > > Hash: SHA1
> > >
> > > On 02/12/2014 10:12 AM, Jakub Hrozek wrote:
> > > > On Wed, Feb 12, 2014 at 10:10:10AM -0500, Stephen Gallagher wrote:
> > > >> Self-nack.
> > > >>
> > > >> Sorry, I just realized I mistakenly rebased the v3 patches, not
> > > >> the v4 ones. I'll get a new set of these out in a short while.
> > > >
> > > > Thanks, the ldb messages were always displayed with this version
> > > > :)
> > > >
> > >
> > > Yup, that's how I realized I made the mistake.
> > >
> > > > I'll refrain from pushing any more patches to master in the
> > > > meantime.
> > >
> > > Thanks.
> > >
> > >
> > > New patches attached. These should be the correct ones now. These
> > > necessitated a rebase of my journald patches as well, so I'll send
> > > those too.
> >
> > all debug levels seem to be working fine
> >
> > ACK
> >
> > I'll just run them through coverity to be sure.
>
> Coverity found some errors which seem completely unrelated - code that
> was prefixed by a comment saying "we should never get here" was marked as
> dead code. I suspect that this is due to a new scanner version and dead
> code is a soft error anyway, so I'm going to push the patches.
>
> Thanks everyone for the work put into the patchset and the patience!
Pushed to master.
More information about the sssd-devel
mailing list