[SSSD] [PATCH] [TRACE] Adding ability to trace 64bit numbers

Jakub Hrozek jhrozek at redhat.com
Tue Apr 13 11:38:51 UTC 2010


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 04/12/2010 05:09 PM, Dmitri Pal wrote:
> Jakub Hrozek wrote:
>> On 04/10/2010 01:45 AM, Dmitri Pal wrote:
>>> Dmitri Pal wrote:
>>>> Addresses ticket https://fedorahosted.org/sssd/ticket/447
>>>> Applies only on to of my previous ini metadata patch.
>>>>
>>>>   
>>> Forgot to say that this is a patch in the title.
>>
>>
>> Nack, this does not compile with the default settings which do not have
>> HAVE_TRACE #defined, you need to add #define TRACE_LNUMBER(level, msg,
>> num) for that case, too.
> 
> The attached patch addresses the issue and adds other macros for signed
> numbers.
> 

The code seems to be OK now, but when you added the signed/unsigned
division it seems to me that more common naming scheme is to treat
signed as implicit and unsigned as explicit...i.e:

TRACE_NUMBER = signed int
TRACE_UNUMBER = unsigned int
TRACE_LNUMBER = signed long int
TRACE_ULNUMBER = unsinged long int

..etc

But that's just a suggestion, not a hard nack..
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAkvEV8sACgkQHsardTLnvCXjHgCfdTuT/yPpQLJKWNMZMsVA6x1t
XDsAn2VSRMdPxOt0BrVRxtUknvpMbRXu
=xgmJ
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list