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 i
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: http://bugs.sun.com/bugdatabase
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: http://bugs.sun.com/b
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: http://cr.openjdk.java.net/~vinnie/800
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.