On Fri, 3 Jan 2020 09:26:43 GMT, Ambarish Rapte wrote:
>> The pull request has been updated with 1 additional commit.
>
> I have added few comments, but have not run tests and sample yet.
> I still need to test your sample app on Mac.
I get the error with your sample app. It fails on Mac or
On Fri, 3 Jan 2020 19:32:23 GMT, Nir Lisker wrote:
>> modules/javafx.graphics/src/main/java/javafx/scene/PointLight.java line 41:
>>
>>> 40: * unless it belongs to a {@code Shape3D} outside of its {@code scope}.
>>> 41: *
>>> 42: * The light's intensity can be set to decrease over distance
On Fri, 3 Jan 2020 09:25:21 GMT, Ambarish Rapte wrote:
>> The pull request has been updated with 1 additional commit.
>
> modules/javafx.graphics/src/main/native-prism-d3d/D3DMeshView.cc line 175:
>
>> 174: lightsAttenuation[a++] = lights[i].attenuation[2];
>> 175:
On Fri, 3 Jan 2020 09:21:32 GMT, Ambarish Rapte wrote:
>> The pull request has been updated with 1 additional commit.
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/sg/prism/NGPointLight.java
> line 64:
>
>> 63:
>> 64: private double ca = DEFAULT_CA;
>> 65:
>
> The
On Fri, 3 Jan 2020 05:17:26 GMT, Ambarish Rapte wrote:
>> The pull request has been updated with 1 additional commit.
>
> modules/javafx.graphics/src/main/java/javafx/scene/PointLight.java line 41:
>
>> 40: * unless it belongs to a {@code Shape3D} outside of its {@code scope}.
>> 41: *
>>
On Fri, 3 Jan 2020 05:08:33 GMT, Ambarish Rapte wrote:
>> The pull request has been updated with 1 additional commit.
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/sg/prism/NGPointLight.java
> line 43:
>
>> 42: private static final double DEFAULT_MAX_RANGE =
>>
> CSR: https://bugs.openjdk.java.net/browse/JDK-8218264
The pull request has been updated with 1 additional commit.
-
Added commits:
- f66e8cf4: Addressing review comments
Changes:
- all: https://git.openjdk.java.net/jfx/pull/43/files
- new: