From: Andrey Repin <anrdaemon@yandex.ru>
To: Takashi Yano <takashi.yano@nifty.ne.jp>, cygwin@cygwin.com
Subject: Re: /dev/clipboard sometimes fails to set CF_UNICODETEXT data.
Date: Tue, 5 Jul 2022 17:46:52 +0300 [thread overview]
Message-ID: <199184658.20220705174652@yandex.ru> (raw)
In-Reply-To: <20220705174058.b4d8cb5ce54f7fac8700aaa9@nifty.ne.jp>
Greetings, Takashi Yano!
> On Tue, 5 Jul 2022 00:42:50 -0700
> Mark Geisert wrote:
>> Hi Takashi,
>>
>> Takashi Yano wrote:
>> > Hello,
>> >
>> > In one of my PCs,
>> > dd if=/dev/urandom count=10 | xxd > /dev/clipboard
>> > sometimes fails to set CF_UNICODETEXT data.
>> > As a result, pasting clipboard to notepad does not work.
>> >
>> > Even in the case, cygnativeformat data is set correctly.
>> > So, "cat /dev/clipboard" works.
>> >
>> > This problem depends on machine very much.
>> >
>> > My one PC with Xeon E3-1281 v3 CPU has a high probability of failure,
>> > however, another machine with Core i7-6700K CPU does not.
>> >
>> > I looked into this problem, and found OpenClipboard() for
>> > CF_UNICODETEXT fails. It seems that OpenClipboard() just
>> > after CloseClipboard() has high probability of failure.
>> >
>> > You can see the following test case immediately stops with error.
>> > Even with Core i7-6700K CPU machine above, the test case fails.
> Additional information:
> In the machine with Xeon E3-1281 v3 CPU, the test case fails
> in a several times of the loop, while it fails after 1000 to
> 3000 times of the loop in the machine with Core i7-6700K CPU.
> The probability of failure depends much on machine.
MFG… this might as well explain the issues OpenOffice having with clipboard
content. I'll forward your findings.
>> > #include <windows.h>
>> > #include <stdio.h>
>> >
>> > int main()
>> > {
>> > for (;;) {
>> > if (!OpenClipboard(0)) {
>> > printf("Open error.\n");
>> > break;
>> > }
>> > if (!EmptyClipboard()) {
>> > printf("Empty error.\n");
>> > break;
>> > }
>> > if (!CloseClipboard()) {
>> > printf("Cloes error.\n");
>> > break;
>> > }
>> > }
>> > return 0;
>> > }
>> >
>> > I also found the patch attached solves the issue.
>> >
>> > I would appreciate any suggestion.
>>
>> Would be neet if this patch wasn't needed, but, oh well. My only comment on the
>> patch would be to possibly call the new wrapper function MyOpenClipboard, or
>> CygOpenClipboard, to help keep the name paired with CloseClipboard.
> Thanks for the comment. IIUC, the camel case name should be
> used only for Win32 APIs related. However, this is the wrapper
> for Win32 API. So, it might be acceptable. In other places,
> some wrapper functions are named as camel case.
> Corinna, WDYT?
>> I suppose it might happen in the future that a faster processor will require the
>> loop count to be made larger. Once again, oh well.
> Maybe. In my most cases, OpenClipboard() successes after one
> or two times of retrying. So, I think max_retry = 10 is enough
> for the time being.
>> My 2c (adjusted for inflation),
> --
> Takashi Yano <takashi.yano@nifty.ne.jp>
--
With best regards,
Andrey Repin
Tuesday, July 5, 2022 17:44:56
Sorry for my terrible english...
prev parent reply other threads:[~2022-07-05 14:50 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-02 3:20 Takashi Yano
2022-07-05 7:42 ` Mark Geisert
2022-07-05 8:40 ` Takashi Yano
2022-07-05 11:42 ` Corinna Vinschen
2022-07-05 14:46 ` Andrey Repin [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=199184658.20220705174652@yandex.ru \
--to=anrdaemon@yandex.ru \
--cc=cygwin@cygwin.com \
--cc=takashi.yano@nifty.ne.jp \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).