Request for review: 7084245: Update usages of InternalError to use exception chaining

David Holmes David.Holmes at oracle.com
Mon Aug 29 00:02:00 UTC 2011


On 29/08/2011 5:35 AM, Sebastian Sickelmann wrote:
> Hi, here is a webrev[1] for some cleanup that i want to integrated in
> tl-repositories.
>
> Alan Bateman had scanned the changes and gave me some good input[3] for
> further discussion here:
>
> The changes to java.util.concurrent should go through Doug Lea's upstream
> CVS. Alan told me that Chris Hegarty is on this topic already. The suggested
> changes for this is here[2].

These changes are somewhat pointless. We don't need to set the cause because 
it will never happen. The classes are cloneable so 
CloneNotSupportedException can not be thrown. If it is then something really 
bizarre has happened hence we throw InternalError. The site of the 
InternalError tells us why it was thrown.

David
-----

> I have changed some classes in awt / sun.java2d maybe someone of the 2d-dev
> maillinglist can look at these changes.
> I also changed some classes in java/securtiy maybe someone of security-dev
> maillinglist can look at these changes.
>
> Let me know if there is a need to split/rebase the main-webrev[1] to review
> and/or push it individually.
>
> Mostly the patch changes exception-chains. But there are some places where
> the patch changes behavoir:
>
> - I removed some printstackTraces in
> sun.java2d.pipe.LoopPipe.getStrokesSpans and sun.misc.Launcher (Alan told me
> that kumar maybe want to have a look at it?).
> - I changed java.text.Format.clone not to return null. I think it will never
> happen. But if so throwing an InternalError seems to be better than
> returning null and let all the extended classes crash in there clone Method
> with a NullPointerException. And so catching an Exception in
> java.text.DecimalFormat.clone is unnecessary.
>
> -- Sebastian
>
> [1] http://oss-patches.24.eu/openjdk8/InternalError/part2/7084245_main_0/
> [2] http://oss-patches.24.eu/openjdk8/InternalError/part2/7084245_concurrent_0/
> [3]
> http://mail.openjdk.java.net/pipermail/core-libs-dev/2011-August/007563.html



More information about the core-libs-dev mailing list