Re: [Qemu-devel] [PATCH v2 5/9] test-string-input-visitor: Add more tests

2018-11-21 Thread Markus Armbruster
David Hildenbrand writes: > On 20.11.18 18:26, Eric Blake wrote: >> On 11/20/18 11:20 AM, Eric Blake wrote: >>> On 11/20/18 11:06 AM, Eric Blake wrote: On 11/20/18 3:25 AM, David Hildenbrand wrote: > Test that very big/small values are not accepted and that ranges with > only one ele

Re: [Qemu-devel] [PATCH v2 5/9] test-string-input-visitor: Add more tests

2018-11-21 Thread David Hildenbrand
On 20.11.18 18:26, Eric Blake wrote: > On 11/20/18 11:20 AM, Eric Blake wrote: >> On 11/20/18 11:06 AM, Eric Blake wrote: >>> On 11/20/18 3:25 AM, David Hildenbrand wrote: Test that very big/small values are not accepted and that ranges with only one element work. Also test that ranges ar

Re: [Qemu-devel] [PATCH v2 5/9] test-string-input-visitor: Add more tests

2018-11-20 Thread Markus Armbruster
Eric Blake writes: > On 11/20/18 11:20 AM, Eric Blake wrote: >> On 11/20/18 11:06 AM, Eric Blake wrote: >>> On 11/20/18 3:25 AM, David Hildenbrand wrote: Test that very big/small values are not accepted and that ranges with only one element work. Also test that ranges are ascending and

Re: [Qemu-devel] [PATCH v2 5/9] test-string-input-visitor: Add more tests

2018-11-20 Thread Eric Blake
On 11/20/18 11:20 AM, Eric Blake wrote: On 11/20/18 11:06 AM, Eric Blake wrote: On 11/20/18 3:25 AM, David Hildenbrand wrote: Test that very big/small values are not accepted and that ranges with only one element work. Also test that ranges are ascending and cannot have more than 65536 elements

Re: [Qemu-devel] [PATCH v2 5/9] test-string-input-visitor: Add more tests

2018-11-20 Thread Eric Blake
On 11/20/18 11:06 AM, Eric Blake wrote: On 11/20/18 3:25 AM, David Hildenbrand wrote: Test that very big/small values are not accepted and that ranges with only one element work. Also test that ranges are ascending and cannot have more than 65536 elements. Rename expect4 to expect5, as we will

Re: [Qemu-devel] [PATCH v2 5/9] test-string-input-visitor: Add more tests

2018-11-20 Thread Eric Blake
On 11/20/18 3:25 AM, David Hildenbrand wrote: Test that very big/small values are not accepted and that ranges with only one element work. Also test that ranges are ascending and cannot have more than 65536 elements. Rename expect4 to expect5, as we will be moving that to a separate ulist test a

[Qemu-devel] [PATCH v2 5/9] test-string-input-visitor: Add more tests

2018-11-20 Thread David Hildenbrand
Test that very big/small values are not accepted and that ranges with only one element work. Also test that ranges are ascending and cannot have more than 65536 elements. Rename expect4 to expect5, as we will be moving that to a separate ulist test after the rework. Signed-off-by: David Hildenbra