tests/bios-tables-test: Compiler warning fix
authorDr. David Alan Gilbert <dgilbert@redhat.com>
Thu, 20 Jul 2017 16:35:36 +0000 (17:35 +0100)
committerMichael S. Tsirkin <mst@redhat.com>
Tue, 1 Aug 2017 21:13:26 +0000 (00:13 +0300)
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>
tests/bios-tables-test.c

index 63da978f0ba5fa73929757d72105fe7608391e05..564da45f659594fdffb29120d56bbdd42ff909b3 100644 (file)
@@ -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);