[Libosinfo] [libosinfo PATCH v2 2/6] media: Use better naming for goto labels

Fabiano Fidêncio fidencio at redhat.com
Thu Dec 20 07:46:43 UTC 2018


Let's replace EXIT for cleanup and ON_ERROR for error and follow a
normal naming convention for the labels.

Signed-off-by: Fabiano Fidêncio <fidencio at redhat.com>
---
 osinfo/osinfo_media.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/osinfo/osinfo_media.c b/osinfo/osinfo_media.c
index fd5e622..1ee1fa5 100644
--- a/osinfo/osinfo_media.c
+++ b/osinfo/osinfo_media.c
@@ -745,14 +745,14 @@ static void on_svd_read(GObject *source,
     if (ret < 0) {
         g_prefix_error(&error,
                        _("Failed to read supplementary volume descriptor: "));
-        goto EXIT;
+        goto cleanup;
     }
     if (ret == 0) {
         g_set_error(&error,
                     OSINFO_MEDIA_ERROR,
                     OSINFO_MEDIA_ERROR_NO_SVD,
                     _("Supplementary volume descriptor was truncated"));
-        goto EXIT;
+        goto cleanup;
     }
 
     data->offset += ret;
@@ -777,7 +777,7 @@ static void on_svd_read(GObject *source,
                     OSINFO_MEDIA_ERROR_NOT_BOOTABLE,
                     _("Install media is not bootable"));
 
-        goto EXIT;
+        goto cleanup;
     }
 
     uri = g_file_get_uri(data->file);
@@ -812,7 +812,7 @@ static void on_svd_read(GObject *source,
                                   OSINFO_MEDIA_PROP_VOLUME_SIZE,
                                   vol_size);
 
-EXIT:
+ cleanup:
     if (error != NULL)
         g_task_return_error(data->res, error);
     else
@@ -838,14 +838,14 @@ static void on_pvd_read(GObject *source,
                                      &error);
     if (ret < 0) {
         g_prefix_error(&error, _("Failed to read primary volume descriptor: "));
-        goto ON_ERROR;
+        goto error;
     }
     if (ret == 0) {
         g_set_error(&error,
                     OSINFO_MEDIA_ERROR,
                     OSINFO_MEDIA_ERROR_NO_PVD,
                     _("Primary volume descriptor was truncated"));
-        goto ON_ERROR;
+        goto error;
     }
 
     data->offset += ret;
@@ -878,7 +878,7 @@ static void on_pvd_read(GObject *source,
                     OSINFO_MEDIA_ERROR_INSUFFICIENT_METADATA,
                     _("Insufficient metadata on installation media"));
 
-        goto ON_ERROR;
+        goto error;
     }
 
     data->offset = 0;
@@ -893,7 +893,7 @@ static void on_pvd_read(GObject *source,
                               data);
     return;
 
-ON_ERROR:
+ error:
     g_object_unref(stream);
     g_task_return_error(data->res, error);
     create_from_location_async_data_free(data);
-- 
2.19.1




More information about the Libosinfo mailing list