Re: RFR: JDK-8295236: Update JavaDoc in javafx.geometry.Point3D [v5]

2022-10-19 Thread Nir Lisker
On Wed, 19 Oct 2022 12:46:03 GMT, Douglas Held wrote: > I think there's something else of quality worth discussing here. Is `==` > appropriate given the values under comparison are `double`? It's not. I have mentioned this exact case in https://bugs.openjdk.org/browse/JDK-8226930. I can assign

Re: RFR: JDK-8295236: Update JavaDoc in javafx.geometry.Point3D [v5]

2022-10-19 Thread Nir Lisker
On Wed, 19 Oct 2022 12:38:54 GMT, Douglas Held wrote: >> modules/javafx.graphics/src/main/java/javafx/geometry/Point3D.java line 417: >> >>> 415: * {@code getX}, {@code getY}, and {@code getZ} methods are equal. >>> 416: * >>> 417: * @param {@code obj} the reference object with wh

Re: RFR: JDK-8295236: Update JavaDoc in javafx.geometry.Point3D [v4]

2022-10-19 Thread Nir Lisker
On Wed, 19 Oct 2022 12:33:37 GMT, Douglas Held wrote: > I thought the code tags produced hyperlinks. Those would be the `@link` tags, or `@linkplain`. - PR: https://git.openjdk.org/jfx/pull/913

Re: RFR: JDK-8295236: Update JavaDoc in javafx.geometry.Point3D [v6]

2022-10-19 Thread Nir Lisker
On Wed, 19 Oct 2022 12:47:31 GMT, Douglas Held wrote: >> The JavaDoc for equals had a copy/paste error. I normalized the text based >> on the JavaDoc for method java.awt.Point#equals. ~~I also changed formatting >> in the method signatures of equals(), hashCode() and toString().~~ >> >> For go

Re: RFR: JDK-8295236: Update JavaDoc in javafx.geometry.Point3D [v5]

2022-10-19 Thread Nir Lisker
On Wed, 19 Oct 2022 12:34:15 GMT, Douglas Held wrote: >> The JavaDoc for equals had a copy/paste error. I normalized the text based >> on the JavaDoc for method java.awt.Point#equals. ~~I also changed formatting >> in the method signatures of equals(), hashCode() and toString().~~ >> >> For go

[sig-nir] Re: Fwd: [Apnic-announce] Results of APNIC 54 SIG and NRO NC elections

2022-10-19 Thread Nguyen Thi Oanh via sig-nir
Dear Sunny and SIG members Firstly, I would like to thank outgoing Zhen Yu for his contribution on the NIR SIG forum. He contributed much for NIR SIG Forum for many years, and hope that Zhen will keep supporting the NIR SIG forum in the future. Secondly, Congratulations to Gaurav Kansal on

Re: RFR: JDK-8295236: Update JavaDoc in javafx.geometry.Point3D [v4]

2022-10-18 Thread Nir Lisker
On Tue, 18 Oct 2022 22:15:35 GMT, Douglas Held wrote: >> The JavaDoc for equals had a copy/paste error. I normalized the text based >> on the JavaDoc for method java.awt.Point#equals. ~~I also changed formatting >> in the method signatures of equals(), hashCode() and toString().~~ >> >> For go

Re: RFR: 8290040: Provide simplified deterministic way to manage listeners [v6]

2022-10-18 Thread Nir Lisker
On Tue, 18 Oct 2022 17:47:38 GMT, Michael Strauß wrote: >> I'd respectfully disagree: in a large application, there might be hundreds >> (a thousand?) Nodes, and only a handful (10-20?) Nodes with this property >> (2%), so it's a waste of RAM. One may think it's only 8Kb on a 64 bit >> machin

Re: [PATCH 0/3] Add qemu-img checksum command using blkhash

2022-10-18 Thread Nir Soffer
On Sun, Sep 18, 2022 at 12:35 PM Nir Soffer wrote: > ping > > Kevin, Hanna, I hope you have time to take a look. > > https://lists.nongnu.org/archive/html/qemu-block/2022-09/msg00021.html Ping again, hopefully someone has time to look at this :-) > > > > On Thu,

Re: [PATCH 0/3] Add qemu-img checksum command using blkhash

2022-10-18 Thread Nir Soffer
On Sun, Sep 18, 2022 at 12:35 PM Nir Soffer wrote: > ping > > Kevin, Hanna, I hope you have time to take a look. > > https://lists.nongnu.org/archive/html/qemu-block/2022-09/msg00021.html Ping again, hopefully someone has time to look at this :-) > > > > On Thu,

Re: RFR: 8217853: Cleanup in the D3D native pipeline [v7]

2022-10-18 Thread Nir Lisker
On Thu, 1 Sep 2022 11:23:44 GMT, Nir Lisker wrote: >> Refactoring and renaming changes to some of the D3D pipeline files and a few >> changes on the Java side. These are various "leftovers" from previous issues >> that we didn't want to touch at the time

