Re: RFR: 8283063: Optimize Observable{List/Set/Map}Wrapper.retainAll/removeAll [v4]

2023-04-04 Thread Michael Strauß
On Sat, 1 Apr 2023 22:14:09 GMT, John Hendrikx wrote: >> Michael Strauß has updated the pull request incrementally with one >> additional commit since the last revision: >> >> addressed review comments, added tests > > Looks really good now! @hjohn Can you re-approve? - PR

Re: RFR: 8264449: Enable reproducible builds with SOURCE_DATE_EPOCH [v12]

2023-04-04 Thread John Neffenger
On Tue, 4 Apr 2023 23:27:38 GMT, Kevin Rushforth wrote: > I started doing some testing today. Thank you, Kevin. I'm almost done with another round of my own testing and will post my results, too. > 2. The format of `VersionInfo.BUILD_TIMESTAMP`, which is used in constructing > the

Re: RFR: 8150709: Mac OSX and German Keyboard Layout (Y/Z) [v7]

2023-04-04 Thread Martin Fox
On Tue, 4 Apr 2023 08:35:52 GMT, Jose Pereda wrote: >> @jperedadnr You set up the test correctly. I think you need to need to allow >> the 'terminal' application to use the accessibility API so the Java robot >> can send keyboard events. I can't provide you with the exact setting since >> I'm

Re: RFR: 8264449: Enable reproducible builds with SOURCE_DATE_EPOCH [v12]

2023-04-04 Thread Kevin Rushforth
On Tue, 4 Apr 2023 15:46:20 GMT, John Neffenger wrote: >> This pull request allows for reproducible builds of JavaFX on Linux, macOS, >> and Windows by defining the `SOURCE_DATE_EPOCH` environment variable. For >> example, the following commands create a reproducible build: >> >> >> $ export

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS [v4]

2023-04-04 Thread Andy Goryachev
On Tue, 4 Apr 2023 21:29:19 GMT, Phil Race wrote: >> This PR addresses some font problems on macOS. >> (1) Garbled Arabic with the System font >> (2) Non-ideal fallback fonts for all fonts >> (3) No bold for System font. >> >> In particular the standard System Font was garbling Arabic text -

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS [v2]

2023-04-04 Thread Phil Race
On Tue, 4 Apr 2023 20:45:42 GMT, Kevin Rushforth wrote: >> Phil Race has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix up Thai font name > > modules/javafx.graphics/src/main/java/com/sun/javafx/font/coretext/CTFactory.java > line 59:

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS [v4]

2023-04-04 Thread Phil Race
> This PR addresses some font problems on macOS. > (1) Garbled Arabic with the System font > (2) Non-ideal fallback fonts for all fonts > (3) No bold for System font. > > In particular the standard System Font was garbling Arabic text - random > glyphs from another font. > The root of this issue

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS [v3]

2023-04-04 Thread Andy Goryachev
On Tue, 4 Apr 2023 21:15:03 GMT, Phil Race wrote: >> This PR addresses some font problems on macOS. >> (1) Garbled Arabic with the System font >> (2) Non-ideal fallback fonts for all fonts >> (3) No bold for System font. >> >> In particular the standard System Font was garbling Arabic text -

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS [v4]

2023-04-04 Thread Phil Race
On Tue, 4 Apr 2023 18:17:55 GMT, Andy Goryachev wrote: >> Phil Race has updated the pull request incrementally with one additional >> commit since the last revision: >> >> formatting and make some fields private > >

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS [v3]

2023-04-04 Thread Phil Race
> This PR addresses some font problems on macOS. > (1) Garbled Arabic with the System font > (2) Non-ideal fallback fonts for all fonts > (3) No bold for System font. > > In particular the standard System Font was garbling Arabic text - random > glyphs from another font. > The root of this issue

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS [v2]

2023-04-04 Thread Kevin Rushforth
On Tue, 4 Apr 2023 20:47:09 GMT, Phil Race wrote: >> This PR addresses some font problems on macOS. >> (1) Garbled Arabic with the System font >> (2) Non-ideal fallback fonts for all fonts >> (3) No bold for System font. >> >> In particular the standard System Font was garbling Arabic text -

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS [v2]

2023-04-04 Thread Phil Race
> This PR addresses some font problems on macOS. > (1) Garbled Arabic with the System font > (2) Non-ideal fallback fonts for all fonts > (3) No bold for System font. > > In particular the standard System Font was garbling Arabic text - random > glyphs from another font. > The root of this issue

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS

