Re: [libvirt] [PATCHv2 00/10] Begin to replace scanf and atoi

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

 



2010/3/31 Matthias Bolte <matthias.bolte@xxxxxxxxxxxxxx>:
> This set of patches is a first step towards removing the scanf and atoi
> usage in libvirt. I began with the simple cases and post this now to get
> some feedback before I start to convert the more difficult cases.
>
> Changes in v2:
> - don't test tmp for NULL
> - note in commit message if parsing is stricter now
> - don't leak in PRUnicharToInt (vbox)
> - strip a trailing \n in openvzGetVPSUUID, as it was done before
> - don't use strspn to skip whitespaces, virStrToLong* does this already
> - fix virStrToLong* return value tests to check for < 0 instead of < 3 or != 1
> - add STRSKIP to skip a given prefix of a string
>
> Thanks to Eric Blake and Jim Meyering.
>
> Matthias
>

I folded in Eric's comments and pushed the whole v2 series.

Matthias

--
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]