Re: RFR: 8312603: ArrayIndexOutOfBoundsException in Marlin when scaleX is 0 [v2]

2024-01-25 Thread Laurent Bourgès
On Thu, 25 Jan 2024 17:16:51 GMT, Laurent Bourgès wrote: >> Fixed scale=0 in DMarlinPrismUtils + added new test Scale0Test.java > > Laurent Bourgès has updated the pull request incrementally with one > additional commit since the last revision: > > fixed test package I fixed new test so syst

Re: Thought experiment: constructing Nodes on a different thread...

2024-01-25 Thread John Hendrikx
Thanks, it seems I misunderstood something fundamental here. I always figured the memory effects are limited to the variables accessed/mutated within a synchronized block (or *could* be limited to just those by selectively flushing data by some advanced VM).  Apparently that's not the case, an

Re: RFR: JDK-8314215 Trailing Spaces before Line Breaks Affect the Center Alignment of Text

2024-01-25 Thread John Hendrikx
On Thu, 25 Jan 2024 23:44:06 GMT, Andy Goryachev wrote: >>> Regarding other whitespace characters. Using \u2003 em space, pasting the >>> following text >> >> I think we can use `Character.isWhitespace`. For the most part it aligns >> with "is this a breaking space". It's not perfect, but we

Re: RFR: JDK-8314215 Trailing Spaces before Line Breaks Affect the Center Alignment of Text [v3]

2024-01-25 Thread John Hendrikx
> There are a number of tickets open related to text rendering: > > https://bugs.openjdk.org/browse/JDK-8314215 > > https://bugs.openjdk.org/browse/JDK-8145496 > > https://bugs.openjdk.org/browse/JDK-8129014 > > They have in common that wrapped text is taking the trailing spaces on each > wrap

Re: RFR: JDK-8314215 Trailing Spaces before Line Breaks Affect the Center Alignment of Text [v2]

2024-01-25 Thread John Hendrikx
> There are a number of tickets open related to text rendering: > > https://bugs.openjdk.org/browse/JDK-8314215 > > https://bugs.openjdk.org/browse/JDK-8145496 > > https://bugs.openjdk.org/browse/JDK-8129014 > > They have in common that wrapped text is taking the trailing spaces on each > wrap

Re: RFR: JDK-8314215 Trailing Spaces before Line Breaks Affect the Center Alignment of Text

2024-01-25 Thread Andy Goryachev
On Thu, 25 Jan 2024 23:23:42 GMT, John Hendrikx wrote: >> Regarding other whitespace characters. >> Using \u2003 em space, pasting the following text >> >> AAABBBCCC >> >> >> the results as seen with MS Word 16.78 (23100802) on macOS 14.2.1 indicate >> that it treats em space as w

Re: RFR: JDK-8314215 Trailing Spaces before Line Breaks Affect the Center Alignment of Text

2024-01-25 Thread John Hendrikx
On Thu, 25 Jan 2024 16:18:37 GMT, Andy Goryachev wrote: > Regarding other whitespace characters. Using \u2003 em space, pasting the > following text I think we can use `Character.isWhitespace`. For the most part it aligns with "is this a breaking space". It's not perfect, but we can do more w

Re: RFR: JDK-8314215 Trailing Spaces before Line Breaks Affect the Center Alignment of Text

2024-01-25 Thread John Hendrikx
On Sun, 10 Sep 2023 20:50:18 GMT, John Hendrikx wrote: > There are a number of tickets open related to text rendering: > > https://bugs.openjdk.org/browse/JDK-8314215 > > https://bugs.openjdk.org/browse/JDK-8145496 > > https://bugs.openjdk.org/browse/JDK-8129014 > > They have in common that w

Integrated: 8308955: MediaPlayer/AudioClip skip data on seek/loop

