Re: [libvirt] [PATCHv2] maint: Kill usage of atoi()
On 11/14/2013 02:22 PM, Peter Krempa wrote:
>> +
>> +exclude_file_name_regexp--sc_prohibit_atoi= \
>> + ^examples/.*\.[ch]$$
>
> Drop this hunk. None of our examples/* use atoi to begin with, so
> there's no reason to exclude them from the syntax check.
>
> ACK with those fixes.
>
This isn't true unfortunately:
Huh, wonder what grep I did that missed that fact?
~/libvirt $ git grep atoi examples/
examples/domsuspend/suspend.c: id = atoi(argv[1]);
and I wanted to avoid changing that file. But if you insist I can tune
that one too.
Yes, it's worth fixing, because our examples should never teach people
to use bad coding practices. atoi() is broken by design, and strtol(),
while more awkward to use, is just as portably present as atoi().
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Attachments:
- signature.asc (application/pgp-signature — 621 bytes)