On Mon, 27 Jan 2025 17:28:14 GMT, Andy Goryachev wrote:
>> I'm not familiar with these methods, but I did wonder why we need extra
>> implementations in the initial comment.
>
> looks like dead code left over from the original implementation.
>
> I removed the class along with the two methods a
On Mon, 27 Jan 2025 16:56:21 GMT, Nir Lisker wrote:
>> modules/javafx.graphics/src/main/java/com/sun/javafx/geom/Dimension2Df.java
>> line 34:
>>
>>> 32: * @see javafx.geometry.Dimension2D Dimension2D - doubles
>>> 33: */
>>> 34: public record Dimension2Df(float width, float height) {}
>>
>>
On Mon, 27 Jan 2025 16:36:28 GMT, Andy Goryachev wrote:
>> Nir Lisker has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Address review comments
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/geom/Dimension2Df.java
> line 34:
>
On Sat, 25 Jan 2025 07:04:41 GMT, Nir Lisker wrote:
>> A small refactoring of the Dimension classes.
>>
>> * `com.sun.javafx.geom.Dimension` was removed and its uses were replaced by
>> `com.sun.javafx.geom.Dimension2D`.
>> * `com.sun.javafx.geom.Dimension2D` became a record.
>> * `javafx.geome
On Mon, 2 Dec 2024 19:45:47 GMT, Kevin Rushforth wrote:
>> modules/javafx.graphics/src/main/java/com/sun/javafx/geom/Dimension2D.java
>> line 31:
>>
>>> 29: * A 2D dimension object that contains a width and a height.
>>> 30: */
>>> 31: public record Dimension2D(float width, float height) {}
>
On Mon, 2 Dec 2024 19:13:20 GMT, Andy Goryachev wrote:
>> Nir Lisker has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Address review comments
>
> modules/javafx.graphics/src/main/java/com/sun/glass/ui/gtk/screencast/TokenStorage.java
> l
> A small refactoring of the Dimension classes.
>
> * `com.sun.javafx.geom.Dimension` was removed and its uses were replaced by
> `com.sun.javafx.geom.Dimension2D`.
> * `com.sun.javafx.geom.Dimension2D` became a record.
> * `javafx.geometry.Dimension2D`: fields became `final`.
>
> I'm not sure w
