Hello all. I have updated the webrev with a final correction from Brian Goetz,
the @throws NPE didn't reflect that it was thrown unconditionally if value is
null.
http://cr.openjdk.java.net/~mduigou/JDK-8029055/2/webrev/
Mike
On Nov 25 2013, at 20:32 , Mike Duigou wrote:
>
> On Nov 24 2013
Changeset: d6c4ae56c079
Author:juh
Date: 2013-12-06 11:36 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d6c4ae56c079
8007967: Infinite loop can happen in
sun.security.provider.certpath.SunCertPathBuilder.depthFirstSearchForward()
Reviewed-by: mullan
! src/share/classes/sun
Changeset: 2d0a0ae7fa9c
Author:ksrini
Date: 2013-12-06 09:07 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/2d0a0ae7fa9c
8029504: Regression: TestDocRootLink test fails on Windows
Reviewed-by: bpatel, jjg
! src/share/classes/com/sun/tools/doclets/formats/html/HtmlDocle
Hello all;
https://bugs.openjdk.java.net/browse/JDK-8029696
Michael McMahon noticed that some links to the anchor #NonInterference were not
using the correct name for the anchor. He prepared a patch which I reviewed,
tested. I also checked to make sure there were no other broken instances.
I'v
Changeset: f8da1f34c65c
Author:darcy
Date: 2013-12-06 11:28 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f8da1f34c65c
8023471: Add compatibility note to AnnotatedElement
Reviewed-by: smarks, jfranck, abuckley
! src/share/classes/java/lang/annotation/Annotation.java
! src/s
On Dec 6 2013, at 02:07 , Chris Hegarty wrote:
> On 06/12/13 08:38, Paul Sandoz wrote:
>> On Dec 5, 2013, at 9:29 PM, Doug Lea wrote:
>>> On 12/05/2013 03:18 PM, Brent Christian wrote:
I'm curious about why this was done:
*** 4452,4462
public final boolean re
On 12/06/2013 09:20 AM, Martin Buchholz wrote:
FYI: When I run javadoc on jsr166 CVS (it's not too hard for y'all to do
this as well), I still see many warnings inherited from openjdk8 source:
$ ant docs
...
[javadoc] Constructing Javadoc information...
[javadoc] Standard Doclet version
On 06/12/13 11:44, Alan Bateman wrote:
On 05/12/2013 05:30, Sergey Lugovoy wrote:
Hi all,
please review the fix
http://cr.openjdk.java.net/~yan/8029451/webrev.01/
for
https://bugs.openjdk.java.net/browse/JDK-8029451
This patch cleanup tidy warnings for generated html documentation,
and do
not
+1,
It can be used with any tag (not just , including , , ,
, etc.)
On 12/6/2013 1:09 AM, Martin Buchholz wrote:
In jsr166-land we recently started using , which I recommend,
e.g.
AbstractQueuedSynchronizer.java:122: * Because checks in
acquire are invoked before
On Thu, Dec 5, 2013 at 3
See https://bugs.openjdk.java.net/browse/JDK-8029676
Stephen
On 26 November 2013 13:20, Stephen Colebourne wrote:
> I took a quick look, but jumped back in horror at the start of the
> Javadoc for the new methods in Map. A Javadoc description should start
> with the positive, not the negative. I
On 05/12/2013 05:30, Sergey Lugovoy wrote:
Hi all,
please review the fix
http://cr.openjdk.java.net/~yan/8029451/webrev.01/
for
https://bugs.openjdk.java.net/browse/JDK-8029451
This patch cleanup tidy warnings for generated html documentation, and do
not affect the appearance of the documentatio
On 06/12/13 08:38, Paul Sandoz wrote:
On Dec 5, 2013, at 9:29 PM, Doug Lea wrote:
On 12/05/2013 03:18 PM, Brent Christian wrote:
I'm curious about why this was done:
*** 4452,4462
public final boolean removeAll(Collection c) {
! Objects.requireNonNull(c);
On Dec 5, 2013, at 9:29 PM, Doug Lea wrote:
> On 12/05/2013 03:18 PM, Brent Christian wrote:
>> I'm curious about why this was done:
>>
>> *** 4452,4462
>> public final boolean removeAll(Collection c) {
>> ! Objects.requireNonNull(c);
>> boolean modified =
13 matches
Mail list logo