Resending with correct Bug id in Subject line.
> On 11-Oct-2017, at 6:57 PM, Guru Hb wrote:
>
> Hi Arun, Murali & Kevin,
>
> Please review the fix for
> JBS : https://bugs.openjdk.java.net/browse/JDK-8187568
> <https://bugs.openjdk.java.net/browse
Hi Arun, Murali & Kevin,
Please review the fix for
JBS : https://bugs.openjdk.java.net/browse/JDK-8187568
<https://bugs.openjdk.java.net/browse/JDK-8187568>
Webrev: http://cr.openjdk.java.net/~ghb/8187568/webrev.00/
<http://cr.openjdk.java.net/~ghb/8187568/webrev.00/>
RC and
Note that there is now a 04 version.
It looks good to me, although someone more familiar with AWT should also
check the AWT changes.
We will need a test program for this (as a follow-on issue if not now).
-- Kevin
Alexander Zvegintsev wrote:
Please review the updated version
http
Please review the updated version
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/02/
Now we are postponing actual window closing, it happens only after we
ensure that native window pointer is valid on Swing side.
Thanks,
Alexander.
On 23/09/2017 08:01, Sergey Bylokhov wrote:
Hi
Hi Kevin and David,
Please review the following:
https://bugs.openjdk.java.net/browse/JDK-8181922
Added support for libav version 57.
Thanks,
Alexander
Hi Kevin, Arun
Please review the below simple fix.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8187366
Webrev: http://cr.openjdk.java.net/~mbilla/8187366/webrev.01/
Thanks,
Murali
-8080729 for more details.
>>
>> To avoid possible problems in multi monitor set-up I suggest that you should
>> replace addChildWindow() with orderWindow() or use some another approach to
>> locate the window above the stage.
>>
>> Thanks,
>> Dmitry
>
Alexander Zvegintsev wrote:
Hi Phil,
Please review the updated fix with reflection incorporated
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/
New issue created JDK-8187803
<https://bugs.openjdk.java.net/browse/JDK-8187803> as JDK counterpart
of this issue.
Thanks,
Alexander.
On
= peer.getOverridenWindowHandle();
<<<<< Dispose the peer
if (ownerWindowPtr != 0) {
//Place window above JavaFX stage
CWrapper.NSWindow.addChildWindow(
ownerWindowPtr, ptr, CWrapper.NSWindow.NSWindowAbove);
<<<<< Boom
}
On 9/21/17 22:56, Alexander Zvegintsev wrote:
Hi Phil,
lti monitor set-up I suggest that you should
replace addChildWindow() with orderWindow() or use some another approach to
locate the window above the stage.
Thanks,
Dmitry
On 22 Sep 2017, at 06:56, Alexander Zvegintsev
wrote:
Hi Phil,
Please review the updated fix with reflection in
problems in multi monitor set-up I suggest that you should
replace addChildWindow() with orderWindow() or use some another approach to
locate the window above the stage.
Thanks,
Dmitry
> On 22 Sep 2017, at 06:56, Alexander Zvegintsev
> wrote:
>
> Hi Phil,
>
> Please review th
.addChildWindow(
ownerWindowPtr, ptr, CWrapper.NSWindow.NSWindowAbove);
<<<<< Boom
}
On 9/21/17 22:56, Alexander Zvegintsev wrote:
Hi Phil,
Please review the updated fix with reflection incorporated
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/
New issue created JDK-8187803
10:56 PM, Alexander Zvegintsev wrote:
Hi Phil,
Please review the updated fix with reflection incorporated
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/
New issue created JDK-8187803
<https://bugs.openjdk.java.net/browse/JDK-8187803> as JDK counterpart
of this issue.
Thanks,
Al
Hi Phil,
Please review the updated fix with reflection incorporated
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/
New issue created JDK-8187803
<https://bugs.openjdk.java.net/browse/JDK-8187803> as JDK counterpart of
this issue.
Thanks,
Alexander.
On 21/09/2017 22:25, Phi
2017 03:38 AM, Alexander Zvegintsev wrote:
Hello,
please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8185634
Hello,
please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8185634
--
Thanks,
Alexander.
Hi all,
Please review a patch for fixing Copy and Paste functionality in FX WebView.
Details in JBS.
JBS : https://bugs.openjdk.java.net/browse/JDK-8187726
WebRev : http://cr.openjdk.java.net/~ghb/rkamath/8187726/webrev.00/
Thanks,
Rajath
Hi all,
Please review a bug fix done for `Drag and Drop use case in WebView` explained
in the JBS
JBS : https://bugs.openjdk.java.net/browse/JDK-8187671
WebRev : http://cr.openjdk.java.net/~ghb/rkamath/8187671/webrev.00/
Thanks,
Rajath
Hi Jonathan,
Request you to review following fix :
Issue : https://bugs.openjdk.java.net/browse/JDK-8089265
Fix : http://cr.openjdk.java.net/~aghaisas/fx/8089265/webrev.0/
Regards,
Ajit
8087528 has been reverted ok.
+1.
Regards
Prasanta
On 9/8/2017 11:28 AM, Guru Hb wrote:
Hi Murali & Arun,
Please do the review for
JBS : https://bugs.openjdk.java.net/browse/JDK-8187361
<https://bugs.openjdk.java.net/browse/JDK-8187361>Webrev :
http://cr.openjdk.java.net/~g
Hi Murali & Arun,
Please do the review for
JBS : https://bugs.openjdk.java.net/browse/JDK-8187361
<https://bugs.openjdk.java.net/browse/JDK-8187361>Webrev :
http://cr.openjdk.java.net/~ghb/8187361/webrev.00/
<http://cr.openjdk.java.net/~ghb/8187361/webrev.00/>
Thanks,
Guru
Hi all,
Please review the patch for enabling HTML5 form validation. With this enabled,
WebKit would by default show validation specific error message dialogs using
Shadow DOM + CSS styled web elements.
This will serve as a placeholder until the FX platform implementation of
Please review the following fix for a critical regression introduced in
10 (and 9.x) due to the VS 2017 compiler upgrade that was done as part
of the recent WebKit update:
https://bugs.openjdk.java.net/browse/JDK-8187043
http://cr.openjdk.java.net/~kcr/8187043/webrev.00/
-- Kevin
Victor,
Please review this change to upgrade the native projects to the latest version
of VS 2017:
JIRA: https://bugs.openjdk.java.net/browse/JDK-8187059
Webrev: http://cr.openjdk.java.net/~cbensen/JDK-8187059/webrev.00/
Chris
Hi Arun & Murali,
Please review the fix for :
JBS : https://bugs.openjdk.java.net/browse/JDK-8170955
<https://bugs.openjdk.java.net/browse/JDK-8170955>
webrev : http://cr.openjdk.java.net/~ghb/8170955/webrev.00/
<http://cr.openjdk.java.net/~ghb/8170955/webrev.00/>
Thanks,
Guru
Hi Guru\Arun,
Please review this simple cleanup of unused third-party library download
scripts.
JBS : https://bugs.openjdk.java.net/browse/JDK-8185314
WebRev : http://cr.openjdk.java.net/~ghb/rkamath/8185314/webrev.00/
Thanks,
Rajath
you should close it as a dup. and return to the previous bug ?
>
> -phil.
>
> On 8/18/17, 2:05 AM, Manajit Halder wrote:
>> Hi Phil,
>>
>> Please review the fix.
>>
>> Regards,
>> Manajit
>>
>>> On 08-Aug-2017, at 4:52 PM,
17, 2:05 AM, Manajit Halder wrote:
Hi Phil,
Please review the fix.
Regards,
Manajit
On 08-Aug-2017, at 4:52 PM, Manajit Halder wrote:
Hi All,
Kindly review the following JavaFx fix.
Bug:
https://bugs.openjdk.java.net/browse/JDK-8185954<https://bugs.openjdk.java.net/browse/JDK-8185954
Hi Phil,
Please review the fix.
Regards,
Manajit
> On 08-Aug-2017, at 4:52 PM, Manajit Halder wrote:
>
> Hi All,
>
> Kindly review the following JavaFx fix.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8185954
> <https://bugs.openjdk.java.net/browse/
you to review.
Regards,
Ajit
Victor,
Please review this refactor to get ready for the new CLI.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8186237
Webrev: http://cr.openjdk.java.net/~cbensen/JDK-8186237/webrev.00/
Chris
Ajit,
I have provided some comments in the bug.
On 8/10/17 8:45 AM, Ajit Ghaisas wrote:
Hi Jonathan,
Request you to review following change :
Issue : https://bugs.openjdk.java.net/browse/JDK-8090462
Fix : http://cr.openjdk.java.net/~aghaisas/fx/e8090462/webrev.2/
Regards
Hi Jonathan,
Request you to review following change :
Issue : https://bugs.openjdk.java.net/browse/JDK-8090462
Fix : http://cr.openjdk.java.net/~aghaisas/fx/e8090462/webrev.2/
Regards,
Ajit
Hi Arun, Murali,
Please review the fix for https://bugs.openjdk.java.net/browse/JDK-8186061
<https://bugs.openjdk.java.net/browse/JDK-8186061>
webrev : http://cr.openjdk.java.net/~ghb/8186061/webrev.00/
<http://cr.openjdk.java.net/~ghb/8186061/webrev.00/>
Thanks,
Guru
Hi,
Please review the below fix.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8185970
Webrev: http://cr.openjdk.java.net/~mbilla/8185970/webrev.00/
Thanks,
Murali
I've scanned through it, and it looks generally OK to me.
-- Jon
On 08/03/2017 05:33 PM, Jonathan Giles wrote:
Jon, Kevin and / or Phil,
Please review the webrev below. It improves the JavaFX javadocs such
that there are (hopefully) no bad links any longer.
https://bugs.openjdk.jav
Hi All,
Kindly review the following JavaFx fix.
Bug: https://bugs.openjdk.java.net/browse/JDK-8185954
<https://bugs.openjdk.java.net/browse/JDK-8185954>
Webrev: http://cr.openjdk.java.net/~mhalder/fx/8185954/webrev.02/
<http://cr.openjdk.java.net/~mhalder/fx/8185954/webrev.02/>
No
Hi,
Please review the below simple fix.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8182977
Webrev: http://cr.openjdk.java.net/~mbilla/8182977/webrev.00/
Thanks,
Murali
Hi,
Please review the below simple fix.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8183928
Webrev: http://cr.openjdk.java.net/~mbilla/8183928/webrev.00/
Thanks,
Murali
Hi,
Please review the simple fix for the following bug,
https://bugs.openjdk.java.net/browse/JDK-8185804
Detailed analysis and fix is updated as JBS comment.
Thanks,
Arun
Please review the webrev below which tries to fix the issue of GetApplication
processed for Null values. Any alternatives or suggestions are welcome. This
fixes the issue presented by the description.
https://bugs.openjdk.java.net/browse/JDK-8176813
<https://bugs.openjdk.java.net/browse/
Jon, Kevin and / or Phil,
Please review the webrev below. It improves the JavaFX javadocs such
that there are (hopefully) no bad links any longer.
https://bugs.openjdk.java.net/browse/JDK-8185767
http://cr.openjdk.java.net/~jgiles/8185767/
Thanks,
--
-- Jonathan
Hi Kevin, Hi Jonathan,
Request you to review the fix,
Bug: https://bugs.openjdk.java.net/browse/JDK-8092206
Webrev: http://cr.openjdk.java.net/~arapte/fx/8092206/webrev.04/
Regards,
Ambarish
Kevin, Phil,
Could you please review the webrev below, which aims to provide new
public API for focus traversal in JavaFX.
https://bugs.openjdk.java.net/browse/JDK-8091673
http://cr.openjdk.java.net/~jgiles/8091673.1
--
-- Jonathan
Kevin, Phil,
Could you please review the webrev below, which aims to remove the
unused two-level focus code that presently ships in JDK 9 and earlier
releases.
https://bugs.openjdk.java.net/browse/JDK-8178519
http://cr.openjdk.java.net/~jgiles/8178519/
Thanks,
--
-- Jonathan
Hi Chien, Hi Kevin,
Request you to review the fix for JDK 10.
Bug: https://bugs.openjdk.java.net/browse/JDK-8181930
Webrev: http://cr.openjdk.java.net/~arapte/fx/8181930/webrev.00/
Regards,
Ambarish
Hi Kevin and David,
Please review the following:
https://bugs.openjdk.java.net/browse/JDK-8185691
Fixed issue when error was send instead of EOS for HTTP Live Streaming.
Thanks,
Alexander
JBS: https://bugs.openjdk.java.net/browse/JDK-8183530
webrev: http://cr.openjdk.java.net/~flar/JDK-8183530/webrev.03/
...jim
Hi,
Please review the below fix.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8184453
Webrev: http://cr.openjdk.java.net/~mbilla/8184453/webrev.00/
Thanks,
Murali
Hi Jonathan, Hi Ajit,
Request you to review this fix:
Bug: https://bugs.openjdk.java.net/browse/JDK-8130904
Webrev: http://cr.openjdk.java.net/~arapte/fx/8130904/webrev.00/
Regards,
Ambarish
Chris,
Please review my changes about deprecating -createbss and -createjar.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8148265
Webrev:http://cr.openjdk.java.net/~vdrozdov/JDK-8148265/webrev.00/
--Victor
Hi,
Please review the following changset,
Webrev: http://cr.openjdk.java.net/~arajkumar/8184448/webrev.01
Bug: https://bugs.openjdk.java.net/browse/JDK-8184448
Detailed analysis is updated as JBS comment.
Thanks,
Arun
Hi,
Please review the following changset,
Webrev: http://cr.openjdk.java.net/~arajkumar/8185132/webrev
Bug: https://bugs.openjdk.java.net/browse/JDK-8185132
Detailed analysis is updated as JBS comment.
Thanks,
Arun
Hi Kevin, Arun,
Please review the below simple fix.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8185138
Webrev: http://cr.openjdk.java.net/~mbilla/8185138/webrev.00/
Thanks,
Murali
/main/native", it will be useful only for review.
2. rt.changeset.gz — Actual changeset file in compressed format which contains
all the changes from “rt” directory, uncompress before using it(gunzip
rt.changeset.gz) and do the following steps.
$ cd rt
$ hg import --no-commit rt.change
Chris,
Please review my changes about adding single instance swing app for ant
version
JIRA: https://bugs.openjdk.java.net/browse/JDK-8184996
Webrev:http://cr.openjdk.java.net/~vdrozdov/JDK-8184996/webrev.00/
<http://cr.openjdk.java.net/%7Evdrozdov/JDK-8184996/webrev.00/>
--Victor
+1
Although I am puzzled why I did not hit this before.
-phil.
On 7/20/17, 4:34 PM, Chris Bensen wrote:
Phil,
Please review this fix to remove the bad characters.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8185017
Webrev: http://cr.openjdk.java.net/~cbensen/JDK-8185017/webrev.00/
Chris
Phil,
Please review this fix to remove the bad characters.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8185017
Webrev: http://cr.openjdk.java.net/~cbensen/JDK-8185017/webrev.00/
Chris
Hello,
please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/10/8181779/00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8181779
--
Thanks,
Alexander.
Chris,
Please review my changes about adding single instance example.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8184688
Webrev: http://cr.openjdk.java.net/~vdrozdov/JDK-8184688/webrev.00/
--Victor
Hi Arun & Murali,
Please review the fix for:
JBS : https://bugs.openjdk.java.net/browse/JDK-8184797
<https://bugs.openjdk.java.net/browse/JDK-8184797>
Webrev : http://cr.openjdk.java.net/~ghb/8184797/webrev.00/
<http://cr.openjdk.java.net/~ghb/8184797/webrev.00/>
Thanks,
Guru
Victor,
Please review this change to remove the LoadLibray.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8184769
Webrev: http://cr.openjdk.java.net/~cbensen/JDK-8184769/webrev.00/
Chris
Chris,
Please review my changes about copying icon to app image.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8183240
Webrev: http://cr.openjdk.java.net/~vdrozdov/JDK-8183240/webrev.00/
--Victor
Chris,
Please review my changes about adding new projects for
jdk.packager.services and jdk.packager
JIRA: https://bugs.openjdk.java.net/browse/JDK-8184301
Webrev: http://cr.openjdk.java.net/~vdrozdov/JDK-8184301/webrev.00
--Victor
Chris,
Please review my changes about removing getSessionSpecificString()
JIRA: https://bugs.openjdk.java.net/browse/JDK-8184077
Webrev: http://cr.openjdk.java.net/~vdrozdov/JDK-8184077/webrev.0/
--Victor
Hi Jonathan,
Please review the below fix,
Bug: https://bugs.openjdk.java.net/browse/JDK-8092206
Webrev: http://cr.openjdk.java.net/~arapte/fx/8092206/webrev.00/
Regards,
Ambarish
Victor,
Please review this change to remove the dead code related to acquiring the
system JRE for a packaged app.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8183246
Webrev: http://cr.openjdk.java.net/~cbensen/JDK-8183246/webrev.00/
Chris
Hi Jonathan,
Request you to review following change :
Issue : https://bugs.openjdk.java.net/browse/JDK-8177636
Fix : http://cr.openjdk.java.net/~aghaisas/fx/e8177636/webrev.1/
Regards,
Ajit
Kevin, Chris,
Please review my changes about Single Instance for Java Packager
(compared to previous webrev, the code was modified according to CSR
JDK-8181293)
JIRA: https://bugs.openjdk.java.net/browse/JDK-8181291
Webrev: http://cr.openjdk.java.net/~vdrozdov/JDK-8175574/webrev.06
Chris,
Please review my changes about correcting registry paths
JIRA: https://bugs.openjdk.java.net/browse/JDK-8182615
Webrev: http://cr.openjdk.java.net/~vdrozdov/JDK-8182615/webrev.00/
--Victor
Victor,
Please review these changes related to the TODOs and FIXMEs in the java
packager code.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8091418
Webrev: http://cr.openjdk.java.net/~cbensen/JDK-8091418/webrev.00/
Chris
Hi All,
Please review the modified fix which is more efficient than the initial fix:
http://cr.openjdk.java.net/~mhalder/fx/8176319/webrev.01/
<http://cr.openjdk.java.net/~mhalder/fx/8176319/webrev.01/>
Regards,
Manajit
> On 30-Jun-2017, at 3:13 PM, Manajit Halder wrote:
&g
Hi All,
Kindly review the following JavaFx fix.
Bug: https://bugs.openjdk.java.net/browse/JDK-8176319
<https://bugs.openjdk.java.net/browse/JDK-8176319>
Webrev: http://cr.openjdk.java.net/~mhalder/fx/8176319/webrev.00/
<http://cr.openjdk.java.net/~mhalder/fx/8176319/webrev.00/&
Hi Kevin, Arun and Murali,
Please review the fix for :
JBS : https://bugs.openjdk.java.net/browse/JDK-8178360
<https://bugs.openjdk.java.net/browse/JDK-8178360>
Webrev : http://cr.openjdk.java.net/~ghb/8178360/webrev.07
<http://cr.openjdk.java.net/~ghb/8178360/webrev.07> (Modified S
Kevin, Victor,
Please review these additions to the Java Packager debug JDK. This change will
bundle the jdk.packager.services module with the resulting application and
cleans up the gradle sourceSet.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8182778
Webrev: http://cr.openjdk.java.net
Hi Jonathan,
Request you to review following fix:
Bug: https://bugs.openjdk.java.net/browse/JDK-8167139
Webrev: http://cr.openjdk.java.net/~pkbalakr/fx/8167139/webrev.00/
Regards,
Prem
Hi,
Chris, Victor,
Please review the following change for fixing the “stripComments” method for
redistributableModules.
Bug URL: https://bugs.openjdk.java.net/browse/JDK-8179399
<https://bugs.openjdk.java.net/browse/JDK-8179399>
Webrev: http://cr.openjdk.java.net/~vrao/JDK-8179399/
Kevin, Victor,
Please review this change for unit tests to create a debug JDK. It’s a little
odd. It only works in a bash shell because java and javac are copied to
java.original and javac.original and replaced with bash scripts which always
supply the --upgrade-module-path containing the
Kevin, Victor,
Please review this change following JDK-8179445 to remove the qualified exports
of com.sun.tools.jdeps:
<https://bugs.openjdk.java.net/browse/JDK-8179445>
JIRA: https://bugs.openjdk.java.net/browse/JDK-JDK-8181738
Webrev: http://cr.openjdk.java.net/~cbensen/JDK-JDK-8181738/
e outdated "faq.html" is
removed.
http://cr.openjdk.java.net/~kcr/8180338/webrev.02/
-- Kevin
Kevin Rushforth wrote:
Hi Jon,
Can you review this simple change to the three .html files (other than
package.html) in the FX docs?
https://bugs.openjdk.java.net/browse/JDK-8180338
http://cr.o
JBS: https://bugs.openjdk.java.net/browse/JDK-8181976
webrev: http://cr.openjdk.java.net/~flar/JDK-8181976/webrev.00/
Simple fix is to carry the double "size requested" values all the way down to where the image pixel scale is determined
(not strictly required, but important for precision) and t
Chris, Kevin
Please review my changes about jdeps ToolProvider
JIRA: https://bugs.openjdk.java.net/browse/JDK-8179445
Webrev: http://cr.openjdk.java.net/~vdrozdov/JDK-8179445/webrev.00/
--Victor
Phil or Jonathan,
Please review the simple fix for a broken javadoc link:
https://bugs.openjdk.java.net/browse/JDK-8181829
The diff for the proposed fix, which is also in JBS, is:
diff --git
a/modules/javafx.graphics/src/main/java/javafx/scene/layout/Background.java
b/modules
Jim or Phil,
Please review the simple fix to move tangent_interpolator.png to the
correct module.
https://bugs.openjdk.java.net/browse/JDK-8181828
The diff for the proposed fix, which is also in JBS, is
Kevin, Victor,
Please review
JIRA: https://bugs.openjdk.java.net/browse/JDK-8177696
Webrev: http://cr.openjdk.java.net/~cbensen/JDK-8177696/webrev.02/
Chris
Hi Jonathan,
Request you to review following fix:
Bug: https://bugs.openjdk.java.net/browse/JDK-8167209
Webrev: http://cr.openjdk.java.net/~pkbalakr/fx/8167209/webrev.00/
Regards,
Prem
Hi Jonathan,
Request you to review following fix :
Bug : https://bugs.openjdk.java.net/browse/JDK-8087978
Fix : http://cr.openjdk.java.net/~aghaisas/fx/8087978/webrev.1/
Regards,
Ajit
Hi Kevin,
Please review the proposed fix:
Webrev : http://cr.openjdk.java.net/~asrivastava/dipak/8089283/webrev.00/
JBS : https://bugs.openjdk.java.net/browse/JDK-8089283
DRT and unit test cases are running fine with this change.
Many thanks,
Dipak
Hello,
please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/10/8181476/00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8181476
--
Thanks,
Alexander.
Hello,
please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/10/8172847/00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8172847
--
Thanks,
Alexander.
Kevin,
Could you please review the javadoc fix here:
https://bugs.openjdk.java.net/browse/JDK-8181169
http://cr.openjdk.java.net/~jgiles/8181169/index.html
Thanks
--
-- Jonathan
Hi Jonathan & Ajit,
Request you to review the fix:
Bug: https://bugs.openjdk.java.net/browse/JDK-8181213
Webrev: http://cr.openjdk.java.net/~arapte/fx/8181213/webrev.00/
Regards,
Ambarish
Hi Jonathan,
Request you to review following fix:
Bug: https://bugs.openjdk.java.net/browse/JDK-8088389
Webrev: HYPERLINK
"http://cr.openjdk.java.net/%7Epkbalakr/fx/8088389/webrev.00/"http://cr.openjdk.java.net/~pkbalakr/fx/8088389/webrev.00/
Regards,
Prem
Hi Jonathan,
Request you to review following fix :
Bug : https://bugs.openjdk.java.net/browse/JDK-8089310
Fix : http://cr.openjdk.java.net/~aghaisas/fx/8089310/webrev.0/
Regards,
Ajit
Hi Chien,
Please review the following simple fix to update the copyright header dates:
https://bugs.openjdk.java.net/browse/JDK-8179644
http://cr.openjdk.java.net/~kcr/8179644/webrev/
Thanks.
-- Kevin
Hi Jon,
Can you review this simple change to the three .html files (other than
package.html) in the FX docs?
https://bugs.openjdk.java.net/browse/JDK-8180338
http://cr.openjdk.java.net/~kcr/8180338/webrev.00/
Thanks.
-- Kevin
Jonathan,
Please review the following fixes to various typos in the
javadoc-generated API docs:
https://bugs.openjdk.java.net/browse/JDK-8180070
http://cr.openjdk.java.net/~kcr/8180070/webrev.00/
Thanks.
-- Kevin
Kevin,
Please review my changes about removing the code that constructs java
cmd based on JAVA_HOME.
JIRA: https://bugs.openjdk.java.net/browse/JDK-8180368
Webrev: webrev: http://cr.openjdk.java.net/~vdrozdov/JDK-8180368/webrev.00/
--Victor
Jonathan,
Please review the following simple fix to add proper copyright headers
to each package.html file and also the three other .html files in our
doc-files directories:
https://bugs.openjdk.java.net/browse/JDK-8180064
http://cr.openjdk.java.net/~kcr/8180064/webrev/
Thanks.
-- Kevin
Hi Dipak,
Since this is the backport for JDK-8088205 the review should ideally be
in a folder with that bug ID.
Note that the commit message *must* use the base bugID and *never* the
backport ID. Nothing should ever reference the backport ID, and the only
thing that would go into the JBS
401 - 500 of 2484 matches
Mail list logo