Am 28.08.2011 06:22, schrieb Dr Andrew John Hughes:
On 14:16 Tue 23 Aug     , Sebastian Sickelmann wrote:

I have rebased my "additional chainings of InternalError" on your commit.

The webrev is:
http://oss-patches.24.eu/openjdk8/InternalError/part2/REBASED_ON_c43af666d130/

Reviews are welcome for this too.

This looks fine to me; mostly trivial stuff.
Yes. It's more a assiduity work*, then real complex.
The only non trival stuff maybe ***src/share/classes/java/text/DecimalFormat.java* <http://dict.leo.org/ende?lp=ende&p=Ci4HO3kMAA&search=assiduity&trestr=0x8001> [0]
in combination with src/share/classes/java/text/Format.java [1]

Is there someone who whats to sponsor this by creating a CR and pushing it? Or are there more reviewers needed?

How is such cleanup-work handled? I can make more assiduity work on this, but is it worth it? Are there drawbacks, additional to my mailing-list spam, by such cleanups? Chaining more aggresivly maybe cause CG to do later cleanup and serialized Exceptions can be more expensive while serializing and transfering over wire(remote-calls etc.).

-- Sebastian

*
- Sebastian

Reply via email to