Re: RFR: JDK-8295236: Update JavaDoc in javafx.geometry.Point3D [v3]

2022-10-18 Thread Nir Lisker
On Tue, 18 Oct 2022 04:47:38 GMT, Ambarish Rapte wrote: >> I think that we should explain what makes 2 objects equal. Even if we don't >> explicitly name the methods used for comparison, we could say "2 points are >> equals if their coordinates are equal". >> >> By the way, I have https://bugs

Re: RFR: JDK-8295236: Update JavaDoc in javafx.geometry.Point3D [v3]

2022-10-17 Thread Nir Lisker
On Mon, 17 Oct 2022 15:24:48 GMT, Ambarish Rapte wrote: >> ...correction, found in toString() as well. > > It would be suitable to align with our existing doc comment in other classes, > for example as here, > > 1. > https://github.com/openjdk/jfx/blob/9768b5e42391e2d48aefffa3a3f6b1de6c30ef9d/

Re: RFR: JDK-8295236: Update JavaDoc in javafx.geometry.Point3D [v3]

2022-10-17 Thread Nir Lisker
On Fri, 14 Oct 2022 15:08:24 GMT, Douglas Held wrote: >> Not strictly enforced, but adding a blank line does aid readability (of the >> source code...the generated docs don't care). > > If I were to add empty lines before JavaDoc tags such as @param etc, then for > consistency I think I should

Re: RFR: JDK-8295236: Update JavaDoc in javafx.geometry.Point3D

2022-10-13 Thread Nir Lisker
On Wed, 12 Oct 2022 17:28:42 GMT, Douglas Held wrote: > The JavaDoc for equals had a copy/paste error. I normalized the text based on > the JavaDoc for method java.awt.Point#equals. I also changed formatting in > the method signatures of equals(), hashCode() and toString(). > > For good measur

Re: RFR: 8294809: ListenerHelper for managing and disconnecting listeners

2022-10-13 Thread Nir Lisker
Subscription.subscribe(label.wrapTextProperty(), value > -> { ... } ); > > They can then be released all at once by doing: > > subscription.unsubscribe(); > > Reading your proposal I see you might be looking to support weak > listeners as well. Personally, I believe

[ovirt-users] Re: Issue with disk import and disk upload - ovirt 4.5.2

2022-10-04 Thread Nir Soffer
ly format conversion. Example usage: ovirt-img upload-disk --engine-url https://example.com \ --username username \ --password-file /path/to/password-file \ --cafile ca.pem \ --storage-domain mydomain \ /path/to/iso See ovirt-im upload-disk --help for mo

ObservableValue::map called twice

2022-09-28 Thread Nir Lisker
led twice. Looks like improper behavior to me. - Nir

Re: RFR: 8290040: Provide simplified deterministic way to manage listeners [v5]

2022-09-26 Thread Nir Lisker
On Thu, 22 Sep 2022 11:04:46 GMT, John Hendrikx wrote: >> This PR adds a new (lazy*) property on `Node` which provides a boolean which >> indicates whether or not the `Node` is currently part of a `Scene`, which in >> turn is part of a currently showing `Window`. >> >> It also adds a new fluen

Re: RFR: 8293883: Add tests/.classpath (Eclipse) [v2]

2022-09-23 Thread Nir Lisker
On Thu, 22 Sep 2022 20:33:44 GMT, Andy Goryachev wrote: >> Adding a tests/.classpath file to include tests/manual/controls source >> folder so Eclipse could see/execute manual tests there. >> >> Also, I would rather move the sources there to a specific package >> (test.manual ?) instead of a d

Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Nir Lisker
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev wrote: > Adding a tests/.classpath file to include tests/manual/controls source folder > so Eclipse could see/execute manual tests there. > > Also, I would rather move the sources there to a specific package > (test.manual ?) instead of a defaul

Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Nir Lisker
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev wrote: > Adding a tests/.classpath file to include tests/manual/controls source folder > so Eclipse could see/execute manual tests there. > > Also, I would rather move the sources there to a specific package > (test.manual ?) instead of a defaul

Re: RFR: 8290040: Provide simplified deterministic way to manage listeners [v4]

2022-09-21 Thread Nir Lisker
On Sun, 4 Sep 2022 19:49:42 GMT, John Hendrikx wrote: >> This PR adds a new (lazy*) property on `Node` which provides a boolean which >> indicates whether or not the `Node` is currently part of a `Scene`, which in >> turn is part of a currently showing `Window`. >> >> It also adds a new fluent

Re: [PATCH 0/3] Add qemu-img checksum command using blkhash

2022-09-18 Thread Nir Soffer
ping Kevin, Hanna, I hope you have time to take a look. https://lists.nongnu.org/archive/html/qemu-block/2022-09/msg00021.html On Thu, Sep 1, 2022 at 5:32 PM Nir Soffer wrote: > > Since blkhash is available only via copr now, the new command is added as > optional feature, buil

Re: [PATCH 0/3] Add qemu-img checksum command using blkhash

2022-09-18 Thread Nir Soffer
ping Kevin, Hanna, I hope you have time to take a look. https://lists.nongnu.org/archive/html/qemu-block/2022-09/msg00021.html On Thu, Sep 1, 2022 at 5:32 PM Nir Soffer wrote: > > Since blkhash is available only via copr now, the new command is added as > optional feature, buil

Re: RFR: 8293839: Documentation memory consistency effects of runLater [v2]

2022-09-15 Thread Nir Lisker
On Thu, 15 Sep 2022 11:52:00 GMT, Jens Lidestrom wrote: >> Prior to this change it was not clear from the documentation if callers of >> Platform#runLater must perform >> any synchronisation to have writes of the calling thread be visible in the >> JavaFX Application >> Thread. It is important

Re: [blink-dev] Re: Intent to Experiment: User Agent Reduction Origin Trial

2022-09-14 Thread Nir M
Hi Mike, Nir from Meta and Noah's peer. would it be possible to give an estimate or a guideline on the permissible magnitude of usage for the Opt-In trial (the one that forces the full reduction of the UserAgent) available? As we would like to conduct an experiment on that, and not de

Re: RFR: 8290040: Provide simplified deterministic way to manage listeners [v3]

2022-09-01 Thread Nir Lisker
On Thu, 1 Sep 2022 13:51:21 GMT, John Hendrikx wrote: >> This PR adds a new (lazy*) property on `Node` which provides a boolean which >> indicates whether or not the `Node` is currently part of a `Scene`, which in >> turn is part of a currently showing `Window`. >> >> It also adds a new fluent

Re: RFR: 8290040: Provide simplified deterministic way to manage listeners [v2]

2022-09-01 Thread Nir Lisker
On Thu, 1 Sep 2022 13:10:49 GMT, John Hendrikx wrote: >> modules/javafx.base/src/main/java/javafx/beans/value/ObservableValue.java >> line 266: >> >>> 264: * {@code condition} evaluates to {@code true}. This allows this >>> {@code ObservableValue} >>> 265: * and the conditional {@cod

Re: RFR: 8290040: Provide simplified deterministic way to manage listeners [v2]

2022-09-01 Thread Nir Lisker
On Thu, 1 Sep 2022 13:38:46 GMT, John Hendrikx wrote: >> modules/javafx.base/src/main/java/javafx/beans/value/ObservableValue.java >> line 274: >> >>> 272: * >>> 273: * Returning {@code null} from the given condition is treated the >>> same as >>> 274: * returning {@code false}

[PATCH 3/3] qemu-img: Speed up checksum

2022-09-01 Thread Nir Soffer
5s ±0.183s | 1.808s ±0.016s | x1.36 | | nbd | 6g | direct| 3.540s ±0.020s | 1.749s ±0.018s | x2.02 | [1] raw image full of zeroes [2] raw fedora 35 image with additional random data, 50% full [3] image [2] exported by qemu-nbd via unix socket Signed-off-by: Nir Soffer --- qemu-i

[PATCH 1/3] qemu-img: Add checksum command

2022-09-01 Thread Nir Soffer
blkhash/ [3] Computing checksum for 8T empty image: qemu-img checksum: 3.7s, sha256sum (estimate): 17,749s Signed-off-by: Nir Soffer --- docs/tools/qemu-img.rst | 22 + meson.build | 10 ++- meson_options.txt | 2 + qemu-img-cmds.hx| 8 ++ qemu-img.c

[PATCH 0/3] Add qemu-img checksum command using blkhash

2022-09-01 Thread Nir Soffer
Since blkhash is available only via copr now, the new command is added as optional feature, built only if blkhash-devel package is installed. Nir Soffer (3): qemu-img: Add checksum command iotests: Test qemu-img checksum qemu-img: Speed up checksum docs/tools/qemu-img.rst

[PATCH 2/3] iotests: Test qemu-img checksum

2022-09-01 Thread Nir Soffer
includes a json map of every test image. Signed-off-by: Nir Soffer --- tests/qemu-iotests/tests/qemu-img-checksum| 149 ++ .../qemu-iotests/tests/qemu-img-checksum.out | 74 + 2 files changed, 223 insertions(+) create mode 100755 tests/qemu-iotests/tests/qemu-img

[PATCH 1/3] qemu-img: Add checksum command

2022-09-01 Thread Nir Soffer
blkhash/ [3] Computing checksum for 8T empty image: qemu-img checksum: 3.7s, sha256sum (estimate): 17,749s Signed-off-by: Nir Soffer --- docs/tools/qemu-img.rst | 22 + meson.build | 10 ++- meson_options.txt | 2 + qemu-img-cmds.hx| 8 ++ qemu-img.c

[PATCH 3/3] qemu-img: Speed up checksum

2022-09-01 Thread Nir Soffer
5s ±0.183s | 1.808s ±0.016s | x1.36 | | nbd | 6g | direct| 3.540s ±0.020s | 1.749s ±0.018s | x2.02 | [1] raw image full of zeroes [2] raw fedora 35 image with additional random data, 50% full [3] image [2] exported by qemu-nbd via unix socket Signed-off-by: Nir Soffer --- qemu-i

[PATCH 2/3] iotests: Test qemu-img checksum

2022-09-01 Thread Nir Soffer
includes a json map of every test image. Signed-off-by: Nir Soffer --- tests/qemu-iotests/tests/qemu-img-checksum| 149 ++ .../qemu-iotests/tests/qemu-img-checksum.out | 74 + 2 files changed, 223 insertions(+) create mode 100755 tests/qemu-iotests/tests/qemu-img

[PATCH 0/3] Add qemu-img checksum command using blkhash

2022-09-01 Thread Nir Soffer
Since blkhash is available only via copr now, the new command is added as optional feature, built only if blkhash-devel package is installed. Nir Soffer (3): qemu-img: Add checksum command iotests: Test qemu-img checksum qemu-img: Speed up checksum docs/tools/qemu-img.rst

Re: RFR: 8290040: Provide simplified deterministic way to manage listeners [v2]

2022-09-01 Thread Nir Lisker
On Thu, 1 Sep 2022 12:35:21 GMT, John Hendrikx wrote: >> `when` sounds fine, but we need to be careful about its relation to the >> `When` class. >> >> This reminds me also that some of the new functionality in `ObservableValue` >> can be used as a replacement for `When`, maybe we should note

Re: RFR: 8217853: Cleanup in the D3D native pipeline [v5]

2022-09-01 Thread Nir Lisker
On Thu, 1 Sep 2022 05:35:09 GMT, Ambarish Rapte wrote: >> I still don't understand. What is `VertexType`? I see a comment on `VsInput` >> "// this is a .hlsl mirror of the vertex format, see VertexTypes.h". If the >> gradle file has `"/DVertexType=VsInput"`, shouldn't it be fine with >> `VsInp

Re: RFR: 8217853: Cleanup in the D3D native pipeline [v7]

2022-09-01 Thread Nir Lisker
ded it to `psConstants` instead. > * `vs2ps` > * Removed the ambient color interpolation, which frees a register (no > change in performance). > * Simplified the structure (what is `LocalBumpOut` and why is it called > `light` and contains `LocalBump`?). > * `Mtl1PS` and `psM

Re: RFR: 8290040: Provide simplified deterministic way to manage listeners [v2]

2022-08-31 Thread Nir Lisker
On Fri, 15 Jul 2022 09:27:00 GMT, John Hendrikx wrote: >> This PR adds a new (lazy*) property on `Node` which provides a boolean which >> indicates whether or not the `Node` is currently part of a `Scene`, which in >> turn is part of a currently showing `Window`. >> >> It also adds a new fluen

Re: RFR: 8290040: Provide simplified deterministic way to manage listeners [v2]

2022-08-31 Thread Nir Lisker
On Fri, 15 Jul 2022 09:07:07 GMT, John Hendrikx wrote: >> John Hendrikx has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Rename showing property to shown as it is already used in subclasses > > modules/javafx.base/src/main/java/javafx/bea

Re: RFR: 8292678 Openjfx: all projects to use JUnit5 (Eclipse)

2022-08-31 Thread Nir Lisker
On Tue, 30 Aug 2022 20:35:17 GMT, Andy Goryachev wrote: > it might affect the initial 'import as a gradle project', and associated > testing. I removed it and didn't have any issue with this. The classpath file is not used when importing a project, just when it's configured, and in this case

Re: RFR: 8292678 Openjfx: all projects to use JUnit5 (Eclipse)

2022-08-30 Thread Nir Lisker
On Tue, 30 Aug 2022 19:09:40 GMT, Andy Goryachev wrote: > I wouldn't do it - it is convenient for git operations via context menu and > also gives access to other non-java parts (doc-files for instance). > > But the main issue is that it is has a > org.eclipse.buildship.core.gradleprojectnatur

Re: RFR: 8292678 Openjfx: all projects to use JUnit5 (Eclipse)

2022-08-30 Thread Nir Lisker
On Tue, 23 Aug 2022 18:38:48 GMT, Andy Goryachev wrote: > This change affects eclipse projects only - all eclipse projects are > configured to use JUnit5. Marked as reviewed by nlisker (Reviewer). Please submit an issue for removing the root project's classpath file. The 2 future changes to t

Re: RFR: 8217853: Cleanup in the D3D native pipeline [v5]

2022-08-30 Thread Nir Lisker
On Fri, 26 Aug 2022 17:34:41 GMT, Ambarish Rapte wrote: >> What do you mean by "revert the type function parameter here"? Doesn't the >> type parameter match after a change to the gradle file? > >> What do you mean by "revert the type function parameter here"? > > `VsOutput main(VsInput vsInput

Re: RFR: 8290765: Remove parent disabled/treeVisible listeners [v2]

2022-08-25 Thread Nir Lisker
On Thu, 25 Aug 2022 22:18:44 GMT, Michael Strauß wrote: >> `Node` adds InvalidationListeners to its parent's `disabled` and >> `treeVisible` properties and calls its own `updateDisabled()` and >> `updateTreeVisible(boolean)` methods when the property values change. >> >> These listeners are no

Re: RFR: 8290765: Remove parent disabled/treeVisible listeners

2022-08-25 Thread Nir Lisker
On Thu, 25 Aug 2022 13:46:26 GMT, Kevin Rushforth wrote: >> Since `getChildren()` is not final, one can easily override it and return >> null. >> Therefore, this check should still be done here. >> Maybe we even need to check that every child is not null, since again I can >> override `getChi

Re: RFR: 8290765: Remove parent disabled/treeVisible listeners

2022-08-25 Thread Nir Lisker
On Thu, 21 Jul 2022 04:43:15 GMT, Michael Strauß wrote: > `Node` adds InvalidationListeners to its parent's `disabled` and > `treeVisible` properties and calls its own `updateDisabled()` and > `updateTreeVisible(boolean)` methods when the property values change. > > These listeners are not req

Re: RFR: 8292678 Openjfx: all projects to use JUnit5 (Eclipse)

2022-08-24 Thread Nir Lisker
On Tue, 23 Aug 2022 18:38:48 GMT, Andy Goryachev wrote: > This change affects eclipse projects only - all eclipse projects are > configured to use JUnit5. Not the `base` module, the root one, called "jfx" or "rt". - PR: https://git.openjdk.org/jfx/pull/883

Re: RFR: 8292678 Openjfx: all projects to use JUnit5 (Eclipse)

2022-08-23 Thread Nir Lisker
On Tue, 23 Aug 2022 18:38:48 GMT, Andy Goryachev wrote: > This change affects eclipse projects only - all eclipse projects are > configured to use JUnit5. I don't see why the base project needs a `.classpath` file at all considering that it doesn't contain sources. - PR: https://

Re: RFR: 8217853: Cleanup in the D3D native pipeline [v5]

2022-08-23 Thread Nir Lisker
On Tue, 2 Aug 2022 17:38:06 GMT, Ambarish Rapte wrote: >> Nir Lisker has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Renamed method > > modules/javafx.graphics/src/main/native-prism-d3d/hlsl/vs2ps.h line 2

Re: RFR: 8217853: Cleanup in the D3D native pipeline [v6]

2022-08-23 Thread Nir Lisker
ded it to `psConstants` instead. > * `vs2ps` > * Removed the ambient color interpolation, which frees a register (no > change in performance). > * Simplified the structure (what is `LocalBumpOut` and why is it called > `light` and contains `LocalBump`?). > * `Mtl1PS` and `psM

Re: RFR: 8217853: Cleanup in the D3D native pipeline [v5]

2022-08-23 Thread Nir Lisker
On Tue, 2 Aug 2022 15:10:50 GMT, Ambarish Rapte wrote: >> Nir Lisker has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Renamed method > > modules/javafx.graphics/src/main/native-prism-d3d/hlsl/Mtl1VS.hl

[jfx19] Integrated: 8286678: Fix mistakes in FX API docs

2022-08-23 Thread Nir Lisker
On Thu, 18 Aug 2022 23:52:48 GMT, Nir Lisker wrote: > Fixes the mistakes in the JBS ticket and some additional minor corrections. This pull request has now been integrated. Changeset: 8b967765 Author: Nir Lisker URL: https://git.openjdk.org/jfx/com

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v5]

2022-08-23 Thread Nir Lisker
> Fixes the mistakes in the JBS ticket and some additional minor corrections. Nir Lisker has updated the pull request incrementally with one additional commit since the last revision: Addressed review comment - Changes: - all: https://git.openjdk.org/jfx/pull/880/files -

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v4]

