[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] Include stdint.h in libvirt.h for INT64_MAX



2011/3/11 Eric Blake <eblake redhat com>:
> On 03/11/2011 02:12 PM, Matthias Bolte wrote:
>>> UL still won't work on 32-bit platforms.  You need the LL (or ULL) suffix.
>>>
>>
>> And now the final v3 for something that should have been a trivial patch :)
>
> application/mbox?  Why not text/plain for the MIME type?

Probably because Google Mail decided to detect this one that way,
typically it uses text/x-diff for attached patches. Unfortunately it
doesn't let me set the MIME type.

>> +++ b/include/libvirt/libvirt.h.in
>> @@ -772,7 +772,7 @@ typedef enum {
>>   * Macro providing the virMemoryParameter value that indicates "unlimited"
>>   */
>>
>> -#define VIR_DOMAIN_MEMORY_PARAM_UNLIMITED (INT64_MAX >> 10)
>> +#define VIR_DOMAIN_MEMORY_PARAM_UNLIMITED 9007199254740991LL /* = INT64_MAX >> 10 */
>
> ACK.
>

Thanks, pushed it, finally.

Matthias


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]