JVMTI OOM handling when arrays / objects are too large
Tomas Hurka
Tomas.Hurka at Sun.COM
Mon Jun 15 00:20:51 PDT 2009
Hi Martin and David,
On 14 Jun 2009, at 21:37, Martin Buchholz wrote:
> I've polished the changes in preparation for committing.
> I'll commit once I have reviewer approval.
I have the same concern as David. I think that throwing OOME, in
places where there is obviously a simple programming error, is not a
good idea. Will it be possible to change it to throw
IllegalArgumentException instead?
I have one more comment about the patch itself - changes to src/share/
vm/gc_interface/collectedHeap.inline.hpp and src/share/vm/utilities/
exceptions.hpp are unrelated to the bugfix. I think it will be better
to separate them to the different changeset.
Bye,
--
Tomas Hurka <mailto:tomas.hurka at sun.com>
NetBeans Profiler http://profiler.netbeans.org
VisualVM http://visualvm.dev.java.net
Software Engineer, Developer Platforms Group
Sun Microsystems, Praha Czech Republic
More information about the serviceability-dev
mailing list