2022-08-23 Thread Nir Lisker
On Tue, 23 Aug 2022 15:36:35 GMT, Nir Lisker wrote: >> Fixes the mistakes in the JBS ticket and some additional minor corrections. > > Nir Lisker has updated the pull request incrementally with one additional > commit since the last revision: > > Addressed review comm

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v4]

2022-08-23 Thread Nir Lisker
> Fixes the mistakes in the JBS ticket and some additional minor corrections. Nir Lisker has updated the pull request incrementally with one additional commit since the last revision: Addressed review comments - Changes: - all: https://git.openjdk.org/jfx/pull/880/fi

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v3]

2022-08-23 Thread Nir Lisker
> Fixes the mistakes in the JBS ticket and some additional minor corrections. Nir Lisker has updated the pull request incrementally with one additional commit since the last revision: Addressed review comments - Changes: - all: https://git.openjdk.org/jfx/pull/880/fi

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v2]

2022-08-23 Thread Nir Lisker
On Mon, 22 Aug 2022 22:21:29 GMT, Kevin Rushforth wrote: >> Nir Lisker has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix typo > > modules/javafx.graphics/src/main/java/javafx/concurrent/ScheduledServi

[jfx19] Integrated: 8291906: Bindings.createXxxBinding inherit incorrect method docs

