[PATCH] conhost: Fix wrapping when write_console finishes on line end.

Gabriel Ivăncescu gabrielopcode at gmail.com
Wed Mar 31 10:24:08 CDT 2021


On 31/03/2021 15:27, Jacek Caban wrote:
> On 3/31/21 1:42 PM, Gabriel Ivăncescu wrote:
>> This makes it pass on Windows, since it always outputs a newline 
>> immediately and never a backspace. In fact, removing the condition at 
>> all will *still* pass on Windows. So this condition is there for Wine? 
>> Is there a reason for it? And what should I keep in mind here to solve 
>> this "properly"?
>>
>> If you remove the condition, of course wine will fail the tests, but 
>> Windows won't. I really don't understand it.
> 
> Windows behavior isn't consistent between versions here. When I wrote 
> those tests it was a relatively fresh feature and its output changed 
> between Windows updates. Tests were written in a way that would accept 
> all of them. Hopefully things have matured now and it's good that 
> behavior of recent versions is better. We should follow that and 
> probably even mark those '\b' code paths as broken.
> 
> Jacek

Ok so it looks like it does add a \b sometimes, but not during the tests 
I added because I added them after setting the output mode. I'm guessing 
the default output mode has some "delayed newline" flag and that differs 
between versions. I'll have to look into it.



More information about the wine-devel mailing list