diff options
| author | Dr. David Alan Gilbert <dgilbert@redhat.com> | 2017-07-20 17:35:36 +0100 |
|---|---|---|
| committer | Michael S. Tsirkin <mst@redhat.com> | 2017-08-02 00:13:26 +0300 |
| commit | 91c38e08af93d7d9df5636e4d7747ce3eb7ff1fc (patch) | |
| tree | d9e6cc7fec720fa2d385d8fc11e5777b13ec11dc /tests/bios-tables-test.c | |
| parent | 2f6b38d171b11c2cedd294e2e7e50e110702f5ef (diff) | |
| download | qemu-91c38e08af93d7d9df5636e4d7747ce3eb7ff1fc.zip qemu-91c38e08af93d7d9df5636e4d7747ce3eb7ff1fc.tar.gz qemu-91c38e08af93d7d9df5636e4d7747ce3eb7ff1fc.tar.bz2 | |
tests/bios-tables-test: Compiler warning fix
gcc 7.1.1 in fedora 26 moans about the:
tables = g_new0(uint32_t, tables_nr)
because it can't convince itself that tables_nr is positive.
This is fallout from g_assert_cmpint no longer necessarily being
no-return; replace it with a plain g_assert.
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Marcel Apfelbaum <marcel@redhat.com>
Diffstat (limited to 'tests/bios-tables-test.c')
| -rw-r--r-- | tests/bios-tables-test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c index 63da978..564da45 100644 --- a/tests/bios-tables-test.c +++ b/tests/bios-tables-test.c @@ -108,7 +108,7 @@ static void test_acpi_rsdt_table(test_data *data) /* compute the table entries in rsdt */ tables_nr = (rsdt_table->length - sizeof(AcpiRsdtDescriptorRev1)) / sizeof(uint32_t); - g_assert_cmpint(tables_nr, >, 0); + g_assert(tables_nr > 0); /* get the addresses of the tables pointed by rsdt */ tables = g_new0(uint32_t, tables_nr); |