On 25 aug 2014, at 16:15, Alan Bateman <alan.bate...@oracle.com> wrote:
> On 25/08/2014 14:35, Staffan Larsen wrote: >> Please review the following change to remove hprof as part of the demo >> package. Instead, the source code has been moved to the jdk.hprof.agent >> module. >> >> bug: https://bugs.openjdk.java.net/browse/JDK-8043936 >> webrev: http://cr.openjdk.java.net/~sla/8043936/webrev.00/ >> > The move of the source code to native/libhprof looks good. > > Having the README and manual in the same directory is probably okay as it has > always lived in the same directory as the source code. One thing is check is > the contents of manual.html as there is at least one place where it > references the directory demo/jvmti/hprof. Good point. I have removed the section about Source Code in the latest version since all OpenJDK source code can be found at the same place: http://cr.openjdk.java.net/~sla/8043936/webrev.01/ I plan to push this version tomorrow unless someone shouts ;) Thanks, /Staffan > > I assume Erik or Magnus will check the make changes, I don't see any issues. > > -Alan >