Re: RFR: 8234920: Add SpotLight to the selection of 3D light types [v22]

2021-06-13 Thread Nir Lisker
On Fri, 11 Jun 2021 12:54:43 GMT, Ambarish Rapte wrote: >> Nir Lisker has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Addressed review comments > > modules/javafx.graphics/src/main/native-prism-d3d/D3DLight.cc line 50: > >> 48: >> 49:

Re: RFR: 8234920: Add SpotLight to the selection of 3D light types [v22]

2021-06-13 Thread Ambarish Rapte
On Fri, 11 Jun 2021 01:30:27 GMT, Nir Lisker wrote: >> Added a SpotLight only to the D3D pipeline currently. >> >> ### API discussion points >> >> - [X] Added `SpotLight` as a subclass of `LightBase`. However, it could >> also be a subclass of `PointLight` as it's a point light with

Re: RFR: 8234920: Add SpotLight to the selection of 3D light types [v22]

2021-06-11 Thread Kevin Rushforth
On Fri, 11 Jun 2021 01:30:27 GMT, Nir Lisker wrote: >> Added a SpotLight only to the D3D pipeline currently. >> >> ### API discussion points >> >> - [X] Added `SpotLight` as a subclass of `LightBase`. However, it could >> also be a subclass of `PointLight` as it's a point light with

Re: RFR: 8234920: Add SpotLight to the selection of 3D light types [v22]

2021-06-10 Thread Nir Lisker
> Added a SpotLight only to the D3D pipeline currently. > > ### API discussion points > > - [X] Added `SpotLight` as a subclass of `LightBase`. However, it could also > be a subclass of `PointLight` as it's a point light with direction and extra > factors. I saw that