Request for review: Rework cause-chaining in Exceptions was:JDK 8 code review request for 6380161 (reflect) Exception from newInstance() not chained to cause.

Sebastian Sickelmann sebastian.sickelmann at gmx.de
Wed Aug 17 21:37:41 UTC 2011


Am 17.08.2011 23:34, schrieb Alexandre Boulgakov:
> On 8/17/2011 2:04 PM, Sebastian Sickelmann wrote:
>> Am 17.08.2011 16:58, schrieb Alexandre Boulgakov:
>>> Hello Sebastian.
>>>
>>> On 8/17/2011 7:43 AM, Sebastian Sickelmann wrote:
>>>> Am 17.08.2011 13:45, schrieb Alan Bateman:
>>>>> joe.darcy at 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...
>> OK. Actually there aren't any warnings. The are all gone after i did 
>> a make clean & make all.
>> Can that be? Is there an issue like : "make all doen't find all 
>> changes to rebuild all dependencies", or is this a quite normal 
>> behavior?
> That doesn't sound right... Are you sure there aren't any warnings? 
> They can get lost among the other output. If you are redirecting 
> output to a file and then checking for warnings, you should make sure 
> to redirect both the standard output (&1) and the standard error (&2) 
> streams since the output gets split between those.
Ohh sorry, i explained it wrong. After clean rebuild the problem that 
there is a warning that is handled as a fatal error was away.  Haven't 
checks for "normal" warnings. I think there are some. I will check if i 
introduce some new warnings, but i don't think so.
>
> That said, you shouldn't have as many as 10 000 on a normal build, but 
> there should be some coming from different tools including javac, the 
> C compiler, the linker, and more.
>
> -Sasha
>>>
>>> Cheers,
>>> Sasha
>>>
>>>>
>>>> -- Sebastian
>>




More information about the core-libs-dev mailing list