DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 7BA5B385828E
Date: Tue, 5 Jul 2022 13:42:45 +0200
From: Corinna Vinschen <corinna-cygwin@cygwin.com>
To: cygwin@cygwin.com
Subject: Re: /dev/clipboard sometimes fails to set CF_UNICODETEXT data.
Message-ID: <YsQjtUwkJXJBj7xV@calimero.vinschen.de>
Mail-Followup-To: cygwin@cygwin.com
References: <20220702122056.39a6c5e49c637d028694412e@nifty.ne.jp>
 <62975088-7bc3-7dde-d08c-3972126304f1@maxrnd.com>
 <20220705174058.b4d8cb5ce54f7fac8700aaa9@nifty.ne.jp>
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <20220705174058.b4d8cb5ce54f7fac8700aaa9@nifty.ne.jp>
Precedence: list
Reply-To: cygwin@cygwin.com
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cygwin-bounces+archive-cygwin=delorie.com@cygwin.com
Sender: "Cygwin" <cygwin-bounces+archive-cygwin=delorie.com@cygwin.com>

On Jul  5 17:40, Takashi Yano wrote:
> On Tue, 5 Jul 2022 00:42:50 -0700
> Mark Geisert wrote:
> > Takashi Yano wrote:
> > > 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?

What about just adding a static inline wrapper called close_clipboard
for symmetry?


Corinna

-- 
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple