Thanks.
On 8 Oct 2013, at 19:33, Sean Mullan wrote:
Looks good to me.
--Sean
On 10/08/2013 02:14 PM, Vincent Ryan wrote:
Please review the following change - it's a simple re-factoring to promote a
nested class to a stand-alone class:
Bug:
Thanks Max.
On 9 Oct 2013, at 01:51, Weijun Wang wrote:
The code change looks fine. Do we need a CCC for it?
Yes. CCC has been approved.
Thanks
Max
p.s. I would use the bugs.openjdk.java.net URL.
Right. My webrev generates the old URL by default. I've changed it to use
OpenJDK in
Please review the following change - it's a simple re-factoring to promote a
nested class to a stand-alone class:
Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8008171
Webrev: http://cr.openjdk.java.net/~vinnie/8008171/webrev.00/
Thanks.
Looks good to me.
--Sean
On 10/08/2013 02:14 PM, Vincent Ryan wrote:
Please review the following change - it's a simple re-factoring to promote a
nested class to a stand-alone class:
Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8008171
Webrev:
The code change looks fine. Do we need a CCC for it?
Thanks
Max
p.s. I would use the bugs.openjdk.java.net URL.
On 10/9/13 2:14 AM, Vincent Ryan wrote:
Please review the following change - it's a simple re-factoring to promote a
nested class to a stand-alone class:
Bug: