On 09/11/2017 01:20 PM, Eric Blake wrote: > Rather than have two similar shutdown functions, where one requires > the use of global_qtest in the header, it is better to have a single > shutdown function that still takes care of cleaning up global_qtest > if it is set. All callers are updated. > > Signed-off-by: Eric Blake <ebl...@redhat.com> > --- > tests/libqtest.h | 11 ----------- > tests/libqtest.c | 6 +++++- [...] > tests/fdc-test.c | 2 +- [...] > tests/ide-test.c | 2 +-
Reviewed-by: John Snow <js...@redhat.com> The rest: "ACK." Change looks sane and I assume the rest of the patch is fairly straightforward mechanical changes. Everything still compiles and runs successfully.