[PATCH 3/5] user32: Don't enable owner in DIALOG_DoDialogBox.

Jacek Caban jacek at codeweavers.com
Mon Apr 11 06:01:27 CDT 2016


On 04/11/16 01:06, Dmitry Timoshkov wrote:
> Jacek Caban <jacek at codeweavers.com> wrote:
>
>> On 04/06/16 15:24, Jacek Caban wrote:
>>> Could you share those tests (WIP is fine) or run my patches against them?
>>  Ping. Did you have a chance to run those tests?
> No. But since you already know that removing DF_OWNERENABLED handling
> is wrong it shouldn't be too hard to add appropriate tests.

No, as far as I know removing is right and I added tests that reach
EnableWindow() in every possible code path. I know only that you said
it's wrong with no prove to back that. I guess that you misinterpreted
results of your own tests and the fact that you claim you didn't bother
to run my patches against those tests tells pretty much all about
quality and intentions of your reviews.

Jacek



More information about the wine-devel mailing list