2023-04-04 Thread Phil Race
On Tue, 4 Apr 2023 18:49:41 GMT, Andy Goryachev wrote: > Thai does not render for me on Ventura 13.3/M1 > > ![Screenshot 2023-04-04 at 11 49 > 03](https://user-images.githubusercontent.com/107069028/229891044-3ff3d365-6c51-48d0-8c30-759f8f376ec9.png) On macOS 12 Thai uses Thonburi from

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS

2023-04-04 Thread Andy Goryachev
On Fri, 24 Mar 2023 21:37:16 GMT, Phil Race wrote: > This PR addresses some font problems on macOS. > (1) Garbled Arabic with the System font > (2) Non-ideal fallback fonts for all fonts > (3) No bold for System font. > > In particular the standard System Font was garbling Arabic text - random

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS

2023-04-04 Thread Andy Goryachev
On Fri, 24 Mar 2023 21:37:16 GMT, Phil Race wrote: > This PR addresses some font problems on macOS. > (1) Garbled Arabic with the System font > (2) Non-ideal fallback fonts for all fonts > (3) No bold for System font. > > In particular the standard System Font was garbling Arabic text - random

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS

2023-04-04 Thread Phil Race
On Mon, 3 Apr 2023 22:55:02 GMT, Andy Goryachev wrote: > > You mention "Thai" amongst them .. but I do see Thai rendered .. > > Thai is not rendered for me, was Ventura 13.1, now Ventura 13.3 on Mac M1, > (left - ventura 13.1 with the earlier commit, right - unpatched master) So no worse off

Re: RFR: 8264449: Enable reproducible builds with SOURCE_DATE_EPOCH [v12]

2023-04-04 Thread John Neffenger
> This pull request allows for reproducible builds of JavaFX on Linux, macOS, > and Windows by defining the `SOURCE_DATE_EPOCH` environment variable. For > example, the following commands create a reproducible build: > > > $ export SOURCE_DATE_EPOCH=$(git log -1 --pretty=%ct) > $ bash gradlew

Integrated: 8286089: Intermittent WebKit build failure on macOS in JavaScriptCore

2023-04-04 Thread Jay Bhaskar
On Tue, 28 Mar 2023 02:08:27 GMT, Jay Bhaskar wrote: > Issue: Error copying file (if different) from > Source/JavaScriptCore/Scripts/wkbuiltins/builtins_generate_separate_header.py" > to > "modules/javafx.web/build/mac/Release/JavaScriptCcripts/builtins_generate_separate_header.py". > > Root

Re: RFR: 8286089: Intermittent WebKit build failure on macOS in JavaScriptCore

2023-04-04 Thread Joeri Sykora
On Tue, 28 Mar 2023 02:08:27 GMT, Jay Bhaskar wrote: > Issue: Error copying file (if different) from > Source/JavaScriptCore/Scripts/wkbuiltins/builtins_generate_separate_header.py" > to > "modules/javafx.web/build/mac/Release/JavaScriptCcripts/builtins_generate_separate_header.py". > > Root

Integrated: 8282359: Intermittent WebKit build failure on Windows: C1090: PDB API call failed, error code 23

2023-04-04 Thread Hima Bindu Meda
On Fri, 31 Mar 2023 07:57:11 GMT, Hima Bindu Meda wrote: > In Jenkins Build, the intermediate failure on windows is observed due to > crash in mspdbserv.exe, which generates Program database (.pdb) files. > Disabled the pdb file generation in order to resolve the build failure. > Sanity testing

Re: RFR: 8282359: Intermittent WebKit build failure on Windows: C1090: PDB API call failed, error code 23 [v4]

2023-04-04 Thread Joeri Sykora
On Mon, 3 Apr 2023 18:50:57 GMT, Hima Bindu Meda wrote: >> In Jenkins Build, the intermediate failure on windows is observed due to >> crash in mspdbserv.exe, which generates Program database (.pdb) files. >> Disabled the pdb file generation in order to resolve the build failure. >> Sanity

Re: RFR: 8283551: ControlAcceleratorSupport menu items listener causes memory leak [v2]

2023-04-04 Thread Dean Wookey
On Fri, 24 Feb 2023 10:04:48 GMT, Dean Wookey wrote: >> Each time a menu would change scenes, a new set of ListChangeListeners would >> be added to the items in the menu. The bigger problem however is that these >> list change listeners have a strong reference to the scene which is >>

Re: RFR: 8283063: Optimize Observable{List/Set/Map}Wrapper.retainAll/removeAll [v5]

2023-04-04 Thread Nir Lisker
On Sun, 2 Apr 2023 23:50:15 GMT, Michael Strauß wrote: >> `Observable{List/Set/Map}Wrapper.retainAll/removeAll` can be optimized for >> some edge cases. >> >> 1. `removeAll(c)`: >> This is a no-op if 'c' is empty. >> For `ObservableListWrapper`, returning early skips an object allocation. For

Re: RFR: JDK-8199216: Memory leak and quadratic layout time with nested nodes (hbox) and pseudo-class in style sheet [v3]

2023-04-04 Thread John Hendrikx
On Tue, 4 Apr 2023 09:31:12 GMT, John Hendrikx wrote: >> I think I am missing something :) >> >> The set that is passed in may indeed be immutable, but it may not be the set >> we have cached. The memory savings only work when I then return the cached >> set; if I return the set you pass in

Re: RFR: JDK-8199216: Memory leak and quadratic layout time with nested nodes (hbox) and pseudo-class in style sheet [v3]

2023-04-04 Thread John Hendrikx
On Tue, 4 Apr 2023 09:26:21 GMT, John Hendrikx wrote: >> Couldn't you just add a fast path by calculating the hash code of the >> immutable set eagerly and storing it in a field? This entire method could >> probably be written to be allocation-free if the set passed into the method >> is

Re: RFR: JDK-8199216: Memory leak and quadratic layout time with nested nodes (hbox) and pseudo-class in style sheet [v3]

2023-04-04 Thread John Hendrikx
On Mon, 3 Apr 2023 23:36:32 GMT, Michael Strauß wrote: >> I don't think there is much of an impact, scene graph size should not affect >> it. The sets involved are tiny, usually consisting of 1 or 2 items and very >> rarely 3 or possibly more(*). The hash code of `PseudoClass` is very fast,

Re: RFR: 8150709: Mac OSX and German Keyboard Layout (Y/Z) [v7]

2023-04-04 Thread Jose Pereda
On Mon, 3 Apr 2023 22:55:16 GMT, Martin Fox wrote: >> tests/manual/events/KeyboardTest.java line 79: >> >>> 77: */ >>> 78: >>> 79: public class KeyboardTest extends Application { >> >> Could you clarify how to run the test and what one should expect? >> >> In my case, I'm running from Mac,