From 11721819a73be817ea6ee2bc721d3d59948967b1 Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Mon, 7 Nov 2011 10:59:07 +0000 Subject: [PATCH] Thin reindent code Drop indention level Add extra internal error. --- lib/metadata/thin_manip.c | 42 ++++++++++++++++++++------------------- 1 file changed, 22 insertions(+), 20 deletions(-) diff --git a/lib/metadata/thin_manip.c b/lib/metadata/thin_manip.c index 5d776429d..c23ec52a5 100644 --- a/lib/metadata/thin_manip.c +++ b/lib/metadata/thin_manip.c @@ -88,27 +88,29 @@ int attach_pool_message(struct lv_segment *seg, dm_thin_message_t type, struct lv_thin_message *tmsg; dm_list_iterate_items(tmsg, &seg->thin_messages) { - if (tmsg->type == type) { - switch (tmsg->type) { - case DM_THIN_MESSAGE_CREATE_SNAP: - case DM_THIN_MESSAGE_CREATE_THIN: - case DM_THIN_MESSAGE_TRIM: - if (tmsg->u.lv == lv) { - log_error("Message referring LV %s already queued for %s.", - tmsg->u.lv->name, seg->lv->name); - return 0; - } - break; - case DM_THIN_MESSAGE_DELETE: - if (tmsg->u.delete_id == delete_id) { - log_error("Delete of device %u already queued for %s.", - tmsg->u.delete_id, seg->lv->name); - return 0; - } - break; - default: - break; + if (tmsg->type != type) + continue; + + switch (tmsg->type) { + case DM_THIN_MESSAGE_CREATE_SNAP: + case DM_THIN_MESSAGE_CREATE_THIN: + case DM_THIN_MESSAGE_TRIM: + if (tmsg->u.lv == lv) { + log_error("Message referring LV %s already queued for %s.", + tmsg->u.lv->name, seg->lv->name); + return 0; } + break; + case DM_THIN_MESSAGE_DELETE: + if (tmsg->u.delete_id == delete_id) { + log_error("Delete of device %u already queued for %s.", + tmsg->u.delete_id, seg->lv->name); + return 0; + } + break; + default: + log_error(INTERNAL_ERROR "Unsupported message type %u.", tmsg->type); + return 0; } }