8u20 review request: RT-35210 - exceptions in WebLauncher on Mac

Jim Graham james.graham at oracle.com
Wed Jan 8 18:07:42 PST 2014


Adding Johan Voss to the list of reviewers since this fix may 
potentially involve an Android #ifdef block.

Johan, the webrev currently listed in the Jira issue below does not yet 
modify the Android block, but I am proposing eliminating the block since 
I think it was a workaround for the bad call to glValidateProgram that 
we are eliminating here (that bad call started causing problems for us 
as well in a way that I'm guessing it once caused for the Android port 
which led to the workaround hack).

Does the Android port still use this code (i.e. does it still use 
prism-es2)?  If so, what do you think about eliminating the hack at 
lines 609-611 (old version) => 586-588 (new version)?

			...jim

On 1/8/14 5:12 PM, Jim Graham wrote:
> Chien, Felipe,
>
> Jira: https://javafx-jira.kenai.com/browse/RT-35210
> webrev: http://cr.openjdk.java.net/~flar/RT-35210/webrev.00/
>
> I will finish the cleanup of the "shader log" logic (RT-35209) as a
> follow-on fix after this is pushed...
>
>              ...jim


More information about the openjfx-dev mailing list