Andrew John Hughes wrote:
2009/11/18 Kelly O'Hair <kelly.oh...@sun.com>:
Need a reviewer.

The jhat change is from keith.rand...@gmail.com. I wasn't able to
come up with a testcase for that change.

The other two just fix a couple of broken tests and remove them from
the ProblemList.

6892742: Improve root set used by jhat
6902323: Fix testcase sun/tools/native2ascii/NativeErrors.java
6902325: Fix testcase sun/tools/jhat/HatHeapDump1Test.java

http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-tl-jhat-6892742/webrev/

-kto




Pretty much the same fix for the JHat issue was filed by one of our
Red Hat interns in March:

http://bugs.openjdk.java.net/show_bug.cgi?id=100026

but sadly completely ignored by anyone at Sun (at least it seems so
from the bug report).  This is one reason why I've tended to not
bother with the OpenJDK bug system...


Sorry. I already have to deal with Sun's internal bug tracking system,
and also JIRA for JavaFX. Learning my way around bugzilla has not been
high my list. I get lost and tired of waiting every time I get into
the bugzilla pages. :^(  I suspect most Sun people may be in the same boat.

How could I at least see some email on these bugs?
Aren't they connected to the various openjdk email aliases?

Sorry I'm not bugzilla-savy. :^(

Anyway, coming to merge that with the version now in OpenJDK6 version
I found that the upstream doesn't update the comment.  So perhaps we
could push:

http://cr.openjdk.java.net/~andrew/jhat/webrev.01/

given a bug ID for this small change.

6909442: Fix comments in test/sun/tools/jhat/HatRun.java

-kto

Reply via email to