2024-01-25 Thread Alexander Matveev
On Tue, 23 Jan 2024 03:13:11 GMT, Alexander Matveev wrote: > This is regression from JDK-8262365. JDK-8262365 introduced support for > hardware pause for audio device. For some reason we will skip ~500 ms of > audio data after such pause. It is not noticeable for large audio files, but > for

Re: RFR: 8305418: [Linux] Replace obsolete XIM as Input Method Editor [v19]

2024-01-25 Thread Martin Fox
On Wed, 17 Jan 2024 17:10:24 GMT, Thiago Milczarek Sayao wrote: >> This replaces obsolete XIM and uses gtk api for IME. >> Gtk uses [ibus](https://github.com/ibus/ibus) >> >> Gtk3+ uses relative positioning (as Wayland does), so I've added a Relative >> positioning on `InputMethodRequest`. >>

Re: RFR: 8305418: [Linux] Replace obsolete XIM as Input Method Editor [v17]

2024-01-25 Thread Martin Fox
On Wed, 17 Jan 2024 17:13:11 GMT, Thiago Milczarek Sayao wrote: >> modules/javafx.graphics/src/main/native-glass/gtk/glass_window_ime.cpp line >> 163: >> >>> 161: g_signal_connect(im_ctx.ctx, "preedit-changed", >>> G_CALLBACK(on_preedit_changed), this); >>> 162: g_signal_conne

Re: HEADS-UP: Threading restriction for Animation play, pause, stop now enforced USE CASE

2024-01-25 Thread Kevin Rushforth
If you mean the changes proposed in Draft PR https://github.com/openjdk/jfx/pull/1349 those are too-intrusive for where we are in JavaFX 22. I see no chance that we will get agreement on the approach and be able to finish the code review and CSR in the next 4 business days. If we later get ge

Re: HEADS-UP: Threading restriction for Animation play, pause, stop now enforced USE CASE

2024-01-25 Thread Michael Strauß
Hi Kevin, have you considered my proposal, which is basically the same approach: it uses runLater internally to dispatch the interaction with AbstractPrimaryTimer (instead of trying to make AbstractPrimaryTimer work under concurrent access). But from the point of view of the caller, the API works

Re: HEADS-UP: Threading restriction for Animation play, pause, stop now enforced USE CASE

2024-01-25 Thread Michael Strauß
Hi John, the threading restrictions are now removed. What I mean by those seemingly contradictory stratements is the following: This change doesn't make Animation inherently thread-safe, so if you configure the animation on thread A, you can't just call play() on thread B and expect it to work. Wh

Re: RFR: 8312603: ArrayIndexOutOfBoundsException in Marlin when scaleX is 0 [v2]

2024-01-25 Thread Laurent Bourgès
> Fixed scale=0 in DMarlinPrismUtils + added new test Scale0Test.java Laurent Bourgès has updated the pull request incrementally with one additional commit since the last revision: fixed test package - Changes: - all: https://git.openjdk.org/jfx/pull/1348/files - new: https:/

Re: RFR: JDK-8314215 Trailing Spaces before Line Breaks Affect the Center Alignment of Text

2024-01-25 Thread Andy Goryachev
On Sun, 10 Sep 2023 20:50:18 GMT, John Hendrikx wrote: > There are a number of tickets open related to text rendering: > > https://bugs.openjdk.org/browse/JDK-8314215 > > https://bugs.openjdk.org/browse/JDK-8145496 > > https://bugs.openjdk.org/browse/JDK-8129014 > > They have in common that w

Re: RFR: JDK-8324337: Cherry-pick WebKit 617.1 stabilization fixes

2024-01-25 Thread Kevin Rushforth
On Thu, 25 Jan 2024 13:32:21 GMT, Hima Bindu Meda wrote: > Cherry-picked changes related to webkit-2.42.4.Verified build on all > platforms. Sanity testing looks fine. Code changes look fine with one suggestion in the added code under `#if PLATFOMR(JAVA)`. I'll test it and finish my review.

Re: RFR: JDK-8314215 Trailing Spaces before Line Breaks Affect the Center Alignment of Text

2024-01-25 Thread Andy Goryachev
On Sun, 10 Sep 2023 20:50:18 GMT, John Hendrikx wrote: > There are a number of tickets open related to text rendering: > > https://bugs.openjdk.org/browse/JDK-8314215 > > https://bugs.openjdk.org/browse/JDK-8145496 > > https://bugs.openjdk.org/browse/JDK-8129014 > > They have in common that w

Re: Thought experiment: constructing Nodes on a different thread...

2024-01-25 Thread Kevin Rushforth
Yes, I think you are missing something. Unless you have an animation that just decides all on it's own to find and attach a newly discovered node to the scene and start rendering it, there will be a synchronization as a result of calling runLater. Consider the following code:     Label label

Re: HEADS-UP: Threading restriction for Animation play, pause, stop now enforced USE CASE

2024-01-25 Thread Kevin Rushforth
And that's why the "be careful" option is not a satisfying solution. Let's proceed with the option to change the implementation of play/pause/stop/start to do the work in a runLater, and change the docs accordingly. It's the only feasible option for JavaFX 22 (other than "do nothing"), and I a

Re: HEADS-UP: Threading restriction for Animation play, pause, stop now enforced USE CASE

2024-01-25 Thread Nir Lisker
> > Option 3 is basically document it as "be careful" and remove the thread > restriction recently introduced, am I correct? > Yes :) IMHO that can simply can't work at all, because this is what > (theoretically) can happen: > 1. Non-FX thread starts animation > - Fields are manipulated in A

