On Tue, Feb 16, 2021 at 4:28 PM Sidong Yang <[email protected]> wrote:
>
> Remove a code that inserting new line in fmt_end() for text mode.
> Old code made a failure in fstest btrfs/006.
>
> Signed-off-by: Sidong Yang <[email protected]>
> ---
> Hi, I've just read mail that Filipe written that some failure about fstest.
> I'm worried about this patch makes other problem. So make it RFC. Thanks.
Fixes the test failure indeed, thanks!
> ---
> common/format-output.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/common/format-output.c b/common/format-output.c
> index f5b12548..96e0dfef 100644
> --- a/common/format-output.c
> +++ b/common/format-output.c
> @@ -124,9 +124,7 @@ void fmt_end(struct format_ctx *fctx)
>
> /* Close, no continuation to print */
>
> - if (bconf.output_format & CMD_FORMAT_TEXT)
> - putchar('\n');
> - else if (bconf.output_format & CMD_FORMAT_JSON) {
> + if (bconf.output_format & CMD_FORMAT_JSON) {
> fmt_dec_depth(fctx);
> fmt_separator(fctx);
> printf("}\n");
> --
> 2.25.1
>
--
Filipe David Manana,
“Whether you think you can, or you think you can't — you're right.”
