From 45c06025da574a6ad1491af32c842d53afb11009 Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Fri, 17 May 2024 16:51:11 +0200 Subject: [PATCH] cleanup: some typos Collection of some typos or invalid uppercase or doublespace cases. --- lib/config/config.c | 4 ++-- lib/format_text/format-text.c | 2 +- lib/lvmpolld/lvmpolld-client.c | 10 +++++----- lib/metadata/lv.c | 4 ++-- lib/metadata/lv_manip.c | 4 ++-- lib/metadata/thin_manip.c | 2 +- tools/args.h | 4 ++-- tools/command.c | 2 +- 8 files changed, 16 insertions(+), 16 deletions(-) diff --git a/lib/config/config.c b/lib/config/config.c index 5dfdee05c..653015874 100644 --- a/lib/config/config.c +++ b/lib/config/config.c @@ -941,7 +941,7 @@ static int _check_value_differs_from_default(struct cft_check_handle *handle, diff = (v_def && (v_def->type != DM_CFG_EMPTY_ARRAY)); break; default: - log_error(INTERNAL_ERROR "inconsistent state reached in _check_value_differs_from_default"); + log_error(INTERNAL_ERROR "Inconsistent state reached in _check_value_differs_from_default."); return 0; } } @@ -2333,7 +2333,7 @@ int load_profile(struct cmd_context *cmd, struct profile *profile) { static char profile_path[PATH_MAX]; if (critical_section()) { - log_error(INTERNAL_ERROR "trying to load profile %s " + log_error(INTERNAL_ERROR "Trying to load profile %s " "in critical section.", profile->name); return 0; } diff --git a/lib/format_text/format-text.c b/lib/format_text/format-text.c index 1fcb51875..e8bab535e 100644 --- a/lib/format_text/format-text.c +++ b/lib/format_text/format-text.c @@ -2266,7 +2266,7 @@ static int _text_pv_add_metadata_area(const struct format_type *fmt, if (fid_get_mda_indexed(fid, pvid, ID_LEN, mda_index)) { if (!_text_pv_remove_metadata_area(fmt, pv, mda_index)) { - log_error(INTERNAL_ERROR "metadata area with index %u already " + log_error(INTERNAL_ERROR "Metadata area with index %u already " "exists on PV %s and removal failed.", mda_index, pv_dev_name(pv)); return 0; diff --git a/lib/lvmpolld/lvmpolld-client.c b/lib/lvmpolld/lvmpolld-client.c index 015cefafb..cf2d25766 100644 --- a/lib/lvmpolld/lvmpolld-client.c +++ b/lib/lvmpolld/lvmpolld-client.c @@ -284,17 +284,17 @@ int lvmpolld_poll_init(const struct cmd_context *cmd, const struct poll_operatio int r = 0; if (!id->uuid) { - log_error(INTERNAL_ERROR "Use of lvmpolld requires uuid set"); + log_error(INTERNAL_ERROR "Use of lvmpolld requires uuid set."); return 0; } if (!id->vg_name) { - log_error(INTERNAL_ERROR "Use of lvmpolld requires vgname set"); + log_error(INTERNAL_ERROR "Use of lvmpolld requires vgname set."); return 0; } if (!id->lv_name) { - log_error(INTERNAL_ERROR "Use of lvmpolld requires lvname set"); + log_error(INTERNAL_ERROR "Use of lvmpolld requires lvname set."); return 0; } @@ -321,7 +321,7 @@ int lvmpolld_poll_init(const struct cmd_context *cmd, const struct poll_operatio log_error(INTERNAL_ERROR "Unsupported poll operation."); } } else - log_error(INTERNAL_ERROR "Unsupported poll operation"); + log_error(INTERNAL_ERROR "Unsupported poll operation."); return r; } @@ -334,7 +334,7 @@ int lvmpolld_request_info(const struct poll_operation_id *id, const struct daemo *finished = 1; if (!id->uuid) { - log_error(INTERNAL_ERROR "use of lvmpolld requires uuid being set"); + log_error(INTERNAL_ERROR "Use of lvmpolld requires uuid being set."); return 0; } diff --git a/lib/metadata/lv.c b/lib/metadata/lv.c index d7d8a78a4..3e8e52e97 100644 --- a/lib/metadata/lv.c +++ b/lib/metadata/lv.c @@ -92,13 +92,13 @@ static struct dm_list *_format_pvsegs(struct dm_pool *mem, const struct lv_segme if (dm_snprintf(extent_str, sizeof(extent_str), ":%" PRIu32 "-%" PRIu32, extent, extent + seg_len - 1) < 0) { - log_error("_format_pvseggs: extent range dm_snprintf failed"); + log_error("_format_pvsegs: extent range dm_snprintf failed."); goto bad; } } else { if (dm_snprintf(extent_str, sizeof(extent_str), "(%" PRIu32 ")", extent) < 0) { - log_error("_format_pvsegs: extent number dm_snprintf failed"); + log_error("_format_pvsegs: extent number dm_snprintf failed."); goto bad; } } diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c index bec363ef8..18331fae8 100644 --- a/lib/metadata/lv_manip.c +++ b/lib/metadata/lv_manip.c @@ -4598,7 +4598,7 @@ int lv_extend(struct logical_volume *lv, "is not locally active.", display_lvname(lv)); /* FIXME Support --force */ if (yes_no_prompt("Do full resync of extended " - "portion of %s? [y/n]: ", + "portion of %s? [y/n]: ", display_lvname(lv)) == 'n') { r = 0; goto_out; @@ -8418,7 +8418,7 @@ struct logical_volume *insert_layer_for_lv(struct cmd_context *cmd, /* Temporary tags for activation of the transient LV */ dm_list_iterate_items(sl, &lv_where->tags) if (!str_list_add(cmd->mem, &layer_lv->tags, sl->str)) { - log_error("Aborting. Unable to tag" + log_error("Aborting. Unable to tag" " transient mirror layer."); return NULL; } diff --git a/lib/metadata/thin_manip.c b/lib/metadata/thin_manip.c index 90544bb55..6f26d02e5 100644 --- a/lib/metadata/thin_manip.c +++ b/lib/metadata/thin_manip.c @@ -128,7 +128,7 @@ int detach_thin_external_origin(struct lv_segment *seg) { if (seg->external_lv) { if (!lv_is_external_origin(seg->external_lv)) { - log_error(INTERNAL_ERROR "Inconsitent external origin."); + log_error(INTERNAL_ERROR "Inconsistent external origin."); return 0; } diff --git a/tools/args.h b/tools/args.h index e456a572b..50201ee09 100644 --- a/tools/args.h +++ b/tools/args.h @@ -880,7 +880,7 @@ arg(systemid_ARG, '\0', "systemid", string_VAL, 0, 0, "the command, leaving the VG inaccessible to the host.\n" "See \\fBlvmsystemid\\fP(7) for more information.\n" "#vgchange\n" - "Changes the system ID of the VG. Using this option requires caution\n" + "Changes the system ID of the VG. Using this option requires caution\n" "because the VG may become foreign to the host running the command,\n" "leaving the host unable to access it.\n" "See \\fBlvmsystemid\\fP(7) for more information.\n") @@ -1100,7 +1100,7 @@ arg(activate_ARG, 'a', "activate", activation_VAL, 0, 0, "The location and name of the underlying device node may depend on\n" "the distribution, configuration (e.g. udev), or release version.\n" "\\fBay\\fP specifies autoactivation, which is used by system-generated\n" - "activation commands. By default, LVs are autoactivated.\n" + "activation commands. By default, LVs are autoactivated.\n" "An autoactivation property can be set on a VG or LV to disable autoactivation,\n" "see --setautoactivation y|n in vgchange, lvchange, vgcreate, and lvcreate.\n" "Display the property with vgs or lvs \"-o autoactivation\".\n" diff --git a/tools/command.c b/tools/command.c index 6e115690b..8740596b9 100644 --- a/tools/command.c +++ b/tools/command.c @@ -665,7 +665,7 @@ static void _add_oo_definition_line(const char *name, const char *line) start = strchr(line, ':') + 2; if (!(oo->line = strdup(start))) { - log_error("Filler to duplicate line %s.", start); + log_error("Failed to duplicate line %s.", start); return; } }