Re: RFR: 8281384: Random chars on paste from Windows clipboard [v8]

2025-04-05 Thread Kevin Rushforth
On Fri, 28 Mar 2025 15:10:46 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v9]

2025-04-05 Thread Oliver Schmidtmer
> Windows programs may reuse a clipboard buffer that is larger than the new > content. In this case de NUL terminator is not at the end of the buffer, but > within it. > The current implementation copys the whole buffer into a text field, > including the NUL terminator and the remaining chars.

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v9]

2025-04-01 Thread duke
On Mon, 31 Mar 2025 17:48:50 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v9]

2025-04-01 Thread Lukasz Kostyra
On Mon, 31 Mar 2025 17:48:50 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v9]

2025-03-31 Thread Kevin Rushforth
On Mon, 31 Mar 2025 17:48:50 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v8]

2025-03-28 Thread Oliver Schmidtmer
> Windows programs may reuse a clipboard buffer that is larger than the new > content. In this case de NUL terminator is not at the end of the buffer, but > within it. > The current implementation copys the whole buffer into a text field, > including the NUL terminator and the remaining chars.

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v7]

2025-03-28 Thread Lukasz Kostyra
On Thu, 27 Mar 2025 13:14:27 GMT, Oliver Schmidtmer wrote: >> modules/javafx.graphics/src/main/native-glass/win/GlassClipboard.cpp line >> 207: >> >>> 205: { >>> 206: addPair(GLASS_TEXT_PLAIN, CF_UNICODETEXT); >>> 207: addPair(GLASS_TEXT_PLAIN_LOCALE, CF_UNICODETEXT); >> >> Seems like

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v7]

2025-03-27 Thread Oliver Schmidtmer
On Thu, 27 Mar 2025 10:06:02 GMT, Lukasz Kostyra wrote: >> Oliver Schmidtmer has updated the pull request incrementally with one >> additional commit since the last revision: >> >> readding flavors with changed mapping > > modules/javafx.graphics/src/main/native-glass/win/GlassClipboard.cpp l

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v7]

2025-03-27 Thread Lukasz Kostyra
On Wed, 26 Mar 2025 20:20:36 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v7]

2025-03-26 Thread Oliver Schmidtmer
> Windows programs may reuse a clipboard buffer that is larger than the new > content. In this case de NUL terminator is not at the end of the buffer, but > within it. > The current implementation copys the whole buffer into a text field, > including the NUL terminator and the remaining chars.

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v6]

2025-03-26 Thread Lukasz Kostyra
On Tue, 25 Mar 2025 18:39:08 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v6]

2025-03-25 Thread Oliver Schmidtmer
> Windows programs may reuse a clipboard buffer that is larger than the new > content. In this case de NUL terminator is not at the end of the buffer, but > within it. > The current implementation copys the whole buffer into a text field, > including the NUL terminator and the remaining chars.

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v4]

2025-03-25 Thread Kevin Rushforth
On Tue, 25 Mar 2025 09:53:10 GMT, Lukasz Kostyra wrote: >> As @mstr2 said, I think the data is always converted to CF_UNICODETEXT. If >> it wasn't there should have been more visible problems, as the current code >> always assumes wide chars. >> >> So I just need to know how much we should cha

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v4]

2025-03-25 Thread Lukasz Kostyra
On Tue, 25 Mar 2025 09:36:46 GMT, Oliver Schmidtmer wrote: >> Could be, but in any case, the way it's impemented right now doesn't seem to >> be right. We should only assume 2-byte characters if what's being pulled >> from the clipboard is actually `CF_UNICODETEXT`. > > As @mstr2 said, I think

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v4]

2025-03-25 Thread Oliver Schmidtmer
On Fri, 28 Feb 2025 16:48:39 GMT, Michael Strauß wrote: >> As the existing code was already pretty optimistic about 2 byte chars, is it >> possible that is already handled somewhere else? >> I'm not sure whether this is done explicitly somewhere or if CF_UNICODETEXT >> is just tried first. > >

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v4]

2025-03-24 Thread Lukasz Kostyra
On Fri, 28 Feb 2025 16:48:39 GMT, Michael Strauß wrote: >> As the existing code was already pretty optimistic about 2 byte chars, is it >> possible that is already handled somewhere else? >> I'm not sure whether this is done explicitly somewhere or if CF_UNICODETEXT >> is just tried first. > >

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v4]

2025-02-28 Thread Michael Strauß
On Thu, 27 Feb 2025 09:59:04 GMT, Oliver Schmidtmer wrote: >> modules/javafx.graphics/src/main/native-glass/win/GlassClipboard.cpp line >> 406: >> >>> 404: cdata = 0; >>> 405: } >>> 406: } else if(CF_TEXT == cf || CF_UNICODETEXT == cf){ >> >> Instead of doi

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v4]

