On 8/22/19 7:42 PM, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > --- > tests/modules-test.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/modules-test.c b/tests/modules-test.c > index 3aef0e5..a8118e9 100644 > --- a/tests/modules-test.c > +++ b/tests/modules-test.c > @@ -4,7 +4,7 @@ > static void test_modules_load(const void *data) > { > QTestState *qts; > - const char **args = data; > + const char **args = (const char **)data; > > qts = qtest_init(NULL); > qtest_module_load(qts, args[0], args[1]);
Why did nobody notice this before? ... some additional words in the patch description would be fine. Anyway: Reviewed-by: Thomas Huth <th...@redhat.com>