Eric Blake <ebl...@redhat.com> writes:
> On 02/14/2017 04:25 AM, Markus Armbruster wrote:
>> Signed-off-by: Markus Armbruster <arm...@redhat.com>
>> ---
>> tests/test-cutils.c | 24 ++++++++++++++++++++++++
>> 1 file changed, 24 insertions(+)
>>
>
> Perhaps you could also add a test of "-". Either way,
The test cases match the ones for qemu_strtol() & friends:
test_qemu_stro*_{empty,whitespace,invalid}(). If we add "-" here, we
should probably add it there as well. I think we totally should if we
were testing a strtol() implementation. But we're only testing a
wrapper, and other test cases cover the "strtol() fails without
converting anything" case, so I think we're okay.
> Reviewed-by: Eric Blake <ebl...@redhat.com>
Thanks!
