On Wed, 23 Apr 2025 12:13:52 GMT, Jayathirth D V <j...@openjdk.org> wrote:
> When no specular color is set while rendering 3D primitives, 0.0 specular > power value is used by default in our shaders. > When same specular power value is used in pow() function in shader it results > in undefined behaviour as mentioned at : > https://registry.khronos.org/OpenGL-Refpages/es3.0/html/pow.xhtml > > By default specular power value should be 32, so now the specular_none.frag > file is updated to use this default value to make sure we don't see 3D > lighting issues on some platforms. This change is tested with Ensemble8 and > fx83dfeatures and i don't see any regressions. > > Also one of our system test _PointLightIlluminationTest_ used to fail because > of this issue. It passes now with this update and it is re-enabled. LGTM. Tested on mac and windows(with es2 pipeline) platform. Shall rely @jayathirthrao for Linux sanity testing. Please wait for 24 hours before integrating. ------------- Marked as reviewed by arapte (Reviewer). PR Review: https://git.openjdk.org/jfx/pull/1791#pullrequestreview-2787441473 PR Comment: https://git.openjdk.org/jfx/pull/1791#issuecomment-2824412363