2022-08-19 Thread Nir Lisker
On Fri, 19 Aug 2022 00:18:37 GMT, Nir Lisker wrote: > Added docs for inherited methods in `Bindings` subclasses. This pull request has now been integrated. Changeset: 5c007833 Author: Nir Lisker URL: https://git.openjdk.org/jfx/commit/5c00783352940b08f6254d2250524ef804984267 St

Re: [jfx19] RFR: 8291906: Bindings.createXxxBinding inherit incorrect method docs

2022-08-19 Thread Nir Lisker
On Sat, 20 Aug 2022 00:27:32 GMT, Kevin Rushforth wrote: >> Added docs for inherited methods in `Bindings` subclasses. > > modules/javafx.base/src/main/java/javafx/beans/binding/Bindings.java line 174: > >> 172: /** >> 173: * Returns an immutable list of the dependencies

Re: [jfx19] RFR: 8291906: Bindings.createXxxBinding inherit incorrect method docs

2022-08-19 Thread Nir Lisker
On Fri, 19 Aug 2022 00:18:37 GMT, Nir Lisker wrote: > Added docs for inherited methods in `Bindings` subclasses. I don't mind either way. I initially bundled it with the issue for fixing API docs, but decoupled it later and just thought it would be fine to do them both. The changes

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v2]

2022-08-19 Thread Nir Lisker
On Fri, 19 Aug 2022 14:54:11 GMT, Andy Goryachev wrote: > Would these changes require a CSR? Or, since they are just corrections, the > API did not really change? Otherwise, looks good. Thank you for cleaning up > the docs! Usually they don't since there are no spec changes. - PR

[jfx19] Withdrawn: 8291906: Bindings.createXxxBinding inherit incorrect method docs

2022-08-18 Thread Nir Lisker
On Thu, 18 Aug 2022 19:21:39 GMT, Nir Lisker wrote: > Simple fixes for the inherited method docs. This pull request has been closed without being integrated. - PR: https://git.openjdk.org/jfx/pull/878

Re: [jfx19] RFR: 8291906: Bindings.createXxxBinding inherit incorrect method docs

2022-08-18 Thread Nir Lisker
On Thu, 18 Aug 2022 19:21:39 GMT, Nir Lisker wrote: > Simple fixes for the inherited method docs. Closed in favor of #881. - PR: https://git.openjdk.org/jfx/pull/878

Re: [jfx19] RFR: 8291906: Bindings.createXxxBinding inherit incorrect method docs [v2]

2022-08-18 Thread Nir Lisker
> Simple fixes for the inherited method docs. Nir Lisker has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains 19 new commits since the l

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v2]

2022-08-18 Thread Nir Lisker
> Fixes the mistakes in the JBS ticket and some additional minor corrections. Nir Lisker has updated the pull request incrementally with one additional commit since the last revision: Fix typo - Changes: - all: https://git.openjdk.org/jfx/pull/880/files - new: ht

[jfx19] RFR: 8291906: Bindings.createXxxBinding inherit incorrect method docs

2022-08-18 Thread Nir Lisker
Added docs for inherited methods in `Bindings` subclasses. - Commit messages: - Initial commit Changes: https://git.openjdk.org/jfx/pull/881/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=881&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8291906 Stats: 42 lines in

[jfx19] RFR: 8291906: Bindings.createXxxBinding inherit incorrect method docs

2022-08-18 Thread Nir Lisker
Simple fixes for the inherited method docs. - Commit messages: - Initial commit - 8290473: update Eclipse .classpath in apps, buildSrc - 8181084: JavaFX show big icons in system menu on macOS with Retina display - 8291908: VirtualFlow creates unneeded empty cells - 8291630: Updat

[jfx19] RFR: 8286678: Fix mistakes in FX API docs

2022-08-18 Thread Nir Lisker
Fixes the mistakes in the JBS ticket and some additional minor corrections. - Commit messages: - Initial commit Changes: https://git.openjdk.org/jfx/pull/880/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=880&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8286678 S

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v3]

2022-08-18 Thread Nir Lisker
On Thu, 18 Aug 2022 23:43:33 GMT, Nir Lisker wrote: >> Fixes the mistakes in the JBS ticket and some additional minor corrections. > > Nir Lisker has refreshed the contents of this pull request, and previous > commits have been removed. The incremental views will show differences

[jfx19] Withdrawn: 8286678: Fix mistakes in FX API docs

2022-08-18 Thread Nir Lisker
On Thu, 18 Aug 2022 19:08:19 GMT, Nir Lisker wrote: > Fixes the mistakes in the JBS ticket and some additional minor corrections. This pull request has been closed without being integrated. - PR: https://git.openjdk.org/jfx/pull/877

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v3]

2022-08-18 Thread Nir Lisker
On Thu, 18 Aug 2022 23:43:33 GMT, Nir Lisker wrote: >> Fixes the mistakes in the JBS ticket and some additional minor corrections. > > Nir Lisker has refreshed the contents of this pull request, and previous > commits have been removed. Incremental views are not available. `g

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v3]

2022-08-18 Thread Nir Lisker
> Fixes the mistakes in the JBS ticket and some additional minor corrections. Nir Lisker has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs

2022-08-18 Thread Nir Lisker
On Thu, 18 Aug 2022 19:08:19 GMT, Nir Lisker wrote: > Fixes the mistakes in the JBS ticket and some additional minor corrections. Looks like the rebase missed a few commits. Not sure why. - PR: https://git.openjdk.org/jfx/pull/877

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs [v2]

2022-08-18 Thread Nir Lisker
> Fixes the mistakes in the JBS ticket and some additional minor corrections. Nir Lisker has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains

Re: [jfx19] RFR: 8286678: Fix mistakes in FX API docs

2022-08-18 Thread Nir Lisker
On Thu, 18 Aug 2022 19:08:19 GMT, Nir Lisker wrote: > Fixes the mistakes in the JBS ticket and some additional minor corrections. It's a PR against the javafx19 branch. I think I forked the master branch though, which is why it is showing all the commits. I might need to rebase, but

[jfx19] RFR: 8286678: Fix mistakes in FX API docs

2022-08-18 Thread Nir Lisker
Fixes the mistakes in the JBS ticket and some additional minor corrections. - Commit messages: - Initial commit - 8290473: update Eclipse .classpath in apps, buildSrc - 8181084: JavaFX show big icons in system menu on macOS with Retina display - 8291908: VirtualFlow creates unneed

Re: RFR: 8290473: update Eclipse .classpath in apps, buildSrc [v11]

2022-08-17 Thread Nir Lisker
On Wed, 17 Aug 2022 22:35:37 GMT, Andy Goryachev wrote: >> If there are folders that aren't generated by the build task, we should note >> it in the build page of the wiki. > > not sure what you are referring to - getLucene task does not create any > folders. no wiki changes are needed. It cr

Re: RFR: 8290473: update Eclipse .classpath in apps, buildSrc [v12]

2022-08-17 Thread Nir Lisker
On Wed, 17 Aug 2022 22:22:02 GMT, Andy Goryachev wrote: >> The goal of this change is to make sure jfx repo can be imported as a gradle >> project in eclipse and all nested projects in the workspace compile with no >> errors. >> >> - updated .classpath entries in apps/ >> - added utf-8 prefs i

Re: RFR: 8290473: update Eclipse .classpath in apps, buildSrc [v12]

2022-08-17 Thread Nir Lisker
On Wed, 17 Aug 2022 22:22:02 GMT, Andy Goryachev wrote: >> The goal of this change is to make sure jfx repo can be imported as a gradle >> project in eclipse and all nested projects in the workspace compile with no >> errors. >> >> - updated .classpath entries in apps/ >> - added utf-8 prefs i

Re: RFR: 8290473: update Eclipse .classpath in apps, buildSrc [v11]

2022-08-17 Thread Nir Lisker
On Wed, 17 Aug 2022 22:17:30 GMT, Kevin Rushforth wrote: >> For lucene, you'd need to run > >> For lucene, you'd need to run >> >> ``` >> /bin/sh gradlew getLucene >> ``` >> >> to bring in the dependencies which are not a part of the repository. > > Btw, `gradle apps` will also work. If ther

Re: RFR: 8290473: update Eclipse .classpath in apps, buildSrc [v8]

2022-08-17 Thread Nir Lisker
On Tue, 9 Aug 2022 21:43:04 GMT, Andy Goryachev wrote: > Thank you all! If there are no more objections, I'd like to get this > integrated, as it will save me from duplicate branches and continuous > cherry-picking. You don't need to do that. Unless 2 branches work on the same code, there will

Re: RFR: 8290473: update Eclipse .classpath in apps, buildSrc [v11]

2022-08-17 Thread Nir Lisker
On Tue, 16 Aug 2022 15:58:38 GMT, Andy Goryachev wrote: >> The goal of this change is to make sure jfx repo can be imported as a gradle >> project in eclipse and all nested projects in the workspace compile with no >> errors. >> >> - updated .classpath entries in apps/ >> - added utf-8 prefs i

Re: RFR: 8217853: Cleanup in the D3D native pipeline [v2]

2022-08-17 Thread Nir Lisker
On Mon, 15 Aug 2022 10:03:11 GMT, Nir Lisker wrote: >> Command to run the system test: >> `gradle -PFULL_TEST=true -PUSE_ROBOT=true systemTests:test --tests >> test.robot.test3d.PointLightIlluminationTest` >> >> I ran all the tests, only above test failed: &g

Re: [External] : Re: Missing 'vcruntime.h'

2022-08-16 Thread Nir Lisker
Nodejs, Rust, F sharp, Selenium... those are indeed some questionable entries for JavaFX. On Tue, Aug 16, 2022 at 6:25 PM John Neffenger wrote: > On 8/15/22 10:18 PM, Nir Lisker wrote: > > I also have entries there that don't exist. I suspect they have to do > > with buildi

Re: [External] : Re: Missing 'vcruntime.h'

2022-08-15 Thread Nir Lisker
se I would have. On Tue, Aug 16, 2022 at 1:26 AM John Neffenger wrote: > On 8/15/22 12:32 AM, Nir Lisker wrote: > > (I have '%SystemRoot%\system32' which is exported to Cygwin correctly). > > That works, too. I have to admit, my troubles on Windows at the time > we

Re: RFR: 8217853: Cleanup in the D3D native pipeline [v2]

2022-08-15 Thread Nir Lisker
On Tue, 2 Aug 2022 10:57:25 GMT, Ambarish Rapte wrote: >> I'll have a look. What command did you run this test with? >> >> Can you also test the point light attenuation under >> `tests\system\src\test\java\test\javafx\scene\lighting3D\PointLightAttenuationTest.java`? >> (I think there's a bit

Re: [External] : Re: Missing 'vcruntime.h'

2022-08-15 Thread Nir Lisker
x27;%SystemRoot%\system32' which is exported to Cygwin correctly). The properties file was still wrong and caused errors. I also run gradle builds from Eclipse, so Cygwin isn't always used. On Sun, Aug 14, 2022 at 8:13 PM John Neffenger wrote: > On 8/13/22 7:29 AM, Nir Lisker

Re: [External] : Re: Missing 'vcruntime.h'

2022-08-13 Thread Nir Lisker
ER or VSINSTALLDIR). > > VS150COMNTOOLS: "C:\\Program Files\\Microsoft Visual > Studio\\2022\\Enterprise\\VC\\Auxiliary\\Build" > > -- Kevin > > > On 8/12/2022 8:06 PM, Nir Lisker wrote: > > Kevin, can you update the windows_tools.properties file for the b

Re: Missing 'vcruntime.h'

2022-08-12 Thread Nir Lisker
Kevin, can you update the windows_tools.properties file for the build page on the wiki [1] for VS 2022 since I see that we updated to use that version? [1] https://wiki.openjdk.org/pages/viewpage.action?pageId=66650334 On Sat, Aug 13, 2022 at 2:06 AM Nir Lisker wrote: > I see the file un

Re: Missing 'vcruntime.h'

2022-08-12 Thread Nir Lisker
t; -- Kevin > > > On 8/12/2022 2:12 PM, Nir Lisker wrote: > > Hi, > > > > I have just run gradle clean and gradle build, and I'm getting an error: > > > > C:/Program Files (x86)/Windows > > Kits/10/include/10.0.17763.0/ucrt\corecrt.h(10): fata

Missing 'vcruntime.h'

2022-08-12 Thread Nir Lisker
lder. Do I need to update something, like VS? - Nir

[jira] [Updated] (FLINK-28933) Support ON CONFLICT DO NOTHING in flink-jdbc-connector for Postgresql dialect

2022-08-11 Thread Nir Tsruya (Jira)
[ https://issues.apache.org/jira/browse/FLINK-28933?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nir Tsruya updated FLINK-28933: --- Summary: Support ON CONFLICT DO NOTHING in flink-jdbc-connector for Postgresql dialect (was

[jira] [Created] (FLINK-28933) Support ON CONFLICT DO NOTHING in flink-jdbc-connector

2022-08-11 Thread Nir Tsruya (Jira)
Nir Tsruya created FLINK-28933: -- Summary: Support ON CONFLICT DO NOTHING in flink-jdbc-connector Key: FLINK-28933 URL: https://issues.apache.org/jira/browse/FLINK-28933 Project: Flink Issue

[jira] [Created] (FLINK-28933) Support ON CONFLICT DO NOTHING in flink-jdbc-connector

2022-08-11 Thread Nir Tsruya (Jira)
Nir Tsruya created FLINK-28933: -- Summary: Support ON CONFLICT DO NOTHING in flink-jdbc-connector Key: FLINK-28933 URL: https://issues.apache.org/jira/browse/FLINK-28933 Project: Flink Issue

Re: RFR: 8217853: Cleanup in the D3D native pipeline [v5]

2022-08-10 Thread Nir Lisker
On Tue, 2 Aug 2022 18:01:06 GMT, Ambarish Rapte wrote: >> Nir Lisker has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Renamed method > > modules/javafx.graphics/src/main/native-prism-d3d/hlsl/vs2ps.h

Re: RFR: 8290473: update Eclipse .classpath in apps, buildSrc [v6]

2022-08-09 Thread Nir Lisker
On Tue, 9 Aug 2022 15:11:49 GMT, Kevin Rushforth wrote: >> This is up to Kevin. It's changing the build file specifically to remove a >> warning for Eclipse Mac users. I don't mind. Might need to do the same for >> the other OS-specific folders. > > While not ideal, I suppose it's OK. I'm still

<    3   4   5   6   7   8   9   10   11   12   >