[PATCH] user32/msgbox: Implement copying contents to clipboard

Alistair Leslie-Hughes leslie_alistair at hotmail.com
Thu Jan 23 01:11:28 CST 2020

Hi Vadim,

On 23/1/20 10:40 am, Vadim Zeitlin wrote:
>   Looking at this patch, I see that it's clearly better than mine in some
> aspects (e.g. it includes tests) but OTOH I'm not sure if using a hook for
> this is really a good idea, and it seems to have resulted in the problems
> as discussed at the bug above. There is also another small but annoying
> difference: the other patch doesn't take Ctrl-Ins into account, even though
> it works in the same way as Ctrl-C under native MSW.
When I developed this patch, I didn't know about Ctrl-Ins acting the 
same as Ctrl-C.

>   So what should/can I do now? Would it be worth to update my patch to
> combine it with the tests from the patch by Alistair Leslie-Hughes? Also,
> mostly for the future, should I have submitted this patch to Wine staging
> instead of here?
The only issue I know about getting these patches upstream is the tests. 
  They need to be updated so they will work on any locale not just English.

If you want to write the patch to allow "Ctrl-Ins", that extends the 
current patchset, I can put it in staging for you.  Then all that would 
be required is a new set of tests.


More information about the wine-devel mailing list