On 04/11/16 01:06, Dmitry Timoshkov wrote:
Jacek Caban jacek@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