[PATCH] ntdll: Truncate and ellipsize debug output if it's too long.

Alexandre Julliard julliard at winehq.org
Fri Aug 27 14:29:47 CDT 2021


"Esme Povirk (she/they)" <esme at codeweavers.com> writes:

> Is there anything I can do to help move this along?
>
> append_output only interacts with one other function, and this doesn't
> change the interface to it, so from a correctness perspective, I think
> this should be easy to verify.

It doesn't look quite correct, you'd need to flush the buffer otherwise
you'll potentially lose even more output.

> From a design perspective, I don't know if this is right, but I
> couldn't think of a better approach. Exposing the buffer size to
> callers seems not ideal to me. Is there an alternative I'm missing?

I'd suggest using debugstr_a/w like everywhere else. If you have to
print an unbounded number of arguments, you can print them one per line.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list