Am 17.08.2011 16:43, schrieb Sebastian Sickelmann:
Am 17.08.2011 13:45, schrieb Alan Bateman:
joe.da...@oracle.com wrote:
I am willing to sponsor this work and I've filed bug 7080020 "Add
conventional constructors to InternalError" for it.
Thanks Joe, and to help this along here's the webrev with Sebastian's
patches:
http://cr.openjdk.java.net/~alanb/7080020/webrev/
I haven't really looked but one thing I did notice is that the
multi-catch usages need to be checked as Sasha cleaned up a few of
these recently.
-Alan.
Thanks for the webrev.
I think it is very hard to start with the complete patch. Or is it
better to review it all at once?
I have created a webrev for my patches 0 and 1 and uploaded it here:
http://oss-patches.24.eu/openjdk8/InternalError/part1/
After i have pulled from tl/jdk i got problems with building.
I got warnings so that i must build with JAVAC_WARNINGS_FATAL=false,
is this ok for tl-repo?
-- Sebastian
Ok. Rebased again.
Now the webrevs are:
http://oss-patches.24.eu/openjdk8/InternalError/part1/REBASED_ON_e59aad6ed600/
http://oss-patches.24.eu/openjdk8/InternalError/part2/REBASED_ON_e59aad6ed600/
Unfortunatly after i have uploaded the webrev(takes some time because
128kbit/sek up is not the fastest)
there was a change that i not rebased on.
rev 2961329a6774
6237353: Remove sun.io package from j2se binary
rebasing on this removes some files in sun.io package these files should
be ignored while reviewing my webrevs.
If someone needs the rebased webrevs, let me know.
-- Sebastian
