[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH v3 1/3] hw/block/fdc: Replace error_setg(&error_abor
From: |
Philippe Mathieu-Daudé |
Subject: |
[Qemu-block] [PATCH v3 1/3] hw/block/fdc: Replace error_setg(&error_abort) by assert() |
Date: |
Thu, 21 Jun 2018 08:40:13 -0300 |
Use assert() instead of error_setg(&error_abort),
as suggested by the "qapi/error.h" documentation:
Please don't error_setg(&error_fatal, ...), use error_report() and
exit(), because that's more obvious.
Likewise, don't error_setg(&error_abort, ...), use assert().
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Acked-by: John Snow <address@hidden>
---
hw/block/fdc.c | 9 +--------
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git a/hw/block/fdc.c b/hw/block/fdc.c
index cd29e27d8f..7c1c57f57f 100644
--- a/hw/block/fdc.c
+++ b/hw/block/fdc.c
@@ -396,16 +396,9 @@ static int pick_geometry(FDrive *drv)
nb_sectors,
FloppyDriveType_str(parse->drive));
}
+ assert(type_match != -1 && "misconfigured fd_format");
match = type_match;
}
-
- /* No match of any kind found -- fd_format is misconfigured, abort. */
- if (match == -1) {
- error_setg(&error_abort, "No candidate geometries present in table "
- " for floppy drive type '%s'",
- FloppyDriveType_str(drv->drive));
- }
-
parse = &(fd_formats[match]);
out:
--
2.18.0.rc2
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-block] [PATCH v3 1/3] hw/block/fdc: Replace error_setg(&error_abort) by assert(),
Philippe Mathieu-Daudé <=