<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 23, 2020 at 3:30 PM Gabriel Ivăncescu <<a href="mailto:gabrielopcode@gmail.com">gabrielopcode@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 23/01/2020 09:11, Alistair Leslie-Hughes wrote:<br>
> Hi Vadim,<br>
> <br>
> On 23/1/20 10:40 am, Vadim Zeitlin wrote:<br>
>>    Looking at this patch, I see that it's clearly better than mine in some<br>
>> aspects (e.g. it includes tests) but OTOH I'm not sure if using a hook for<br>
>> this is really a good idea, and it seems to have resulted in the problems<br>
>> as discussed at the bug above. There is also another small but annoying<br>
>> difference: the other patch doesn't take Ctrl-Ins into account, even though<br>
>> it works in the same way as Ctrl-C under native MSW.<br>
> When I developed this patch, I didn't know about Ctrl-Ins acting the<br>
> same as Ctrl-C.<br>
> <br>
>><br>
>>    So what should/can I do now? Would it be worth to update my patch to<br>
>> combine it with the tests from the patch by Alistair Leslie-Hughes? Also,<br>
>> mostly for the future, should I have submitted this patch to Wine staging<br>
>> instead of here?<br>
> The only issue I know about getting these patches upstream is the tests.<br>
>    They need to be updated so they will work on any locale not just English.<br>
> <br>
> If you want to write the patch to allow "Ctrl-Ins", that extends the<br>
> current patchset, I can put it in staging for you.  Then all that would<br>
> be required is a new set of tests.<br>
> <br>
> Regards<br>
> Alistair.<br>
> <br>
<br>
Also FWIW, I haven't found any problems with the staging patch for a <br>
long time ever since it has been updated to use a thread local hook <br>
instead of global, so that's not an issue.<br></blockquote><div><br></div><div>Both hook and changing general dialog code just for that is not great. Is there really no way to intercept this? Does actual window procedure still receive WM_KEY*?</div></div></div>