Re: [PATCH 02/12] storage: avoid mishandling backing store > 2GB

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

 



On 06/08/2011 10:05 AM, Richard W.M. Jones wrote:
> On Tue, Jun 07, 2011 at 03:34:58PM +0200, Matthias Bolte wrote:
>> Using unsigned int instead of uint32_t would also work and avoid
>> stdint.h types that the libvirt codebase avoids.
> 
> Out of interest, why is libvirt avoiding these int types?

Actually, I don't see anything wrong with using these types internally
(which this file was) where it makes sense; it just becomes a bit more
of a maintenance burden to remember to use the correct PRI* macros
rather than raw %d and friends when printing such values.  But for the
public APIs, we've chosen to avoid those types and to instead stick to
native int types, so that our public headers do not break in the face of
broken <stdint.h> files (it's sad how many platforms where gnulib ends
up replacing <stdint.h> because of bugs).

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]