On Fri, Jan 16, 2015 at 11:41:19AM +0100, Pavel Březina wrote:
On 01/16/2015 11:39 AM, Jakub Hrozek wrote:
>On Fri, Jan 16, 2015 at 11:29:27AM +0100, Pavel Březina wrote:
>>One more question - uid_t is unsigned int. Should we use it in FindById
>>instead of int64_t which was taken from the original design page?
>
>uid_t is unsigned 32bit on most (all?) platforms. IIRC int64_t was
>chosen so that we cover the 32bit value range and can use the negative
>range for errors.
I don't think I follow. What errors? Errors are reported through D-Bus error
message, not through an input parameter.
OK, for input parameters and properties it makes sense to use unsigned
number. I just checked AccountsService and they use UInt64 -- not sure
why, can you check with Stef or Matthias Clasen ? I can't think of other
reason than representing undefined or extra values, though.
In SSSD itself, we use uint32_t to represent UIDs anyway.