Alexandre Julliard julliard@winehq.org wrote:
The tests clearly show that this commit is wrong. If there is an application or a test case which fails because of this, it would be helpful to at least mention it.
It fixes the test for .bss protection, in ntdll:info.c.
I don't see that test failing before that commit or after the revert.
It fails with PE binaries but not Unix ones.
So the test doesn't really exist? Why do you think that a failure you see is a loader thing and not something that happens at run-time? Please add a real test and I'll have a look.