Re: Review request for JDK-8067346: Swing submenu has a changed starting offset

2015-09-30 Thread Sergey Bylokhov
Looks fine. On 30.09.15 14:45, Alexander Scherbatiy wrote: The fix looks good to me. Thanks, Alexandr. On 9/30/2015 1:01 PM, Rajeev Chamyal wrote: Hello Alexander, Thanks for the review. I have updated webrev as per review comments. Bug: https://bugs.openjdk.java.net/browse/JDK-80

Re: Review request for JDK-8067346: Swing submenu has a changed starting offset

2015-09-30 Thread Alexander Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 9/30/2015 1:01 PM, Rajeev Chamyal wrote: Hello Alexander, Thanks for the review. I have updated webrev as per review comments. Bug: https://bugs.openjdk.java.net/browse/JDK-8067346 Webrev: http://cr.openjdk.java.net/~psadhukhan/rajeev/8067

Re: Review request for JDK-8067346: Swing submenu has a changed starting offset

2015-09-30 Thread Rajeev Chamyal
Hello Alexander, Thanks for the review. I have updated webrev as per review comments. Bug: https://bugs.openjdk.java.net/browse/JDK-8067346 Webrev: http://cr.openjdk.java.net/~psadhukhan/rajeev/8067346/webrev.02/ Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sen

Re: RFR: [9] [JDK-8081491] The case print incomplete.

2015-09-30 Thread prasanta sadhukhan
ok. I changed it initially to see if it helps in printing same number of rows as displayed but forgot to change it back after adding the rMax decrement in SwingUtilities.getUnwrappedParent(table) check. Please find the updated webrev: http://cr.openjdk.java.net/~psadhukhan/8081491/webrev.10/ R