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
Fri Aug 19 21:39:18 UTC 2011


Am 17.08.2011 23:31, schrieb Sebastian Sickelmann:
> 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
I change the code style for multi-catch and fixed the bug that there 
were two Classes with <<<<< >>>>> lines in it.
Sorry for this failure with the <<<<<< >>>>>> lines

The new webrevs are here:

http://oss-patches.24.eu/openjdk8/InternalError/part1/REBASED_ON_46b53f80ab0a
http://oss-patches.24.eu/openjdk8/InternalError/part2/REBASED_ON_46b53f80ab0a

-- Sebastian



More information about the core-libs-dev mailing list