Re: RFR | 8073779: StackOverflowError called StackOverflowException in javadoc

2015-02-24 Thread Martin Buchholz
Approved! On Tue, Feb 24, 2015 at 8:44 AM, Ivan Gerasimov wrote: > Hi! > > A typo populated over a few places. > > Fix: s/StackOverflowException/StackOverflowError/g > > BUGURL: https://bugs.openjdk.java.net/browse/JDK-8073779 > WEBREV: http://cr.openjdk.java.net/~igerasim/8073779/0/webrev/ > >

Re: RFR | 8073779: StackOverflowError called StackOverflowException in javadoc

2015-02-24 Thread Seán Coffey
On 24/02/2015 17:08, Ivan Gerasimov wrote: Right. I shouldn't have pushed it that quick. Sigh Nothing important. Just thought it might be a useful step while you were there! It's something we can keep in mind for a later fix. regards, Sean. On 24.02.2015 19:53, Seán Coffey wrote: Nice catc

Re: RFR | 8073779: StackOverflowError called StackOverflowException in javadoc

2015-02-24 Thread Ivan Gerasimov
Right. I shouldn't have pushed it that quick. Sigh On 24.02.2015 19:53, Seán Coffey wrote: Nice catch Ivan. You could also clean up the typo in various test cases relating to it also perhaps : langtools/test/tools/javac/generics/inference/8043725/T8043725.java jdk/test/java/util/Hashtable/Self

Re: RFR | 8073779: StackOverflowError called StackOverflowException in javadoc

2015-02-24 Thread Seán Coffey
Nice catch Ivan. You could also clean up the typo in various test cases relating to it also perhaps : langtools/test/tools/javac/generics/inference/8043725/T8043725.java jdk/test/java/util/Hashtable/SelfRef.java jdk/test/javax/swing/border/Test6461042.java langtools/test/tools/javac/defaultMetho

Re: RFR | 8073779: StackOverflowError called StackOverflowException in javadoc

2015-02-24 Thread Ivan Gerasimov
Thanks! pushed. On 24.02.2015 19:46, Martin Buchholz wrote: Approved! On Tue, Feb 24, 2015 at 8:44 AM, Ivan Gerasimov mailto:ivan.gerasi...@oracle.com>> wrote: Hi! A typo populated over a few places. Fix: s/StackOverflowException/StackOverflowError/g BUGURL: https://bugs.

RFR | 8073779: StackOverflowError called StackOverflowException in javadoc

2015-02-24 Thread Ivan Gerasimov
Hi! A typo populated over a few places. Fix: s/StackOverflowException/StackOverflowError/g BUGURL: https://bugs.openjdk.java.net/browse/JDK-8073779 WEBREV: http://cr.openjdk.java.net/~igerasim/8073779/0/webrev/ Sincerely yours, Ivan