Additional Information to JDK-8150263 ObservableListWrapper

2020-02-04 Thread Peter, Benjamin
Hi devs, about JDK-8150263 : ObservableListWrapper fires change notifications from sort() and sort(Comparator) even if list is not changed. I stumbled about the problem and prepared a minimal test case. Is anyone here who could add that to the report? I am not a contributor. I can also

Re: set Destination attribute in printReqAttrSet

2020-02-04 Thread Philip Race
I found that there were some bugs in the JDK code that FX calls that need to be fixed first. I started on those but it was reasonably complex to test and so that is still WIP and then the FX part should be possible. -phil. On 2/4/20, 2:52 AM, Neacsu Cristian wrote: Hello, Any updates

Re: [Rev 02] RFR: 8237469: Inherited styles don't update when node is moved

2020-02-04 Thread Kevin Rushforth
On Wed, 5 Feb 2020 00:14:15 GMT, Dean Wookey wrote: >> Everything passes with the fix and 5 of the new tests fail without the fix. >> >> removingThenAddingNodeToDifferentBranchGetsNewFontStyleTest >> movingBranchToDifferentBranchGetsNewCssVariableTest >>

Re: [Rev 02] RFR: 8237469: Inherited styles don't update when node is moved

2020-02-04 Thread Kevin Rushforth
On Tue, 28 Jan 2020 09:47:24 GMT, Ajit Ghaisas wrote: >> The pull request has been updated with 1 additional commit. > > Marked as reviewed by aghaisas (Reviewer). I updated the PR title to match the JBS title, so that the commit message will be correct when this PR is integrated.

Re: [Rev 01] RFR: 8236259: MemoryLeak in ProgressIndicator

2020-02-04 Thread Kevin Rushforth
On Mon, 27 Jan 2020 15:26:48 GMT, Florian Kirmaier wrote: >> The use of static analysis tools to catch certain types of problems is >> orthogonal to a regression test to validate a bug fix of a specific memory >> leak. >> >> @FlorianKirmaier as mentioned previously, please file a new JBS

Re: [Rev 01] RFR: 8238434: Ensemble: Update version of Lucene to 7.7.2

2020-02-04 Thread Kevin Rushforth
> Update the version of Lucene used by Ensemble to the latest release of Lucene > 7 which is version 7.7.2. > > Per the instructions in > [`UPDATING-lucene.txt`](https://github.com/openjdk/jfx/blob/master/apps/samples/Ensemble8/UPDATING-lucene.txt), > I made the following chages: > > 1.

Re: RFR: 8238434: Ensemble: Update version of Lucene to 7.7.2

2020-02-04 Thread Kevin Rushforth
On Tue, 4 Feb 2020 16:41:20 GMT, Kevin Rushforth wrote: >> Update the version of Lucene used by Ensemble to the latest release of >> Lucene 7 which is version 7.7.2. >> >> Per the instructions in >>

Re: Memory leak issues related to 2 PRs in OpenJFX

2020-02-04 Thread Kevin Rushforth
The first of these bugs, 8090322, did introduce at least one memory leak for which 8216377 was the fix. I'd prefer to resolve the underlying problem rather than just making those listeners weak, so it seems worth spending some more time to try to narrow it down. Regarding the out-of-order

Memory leak issues related to 2 PRs in OpenJFX

2020-02-04 Thread Ed Kennard
Hi everyone, I’ve been migrating our codebase to Java 11 LTS and OpenJFX 14. One of our GC-related unit tests is failing now, where one of our custom Tab controls is supposed to be GC’d. The test passes against Java 8. When our GC-related unit tests fail they output exactly what it is that’s

OpenJFX 14 is in Rampdown Phase Two (RDP2)

2020-02-04 Thread Kevin Rushforth
To: OpenJFX Developers As a reminder, OpenJFX 14 is now in Rampdown Phase Two RDP2. [1] During RDP2, all bug fixes (except for docs and test fixes), and all enhancements will need explicit approval to go in to openjfx/14-dev. The bar for approving bug fixes is appropriately high at this

Re: RFR: 8238434: Ensemble: Update version of Lucene to 7.7.2

2020-02-04 Thread Kevin Rushforth
On Tue, 4 Feb 2020 16:39:22 GMT, Kevin Rushforth wrote: > Update the version of Lucene used by Ensemble to the latest release of Lucene > 7 which is version 7.7.2. > > Per the instructions in >

RFR: 8238434: Ensemble: Update version of Lucene to 7.7.2

2020-02-04 Thread Kevin Rushforth
Update the version of Lucene used by Ensemble to the latest release of Lucene 7 which is version 7.7.2. Per the instructions in [`UPDATING-lucene.txt`](https://github.com/openjdk/jfx/blob/master/apps/samples/Ensemble8/UPDATING-lucene.txt), I made the following chages: 1. Bumped the version

Re: set Destination attribute in printReqAttrSet

2020-02-04 Thread Neacsu Cristian
What I try to achieve, is to use "Microsoft Print to PDF" without inserting the path manually. In java 8 was possible to force adding that Destination parameter through reflection (again an ugly hack), because the API do not expose it. Now being encapsulated in Java 9+ is not achievable anymore...

Re: set Destination attribute in printReqAttrSet

2020-02-04 Thread Dan Howard
Have you tried using normal printing in Java? Just to print something with the usual AWT etc APIs.  If that works you could convert the node to an image to print. On 2/4/2020 5:52 AM, Neacsu Cristian wrote: Hello, Any updates regarding this matter? :-s Thank you in advance, Cristian-Stefan

Re: set Destination attribute in printReqAttrSet

2020-02-04 Thread Neacsu Cristian
Hello, Any updates regarding this matter? :-s Thank you in advance, Cristian-Stefan On Sat, May 11, 2019 at 2:36 AM Neacsu Cristian < neacsu.cristianste...@gmail.com> wrote: > Damn ... Thank you for your reply and for the ticket submitted. > I created an invoice application for my own company,