From: "Zebediah Figura (she/her)" <zfigura@codeweavers.com>
Subject: Re: [PATCH v2 4/6] robocopy: Add basic output
Message-Id: <814799cb-af77-53f6-22ca-10157a538e25@codeweavers.com>
Date: Fri, 17 Sep 2021 00:45:20 -0500
In-Reply-To: <20210915215700.424685-4-others.meder@gmail.com>
References: <20210915215700.424685-1-others.meder@gmail.com> <20210915215700.424685-4-others.meder@gmail.com>

On 9/15/21 16:56, Florian Eder wrote:
> +static void print_header(void)
> +{
> +    UINT i;
> +
> +    output_message(format_string(STRING_HEADER));
> +
> +    if (!options.source) output_message(format_string(STRING_SOURCE), L"-");
> +    else output_message(format_string(STRING_SOURCE), strip_path_prefix(options.source));
> +
> +    if (!options.destination) output_message(format_string(STRING_DESTINATION), L"-");
> +    else output_message(format_string(STRING_DESTINATION), strip_path_prefix(options.destination));
> +
> +    output_message(format_string(STRING_FILES), options.files->array[0]);
> +    for (i = 1; i < options.files->size; i++)
> +        output_message(L"                 %1\n", options.files->array[i]);

Sorry if I was unclear. What I was trying to propose was to print the 
first file on the line *after* the "Files:" header, rather than on the 
same line. Then you can align all the paths the same arbitrary amount.

> +}
> +