Hi Petr,
The fix looks fine. I see that we're not checking the isSWTInWebStart()
case (see awt.m), but I don't immediately see any way to check this
actually, because the JVM isn't started yet when the splashscreen is
initializing. So I think we're good with the current solution. Although
it would be good to document somewhere (in a comment in the code?) that
when running an SWT app in WebStart, the AWT splashscreen is unsupported.
--
best regards,
Anthony
On 4/11/2014 3:14 PM, Petr Pchelko wrote:
Hello, AWT Team.
Please review the fix for the issue:
https://bugs.openjdk.java.net/browse/JDK-8039926
The fix is available at:
http://cr.openjdk.java.net/~pchelko/9/8039926/webrev.00/
When running SWT we should not start the main event loop for splashscreen
because it breaks SWT.
The only way understand that in splashscreen code is to check the system
property launcher sets on
the startup. We do the same in awt.m, but we cannot reuse the code because
splash and awt are not
statically linked.
With best regards. Petr.
