From: "Darrick J. Wong" <djwong@kernel.org>
To: Andreas Dilger <adilger@dilger.ca>
Cc: tytso@mit.edu, linux-ext4@vger.kernel.org
Subject: Re: [PATCH] e2fsck: map PROMPT_* values to prompt messages
Date: Wed, 8 Dec 2021 08:42:38 -0800 [thread overview]
Message-ID: <20211208164238.GA69182@magnolia> (raw)
In-Reply-To: <20211208075112.85649-1-adilger@dilger.ca>
On Wed, Dec 08, 2021 at 12:51:12AM -0700, Andreas Dilger wrote:
> It isn't totally clear when searching the code for PROMPT_*
> constants from problem codes where these messages come from.
> Similarly, there isn't a direct mapping from the prompt string
> to the constant.
>
> Add comments that make this mapping more clear.
>
> Signed-off-by: Andreas Dilger <adilger@dilger.ca>
> ---
> e2fsck/problem.c | 46 +++++++++++++++++++++++-----------------------
> 1 file changed, 23 insertions(+), 23 deletions(-)
>
> diff --git a/e2fsck/problem.c b/e2fsck/problem.c
> index 757b5d56..2d02468c 100644
> --- a/e2fsck/problem.c
> +++ b/e2fsck/problem.c
> @@ -50,29 +50,29 @@
> * to fix a problem.
> */
> static const char *prompt[] = {
> - N_("(no prompt)"), /* 0 */
> - N_("Fix"), /* 1 */
> - N_("Clear"), /* 2 */
> - N_("Relocate"), /* 3 */
> - N_("Allocate"), /* 4 */
> - N_("Expand"), /* 5 */
> - N_("Connect to /lost+found"), /* 6 */
> - N_("Create"), /* 7 */
> - N_("Salvage"), /* 8 */
> - N_("Truncate"), /* 9 */
> - N_("Clear inode"), /* 10 */
> - N_("Abort"), /* 11 */
> - N_("Split"), /* 12 */
> - N_("Continue"), /* 13 */
> - N_("Clone multiply-claimed blocks"), /* 14 */
> - N_("Delete file"), /* 15 */
> - N_("Suppress messages"),/* 16 */
> - N_("Unlink"), /* 17 */
> - N_("Clear HTree index"),/* 18 */
> - N_("Recreate"), /* 19 */
> - N_("Optimize"), /* 20 */
> - N_("Clear flag"), /* 21 */
> - "", /* 22 */
> + N_("(no prompt)"), /* PROMPT_NONE = 0 */
Why not make it even clearer and mismerge proof:
static const char *prompt[] = {
[0] = N_("(no prompt")), /* null value test */
[PROMPT_FIX] = N_("Fix"), /* 1 */
[PROMPT_CLEAR] = N_("Clear"), /* 2 */
...
};
--D
> + N_("Fix"), /* PROMPT_FIX = 1 */
> + N_("Clear"), /* PROMPT_CLEAR = 2 */
> + N_("Relocate"), /* PROMPT_RELOCATE = 3 */
> + N_("Allocate"), /* PROMPT_CREATE = 4 */
> + N_("Expand"), /* PROMPT_EXPAND = 5 */
> + N_("Connect to /lost+found"), /* PROMPT_CONNECT = 6 */
> + N_("Create"), /* PROMPT_CREATE = 7 */
> + N_("Salvage"), /* PROMPT_SALVAGE = 8 */
> + N_("Truncate"), /* PROMPT_TRUNCATE = 9 */
> + N_("Clear inode"), /* PROMPT_CLEAR_INODE = 10 */
> + N_("Abort"), /* PROMPT_ABORT = 11 */
> + N_("Split"), /* PROMPT_SPLIT = 12 */
> + N_("Continue"), /* PROMPT_CONTINUE = 13 */
> + N_("Clone multiply-claimed blocks"), /* PROMPT_CLONE = 14 */
> + N_("Delete file"), /* PROMPT_DELETE = 15 */
> + N_("Suppress messages"), /* PROMPT_SUPPRESS = 16 */
> + N_("Unlink"), /* PROMPT_UNLINK = 17 */
> + N_("Clear HTree index"), /* PROMPT_CLEAR_HTREE = 18 */
> + N_("Recreate"), /* PROMPT_RECREATE = 19 */
> + N_("Optimize"), /* PROMPT_OPTIMIZE = 20 */
> + N_("Clear flag"), /* PROMPT_CLEAR_FLAG = 21 */
> + "", /* PROMPT_NULL = 22 */
> };
>
> /*
> --
> 2.25.1
>
next prev parent reply other threads:[~2021-12-08 16:42 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-08 7:51 [PATCH] e2fsck: map PROMPT_* values to prompt messages Andreas Dilger
2021-12-08 16:42 ` Darrick J. Wong [this message]
2021-12-09 21:55 ` Andreas Dilger
2021-12-11 0:51 ` Darrick J. Wong
2022-02-28 23:58 ` Andreas Dilger
2022-05-03 3:31 ` Theodore Ts'o
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211208164238.GA69182@magnolia \
--to=djwong@kernel.org \
--cc=adilger@dilger.ca \
--cc=linux-ext4@vger.kernel.org \
--cc=tytso@mit.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).