The main review and code approval for this belongs on the core-libs-...@openjdk.java.net and that looks to be progressing well.

Approved for 7u-dev push once you have code review approval. Please consider porting any related enhancements (to jdk8 code) into the 7u-dev repo also (once available). On that note, would it be easier to push identical changesets to 7u & 8 for now ? I'll leave that for you to decide.

Regards,
Sean.

On 21/11/2012 14:38, Staffan Larsen wrote:
I have recently started a review thread [1] on the core-libs-dev alias about a 
change to add instrumentation for I/O calls. This change was targeted at both 
7u12 and 8. The consensus in that thread is that the changes look good, but 
should ideally be replaced by dynamic bytecode instrumentation. Because of the 
short time remaining for 7u12, I will not have time to redo the implementation 
for this release. Thus the suggested plan was to make the proposed change in 
7u12 and do dynamic bytecode instrumentation for 8.

So, this is seeking approval for integrating the changes [2] in the original 
thread into 7u-dev.

Thanks,
/Staffan


[1] 
http://mail.openjdk.java.net/pipermail/core-libs-dev/2012-November/012217.html
[2] http://cr.openjdk.java.net/~sla/8003322/webrev.01/

Reply via email to