Hi, Sergey.
The fix looks good, but can you reformat the changes in JTable before the push, it seems that this file use 80-line alignment.

On 10.04.2014 17:19, sergey malenkov wrote:
Hello,

Could you please review the second version of the fix:
http://cr.openjdk.java.net/~malenkov/8039750.9.1/
Small type in the JTable class was fixed. Tests were added.

Thanks,
SAM

On 09.04.2014 19:40, sergey malenkov wrote:
Hello,

Could you please review the following fix:
http://cr.openjdk.java.net/~malenkov/8039750.9.0/
http://bugs.openjdk.java.net/browse/JDK-8039750

The MetalLazyValue is removed, all usages replaced with the LazyValue lambdas. The TableLazyValue is removed, all usages replaced with the LazyValue lambdas.

Thanks,
SAM




--
Best regards, Sergey.

Reply via email to