<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">You can add me as the reviewer. But I think you need to send an email out to <div><a href="mailto:core-libs-dev@openjdk.java.net">core-libs-dev@openjdk.java.net</a> warning them that you have added the jfr tests to testset=core.</div><div>So that if they run into jfr failures, they know who to contact.</div><div><br></div><div>-kto</div><div><br><div><div>On Jun 4, 2012, at 6:03 AM, Staffan Larsen wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Can someone please review this change?</div><div><br></div><div>Thanks,</div><div>/Staffan</div><br><div><div>On 25 maj 2012, at 15:41, Staffan Larsen wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Please review the following change to makefiles to add a target for the JFR tests.<div><br></div><div>Webrev: <a href="http://cr.openjdk.java.net/~sla/7165257/webrev.01/">http://cr.openjdk.java.net/~sla/7165257/webrev.01/</a></div><div><br></div><div>Thanks,</div><div>/Staffan</div></div></blockquote></div><br></div></blockquote></div><br></div></body></html>