This patch uses printbuf_indent_add_nextline() to set a consistent indentation level for error messages of invalid compression.
In my previous patch [1], the newline is added by using '\n' in the argument of prt_str(). This patch replaces '\n' with prt_newline() to make indentation level work correctly. [1] Link: https://lore.kernel.org/[email protected] Signed-off-by: Integral <[email protected]> --- fs/bcachefs/compress.c | 12 ++++++++---- fs/bcachefs/opts.c | 1 + 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/fs/bcachefs/compress.c b/fs/bcachefs/compress.c index d68c3c7896a3..a107aa88a875 100644 --- a/fs/bcachefs/compress.c +++ b/fs/bcachefs/compress.c @@ -713,8 +713,10 @@ int bch2_opt_compression_parse(struct bch_fs *c, const char *_val, u64 *res, level_str = p; ret = match_string(bch2_compression_opts, -1, type_str); - if (ret < 0 && err) - prt_str(err, "invalid compression type\n"); + if (ret < 0 && err) { + prt_str(err, "invalid compression type"); + prt_newline(err); + } if (ret < 0) goto err; @@ -728,8 +730,10 @@ int bch2_opt_compression_parse(struct bch_fs *c, const char *_val, u64 *res, ret = -EINVAL; if (!ret && level > 15) ret = -EINVAL; - if (ret < 0 && err) - prt_str(err, "invalid compression level\n"); + if (ret < 0 && err) { + prt_str(err, "invalid compression level"); + prt_newline(err); + } if (ret < 0) goto err; diff --git a/fs/bcachefs/opts.c b/fs/bcachefs/opts.c index cd7b0cd293c7..f35fde896253 100644 --- a/fs/bcachefs/opts.c +++ b/fs/bcachefs/opts.c @@ -365,6 +365,7 @@ int bch2_opt_parse(struct bch_fs *c, struct printbuf *err) { ssize_t ret; + printbuf_indent_add_nextline(err, 2); switch (opt->type) { case BCH_OPT_BOOL: base-commit: 4d37602fceff942694069cf33cc55863277bd1c2 -- 2.49.0
