Hello Sebastian.
On 8/17/2011 7:43 AM, Sebastian Sickelmann wrote:
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?
You should not set JAVAC_WARNINGS_FATAL manually, and you can ignore any
preexisting warnings (although you should not introduce any new ones).
The makefiles where it is safe to compile with JAVAC_WARNINGS_FATAL will
set it appropriately. There is an ongoing effort to reduce javac
warnings (including lint warnings, which are not currently enabled in
all parts of the code). If you build with JAVAC_MAX_WARNINGS=true, you
will notice that there are just over 10,000 javac build warnings in
tl/jdk...
Cheers,
Sasha
-- Sebastian
