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

[Matthias Bolte <matthias bolte googlemail com>]

2010/3/31 Matthias Bolte <matthias bolte googlemail com>:
> 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