[SSSD] [PATCHES] New DEBUG facility
Jan Zelený
jzeleny at redhat.com
Thu Aug 18 15:01:13 UTC 2011
> Dne 17.8.2011 15:29, Jan Zelený napsal(a):
> >>> Patch #0006:
> >>> -<nitpick> please just for safety use parentheses in ternary
> >>> operator (line 92 and couple more similar)</nitpick>
> >>
> >> Done and also moved to macro.
> >
> > You still don't have parentheses around the condition in ternary
> > operator. No big deal though, it doesn't concern me in the macro since
> > bringing an error to one possible place is less likely than to one of
> > many possible places.
>
> Sorry about that. Now it's done :-)
Oh, I just noticed what originally happened: you put the parentheses around
the entire ternary operator instead of the condition as I had in mind. However
I have no problem with the way it's done now.
>
> > One more thing: I don't like the invalidate macro. The assignment with
> > the comment you provided is clear enough I think.
>
> Removed.
>
> I also noticed that printf provides # flag, that adds 0x when you want
> output in hex. So I changed 0x%.4x to %#.4x.
I told myself that there must be something like that ;-) Thanks for saving me
the work and finding that out on your own.
Ack to all 6 patches
Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110818/e49f6805/attachment.sig>
More information about the sssd-devel
mailing list