RFR: JDK-8324337: Cherry-pick WebKit 617.1 stabilization fixes

2024-01-25 Thread Hima Bindu Meda
Cherry-picked changes related to webkit-2.42.4.Verified build on all platforms. Sanity testing looks fine. - Commit messages: - Resolve compilation error - Cherrypick changes related to 2.42.4 Changes: https://git.openjdk.org/jfx/pull/1350/files Webrev: https://webrevs.openjdk.or

Re: Thought experiment: constructing Nodes on a different thread...

2024-01-25 Thread Nir Lisker
Isn't that exactly what the Worker interface and its Task and Service implementations are for? Service says A Service is a non-visual component encapsulating the information required > to perform some work on one or more background threads. As part of the > JavaFX UI library, the Service knows abo

Integrated: 8324270: Update boot JDK to 21.0.2

2024-01-25 Thread Ambarish Rapte
On Mon, 22 Jan 2024 13:45:35 GMT, Ambarish Rapte wrote: > JDK 21.0.2 is now released : https://jdk.java.net/21/ > Did a full CI build including Webkit and executed all headful tests. This pull request has now been integrated. Changeset: 9e7e8e1d Author:Ambarish Rapte URL: https://gi

Thought experiment: constructing Nodes on a different thread...

2024-01-25 Thread John Hendrikx
All this threading talk has made me wonder something: Let's say there are two threads, the FX thread and Thread 1.  I do the following: - On thread 1: create Label - On thread 1: set Label text to "xyz" I now attach this label to an active Scene graph. This should be done on the FX thread as

Re: HEADS-UP: Threading restriction for Animation play, pause, stop now enforced USE CASE

2024-01-25 Thread John Hendrikx
Hi Michael, I'm not quite sure I see the point of this change.  The PR did not remove the threading restrictions on play/stop. I'm also confused by the seemingly contradictory statements: - this proposal does NOT allow Animation.play/stop/etc. to be "called on any thread" - It merely removes

Re: HEADS-UP: Threading restriction for Animation play, pause, stop now enforced USE CASE

2024-01-25 Thread John Hendrikx
On 24/01/2024 22:06, Nir Lisker wrote: This is the ballpark of what I meant with "the downside might be some surprise when these methods behave differently". That can be documented, and basically already is (because that is what the "asynchronous" is alluding to, the fact that after calling "pl