Re: [9] Review request: 8172526: Third-party licenses for javafx.* modules are missing

2017-02-13 Thread Mandy Chung
+1 Mandy > On Feb 13, 2017, at 5:30 PM, Kevin Rushforth > wrote: > > Fixed. Thanks. > > http://cr.openjdk.java.net/~kcr/8172526/webrev.01/ > > > -- Kevin > > > Mandy Chung wrote: >> >>> On Feb 13, 2017, at 1:50 PM, Kevin Rushforth >>

Re: [9] Review request: 8172526: Third-party licenses for javafx.* modules are missing

2017-02-13 Thread Kevin Rushforth
Fixed. Thanks. http://cr.openjdk.java.net/~kcr/8172526/webrev.01/ -- Kevin Mandy Chung wrote: On Feb 13, 2017, at 1:50 PM, Kevin Rushforth wrote: Dave & Mandy, Please review the following fix to deliver the JavaFX third-party license content into the JDK build. https://bugs.openjdk.java.

Re: [9] Review request: 8172526: Third-party licenses for javafx.* modules are missing

2017-02-13 Thread Mandy Chung
> On Feb 13, 2017, at 1:50 PM, Kevin Rushforth > wrote: > > Dave & Mandy, > > Please review the following fix to deliver the JavaFX third-party license > content into the JDK build. > > https://bugs.openjdk.java.net/browse/JDK-8172526 > http://cr.openjdk.java.net/~kcr/8172526/webrev.00/ ja

[9] Review request: 8172526: Third-party licenses for javafx.* modules are missing

2017-02-13 Thread Kevin Rushforth
Dave & Mandy, Please review the following fix to deliver the JavaFX third-party license content into the JDK build. https://bugs.openjdk.java.net/browse/JDK-8172526 http://cr.openjdk.java.net/~kcr/8172526/webrev.00/ Thanks. -- Kevin

[9] Review request for 8174866: User Agent String for JavaFX is not correct

2017-02-13 Thread Murali Billa
  Hi Kevin, Arun, Guru Please review the simple fix for ­ JDK-8174866 JIRA: https://bugs.openjdk.java.net/browse/JDK-8174866   Webrev: http://cr.openjdk.java.net/~mbilla/8174866/webrev.00/   Thanks, Murali

[9] Review request: JDK-8174842 Uninitialized variable in PosixPlatform.cpp

2017-02-13 Thread Chris Bensen
Victor, Please review the fix for initializing the uninitialized variable. JIRA: https://bugs.openjdk.java.net/browse/JDK-8174842 Webrev: http://cr.openjdk.java.net/~cbensen/JDK-8174842/webrev.00/