On 22/11/2013 1:02 AM, Dave Pointon wrote:
Hi again, Magnus ,
On Fri, 2013-11-22 at 00:35 +1000, David Holmes wrote:
On 21/11/2013 11:05 PM, Magnus Ihse Bursie wrote:
On 2013-11-19 13:47, Dave Pointon wrote:
The updated webrev can be found at
http://cr.openjdk.java.net/~ngmr/JI-9008287/webrev.02/
The actual code looks better, but I'm not so sure about the copyright
header. The actual change was indeed very small, and is more or less a
variation of what I suggested earlier.
If you insist on keeping it, I'll need to check with legal support
first. Otherwise I think it looks good (but I'm not a formal reviewer).
That is my position as a Reviewer as well.
Meanwhile we should see about migrating the bug from JI to JDK.
Thanks,
David
/Magnus
I've just received an update WRT the copyright change - it seems that I
was inadvertently over zealous in the application of instructions and
given the (lack of the) magnitude of the change, I should not have
included the copyright statement.
Would you like me to generate and indeed post, a new webrev ?
Please do. I moved the JI issue to JDK:
https://bugs.openjdk.java.net/browse/JDK-8028794
Not sure what your plans was for getting this pushed but we are in a
phase were an approval process has to be followed to make changes to 8.
The 9 forest will hopefully be open "real soon now" and this could be
backported to 8u after that.
Thanks,
David
Rgds ,