Re: [Qemu-devel] [PATCH v14 09/19] tests: Add check-qnull

2016-04-14 Thread Eric Blake
On 04/14/2016 11:37 AM, Markus Armbruster wrote: > There's one unusual bit: the other check-q*.c leave the visitor stuff to > the test-*-visitor.c, but this one doesn't. Hmm. Yeah, but the other test-*-visitor don't muck with internal reference counts of qnull_. I guess I just call it out more p

Re: [Qemu-devel] [PATCH v14 09/19] tests: Add check-qnull

2016-04-14 Thread Markus Armbruster
There's one unusual bit: the other check-q*.c leave the visitor stuff to the test-*-visitor.c, but this one doesn't. Hmm.

Re: [Qemu-devel] [PATCH v14 09/19] tests: Add check-qnull

2016-04-14 Thread Markus Armbruster
Eric Blake writes: > Add a new test, for checking reference counting of qnull(). As > part of the new file, move a previous reference counting change > added in commit a861564 into a more logical place. into or to? Those darn prepositions... > Signed-off-by: Eric Blake > > --- > v14: no chang

[Qemu-devel] [PATCH v14 09/19] tests: Add check-qnull

2016-04-08 Thread Eric Blake
Add a new test, for checking reference counting of qnull(). As part of the new file, move a previous reference counting change added in commit a861564 into a more logical place. Signed-off-by: Eric Blake --- v14: no change v13: no change v12: new patch --- tests/check-qnull.c | 60 +