2025-02-27 Thread Oliver Schmidtmer
On Thu, 27 Feb 2025 03:40:47 GMT, Michael Strauß wrote: >> Oliver Schmidtmer has updated the pull request incrementally with one >> additional commit since the last revision: >> >> cleanup > > modules/javafx.graphics/src/main/native-glass/win/GlassClipboard.cpp line 406: > >> 404:

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v5]

2025-02-27 Thread Oliver Schmidtmer
> Windows programs may reuse a clipboard buffer that is larger than the new > content. In this case de NUL terminator is not at the end of the buffer, but > within it. > The current implementation copys the whole buffer into a text field, > including the NUL terminator and the remaining chars.

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v4]

2025-02-26 Thread Michael Strauß
On Wed, 26 Feb 2025 18:28:28 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v3]

2025-02-26 Thread Oliver Schmidtmer
> Windows programs may reuse a clipboard buffer that is larger than the new > content. In this case de NUL terminator is not at the end of the buffer, but > within it. > The current implementation copys the whole buffer into a text field, > including the NUL terminator and the remaining chars.

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v4]

2025-02-26 Thread Oliver Schmidtmer
> Windows programs may reuse a clipboard buffer that is larger than the new > content. In this case de NUL terminator is not at the end of the buffer, but > within it. > The current implementation copys the whole buffer into a text field, > including the NUL terminator and the remaining chars.

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v2]

2025-02-26 Thread Kevin Rushforth
On Tue, 25 Feb 2025 15:24:39 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v2]

2025-02-26 Thread John Hendrikx
On Tue, 25 Feb 2025 15:24:39 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v2]

2025-02-26 Thread Kevin Rushforth
On Wed, 26 Feb 2025 13:22:16 GMT, Kevin Rushforth wrote: > I can't reproduce the original bug on Windows 11. Actually, I didn't run the test correctly earlier. I can reproduce it locally, so I'll be able to verify the fix (once we resolve the outstanding questions). - PR Comment:

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v2]

2025-02-26 Thread Kevin Rushforth
On Tue, 25 Feb 2025 15:24:39 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v2]

2025-02-26 Thread Lukasz Kostyra
On Tue, 25 Feb 2025 15:24:39 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v2]

2025-02-26 Thread Kevin Rushforth
On Tue, 25 Feb 2025 15:24:39 GMT, Oliver Schmidtmer wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a tex

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v2]

2025-02-26 Thread Kevin Rushforth
On Tue, 25 Feb 2025 15:21:22 GMT, Oliver Schmidtmer wrote: >> modules/javafx.graphics/src/main/java/com/sun/glass/ui/win/WinSystemClipboard.java >> line 258: >> >>> 256: int nullTerm = data.length - 2; >>> 257: for (int i = 0; i < data.length; i += 2) {

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v2]

2025-02-25 Thread Oliver Schmidtmer
On Tue, 25 Feb 2025 14:36:59 GMT, Kevin Rushforth wrote: >> Oliver Schmidtmer has updated the pull request incrementally with one >> additional commit since the last revision: >> >> check both UTF16 bytes > > modules/javafx.graphics/src/main/java/com/sun/glass/ui/win/WinSystemClipboard.java >

Re: RFR: 8281384: Random chars on paste from Windows clipboard [v2]

2025-02-25 Thread Oliver Schmidtmer
> Windows programs may reuse a clipboard buffer that is larger than the new > content. In this case de NUL terminator is not at the end of the buffer, but > within it. > The current implementation copys the whole buffer into a text field, > including the NUL terminator and the remaining chars.

Re: RFR: 8281384: Random chars on paste from Windows clipboard

2025-02-25 Thread Kevin Rushforth
On Tue, 25 Feb 2025 14:36:38 GMT, Kevin Rushforth wrote: >> Windows programs may reuse a clipboard buffer that is larger than the new >> content. In this case de NUL terminator is not at the end of the buffer, but >> within it. >> The current implementation copys the whole buffer into a text f

Re: RFR: 8281384: Random chars on paste from Windows clipboard

2025-02-25 Thread Kevin Rushforth
On Tue, 25 Feb 2025 13:25:07 GMT, Oliver Schmidtmer wrote: > Windows programs may reuse a clipboard buffer that is larger than the new > content. In this case de NUL terminator is not at the end of the buffer, but > within it. > The current implementation copys the whole buffer into a text fi

Re: RFR: 8281384: Random chars on paste from Windows clipboard

2025-02-25 Thread Kevin Rushforth
On Tue, 25 Feb 2025 13:25:07 GMT, Oliver Schmidtmer wrote: > Windows programs may reuse a clipboard buffer that is larger than the new > content. In this case de NUL terminator is not at the end of the buffer, but > within it. > The current implementation copys the whole buffer into a text fi

RFR: 8281384: Random chars on paste from Windows clipboard

2025-02-25 Thread Oliver Schmidtmer
Windows programs may reuse a clipboard buffer that is larger than the new content. In this case de NUL terminator is not at the end of the buffer, but within it. The current implementation copys the whole buffer into a text field, including the NUL terminator and the remaining chars. The JIRA