RFR: 8024508 Fix doclint issues in com.sun.nio.sctp

2013-09-10 Thread Chris Hegarty

Trivial doclint issues in the com.sun.nio.sctp public package.

diff -r 4fd7abaf0a5f src/share/classes/com/sun/nio/sctp/Association.java
--- a/src/share/classes/com/sun/nio/sctp/Association.java   Mon Sep 
09 13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/Association.java   Tue Sep 
10 10:08:09 2013 +0100

@@ -58,6 +58,13 @@ public class Association {

 /**
  * Initializes a new instance of this class.
+ *
+ * @param  associationID
+ * The association ID
+ * @param  maxInStreams
+ * The maximum number of inbound streams
+ * @param  maxOutStreams
+ * The maximum number of outbound streams
  */
 protected Association(int associationID,
   int maxInStreams,
diff -r 4fd7abaf0a5f 
src/share/classes/com/sun/nio/sctp/IllegalReceiveException.java
--- a/src/share/classes/com/sun/nio/sctp/IllegalReceiveException.java 
Mon Sep 09 13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/IllegalReceiveException.java 
Tue Sep 10 10:08:09 2013 +0100

@@ -41,6 +41,9 @@ public class IllegalReceiveException ext

 /**
  * Constructs an instance of this class with the specified message.
+ *
+ * @param  msg
+ * The String that contains a detailed message
  */
 public IllegalReceiveException(String msg) {
 super(msg);
diff -r 4fd7abaf0a5f 
src/share/classes/com/sun/nio/sctp/IllegalUnbindException.java
--- a/src/share/classes/com/sun/nio/sctp/IllegalUnbindException.java 
Mon Sep 09 13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/IllegalUnbindException.java 
Tue Sep 10 10:08:09 2013 +0100

@@ -41,6 +41,9 @@ public class IllegalUnbindException exte

 /**
  * Constructs an instance of this class with the specified 
detailed message.

+ *
+ * @param  msg
+ * The String that contains a detailed message
  */
 public IllegalUnbindException(String msg) {
 super(msg);
diff -r 4fd7abaf0a5f 
src/share/classes/com/sun/nio/sctp/InvalidStreamException.java
--- a/src/share/classes/com/sun/nio/sctp/InvalidStreamException.java 
Mon Sep 09 13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/InvalidStreamException.java 
Tue Sep 10 10:08:09 2013 +0100

@@ -40,6 +40,9 @@ public class InvalidStreamException exte

 /**
  * Constructs an instance of this class with the specified 
detailed message.

+ *
+ * @param  msg
+ * The String that contains a detailed message
  */
 public InvalidStreamException(String msg) {
 super(msg);
diff -r 4fd7abaf0a5f src/share/classes/com/sun/nio/sctp/MessageInfo.java
--- a/src/share/classes/com/sun/nio/sctp/MessageInfo.java   Mon Sep 
09 13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/MessageInfo.java   Tue Sep 
10 10:08:09 2013 +0100

@@ -48,7 +48,7 @@ import java.net.SocketAddress;
  * longer required to be sent after the time period expires. It is not 
a hard
  * timeout and may be influenced by whether the association supports 
the partial
  * reliability extension, href=http://www.ietf.org/rfc/rfc3758.txt>RFC 3758

- * 
+ * .
  *
  *  {@code MessageInfo} instances are not safe for use by multiple 
concurrent
  * threads. If a MessageInfo is to be used by more than one thread 
then access

diff -r 4fd7abaf0a5f src/share/classes/com/sun/nio/sctp/Notification.java
--- a/src/share/classes/com/sun/nio/sctp/Notification.java  Mon Sep 
09 13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/Notification.java  Tue Sep 
10 10:08:09 2013 +0100

@@ -40,6 +40,8 @@ public interface Notification {
 public interface Notification {
 /**
  * Returns the association that this notification is applicable to.
+ *
+ * @return  The association
  */
 public Association association();
 }
diff -r 4fd7abaf0a5f src/share/classes/com/sun/nio/sctp/SctpChannel.java
--- a/src/share/classes/com/sun/nio/sctp/SctpChannel.java   Mon Sep 
09 13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/SctpChannel.java   Tue Sep 
10 10:08:09 2013 +0100

@@ -59,7 +59,7 @@ import java.nio.channels.SelectionKey;
  * {@link #setOption(SctpSocketOption,Object) setOption} method. An SCTP
  * channel support the following options:
  * 
- * 
+ * 
  *   
  * Option Name
  * Description
@@ -636,6 +636,9 @@ public abstract class SctpChannel
 /**
  * Returns the value of a socket option.
  *
+ * @param   
+ *  The type of the socket option value
+ *
  * @param   name
  *  The socket option
  *
@@ -658,6 +661,9 @@ public abstract class SctpChannel

 /**
  * Sets the value of a socket option.
+ *
+ * @param   
+ *  The type of the socket option value
  *
  * @param   name
  *  The socket option
@@ -752,6 +758,9 @@ public abstract class SctpChannel
  * the {@code receive} method of this channel, if it does an
  * {@lin

Re: RFR: 8024508 Fix doclint issues in com.sun.nio.sctp

2013-09-10 Thread Alan Bateman

On 10/09/2013 10:09, Chris Hegarty wrote:

Trivial doclint issues in the com.sun.nio.sctp public package.


Looks okay to me.

I'd suggest checking the bug database as there are existing bugs for 
this same issue.


-Alan


Re: RFR: 8024508 Fix doclint issues in com.sun.nio.sctp

2013-09-10 Thread Andreas Rieber

Minor hint, the copyright header year update is missing.

Andreas

On 10.09.2013 11:09, Chris Hegarty wrote:

Trivial doclint issues in the com.sun.nio.sctp public package.

diff -r 4fd7abaf0a5f src/share/classes/com/sun/nio/sctp/Association.java
--- a/src/share/classes/com/sun/nio/sctp/Association.java Mon Sep 09 
13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/Association.java Tue Sep 10 
10:08:09 2013 +0100

@@ -58,6 +58,13 @@ public class Association {

 /**
  * Initializes a new instance of this class.
+ *
+ * @param  associationID
+ * The association ID
+ * @param  maxInStreams
+ * The maximum number of inbound streams
+ * @param  maxOutStreams
+ * The maximum number of outbound streams
  */
 protected Association(int associationID,
   int maxInStreams,
diff -r 4fd7abaf0a5f 
src/share/classes/com/sun/nio/sctp/IllegalReceiveException.java
--- a/src/share/classes/com/sun/nio/sctp/IllegalReceiveException.java 
Mon Sep 09 13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/IllegalReceiveException.java 
Tue Sep 10 10:08:09 2013 +0100

@@ -41,6 +41,9 @@ public class IllegalReceiveException ext

 /**
  * Constructs an instance of this class with the specified message.
+ *
+ * @param  msg
+ * The String that contains a detailed message
  */
 public IllegalReceiveException(String msg) {
 super(msg);
diff -r 4fd7abaf0a5f 
src/share/classes/com/sun/nio/sctp/IllegalUnbindException.java
--- a/src/share/classes/com/sun/nio/sctp/IllegalUnbindException.java 
Mon Sep 09 13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/IllegalUnbindException.java 
Tue Sep 10 10:08:09 2013 +0100

@@ -41,6 +41,9 @@ public class IllegalUnbindException exte

 /**
  * Constructs an instance of this class with the specified 
detailed message.

+ *
+ * @param  msg
+ * The String that contains a detailed message
  */
 public IllegalUnbindException(String msg) {
 super(msg);
diff -r 4fd7abaf0a5f 
src/share/classes/com/sun/nio/sctp/InvalidStreamException.java
--- a/src/share/classes/com/sun/nio/sctp/InvalidStreamException.java 
Mon Sep 09 13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/InvalidStreamException.java 
Tue Sep 10 10:08:09 2013 +0100

@@ -40,6 +40,9 @@ public class InvalidStreamException exte

 /**
  * Constructs an instance of this class with the specified 
detailed message.

+ *
+ * @param  msg
+ * The String that contains a detailed message
  */
 public InvalidStreamException(String msg) {
 super(msg);
diff -r 4fd7abaf0a5f src/share/classes/com/sun/nio/sctp/MessageInfo.java
--- a/src/share/classes/com/sun/nio/sctp/MessageInfo.java Mon Sep 09 
13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/MessageInfo.java Tue Sep 10 
10:08:09 2013 +0100

@@ -48,7 +48,7 @@ import java.net.SocketAddress;
  * longer required to be sent after the time period expires. It is 
not a hard
  * timeout and may be influenced by whether the association supports 
the partial
  * reliability extension, href=http://www.ietf.org/rfc/rfc3758.txt>RFC 3758

- * 
+ * .
  *
  *  {@code MessageInfo} instances are not safe for use by multiple 
concurrent
  * threads. If a MessageInfo is to be used by more than one thread 
then access

diff -r 4fd7abaf0a5f src/share/classes/com/sun/nio/sctp/Notification.java
--- a/src/share/classes/com/sun/nio/sctp/Notification.java Mon Sep 09 
13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/Notification.java Tue Sep 10 
10:08:09 2013 +0100

@@ -40,6 +40,8 @@ public interface Notification {
 public interface Notification {
 /**
  * Returns the association that this notification is applicable to.
+ *
+ * @return  The association
  */
 public Association association();
 }
diff -r 4fd7abaf0a5f src/share/classes/com/sun/nio/sctp/SctpChannel.java
--- a/src/share/classes/com/sun/nio/sctp/SctpChannel.java Mon Sep 09 
13:44:30 2013 +0100
+++ b/src/share/classes/com/sun/nio/sctp/SctpChannel.java Tue Sep 10 
10:08:09 2013 +0100

@@ -59,7 +59,7 @@ import java.nio.channels.SelectionKey;
  * {@link #setOption(SctpSocketOption,Object) setOption} method. An SCTP
  * channel support the following options:
  * 
- * 
+ * 
  *   
  * Option Name
  * Description
@@ -636,6 +636,9 @@ public abstract class SctpChannel
 /**
  * Returns the value of a socket option.
  *
+ * @param   
+ *  The type of the socket option value
+ *
  * @param   name
  *  The socket option
  *
@@ -658,6 +661,9 @@ public abstract class SctpChannel

 /**
  * Sets the value of a socket option.
+ *
+ * @param   
+ *  The type of the socket option value
  *
  * @param   name
  *  The socket option
@@ -752,6 +758,9 @@ public abstract class SctpChannel
  * the {@c

Re: RFR: 8024508 Fix doclint issues in com.sun.nio.sctp

2013-09-10 Thread Alan Bateman

On 10/09/2013 10:27, Andreas Rieber wrote:

Minor hint, the copyright header year update is missing.

Andreas
We're not strictly required to do that each time, it's a confusing point 
that comes up regularly (with the result that we are inconsistent). 
There's a script checked into the top-level repo that can be used to do 
bulk updates. I don't know how often it is run, hopefully at least once 
a year.


-Alan


Re: RFR: 8024508 Fix doclint issues in com.sun.nio.sctp

2013-09-10 Thread Andreas Rieber

On 10.09.2013 11:36, Alan Bateman wrote:

On 10/09/2013 10:27, Andreas Rieber wrote:

Minor hint, the copyright header year update is missing.

Andreas
We're not strictly required to do that each time, it's a confusing 
point that comes up regularly (with the result that we are 
inconsistent). There's a script checked into the top-level repo that 
can be used to do bulk updates. I don't know how often it is run, 
hopefully at least once a year.


-Alan
OK, thanks for the info. Just most of the time i see that the year gets 
also updated with changes if not done already.


Andreas



hg: jdk8/tl/jdk: 8023878: (fs) TEST_BUG java/nio/file/WatchService/SensitivityModifier.java fails intermittently

2013-09-10 Thread alan . bateman
Changeset: 909aced59bef
Author:alanb
Date:  2013-09-10 10:42 +0100
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/909aced59bef

8023878: (fs) TEST_BUG java/nio/file/WatchService/SensitivityModifier.java 
fails intermittently
Reviewed-by: alanb
Contributed-by: yiming.w...@oracle.com

! test/java/nio/file/WatchService/SensitivityModifier.java



Re: RFR: 8024508 Fix doclint issues in com.sun.nio.sctp

2013-09-10 Thread Chris Hegarty
As a point of process, and since the copyright year range update is not 
strictly required, I typically leave it out of individual "bug" fixes. I 
have seen too many times where it has caused unnecessary conflicts when 
sync'ing or backporting.


-Chris.

On 09/10/2013 10:44 AM, Andreas Rieber wrote:

On 10.09.2013 11:36, Alan Bateman wrote:

On 10/09/2013 10:27, Andreas Rieber wrote:

Minor hint, the copyright header year update is missing.

Andreas

We're not strictly required to do that each time, it's a confusing
point that comes up regularly (with the result that we are
inconsistent). There's a script checked into the top-level repo that
can be used to do bulk updates. I don't know how often it is run,
hopefully at least once a year.

-Alan

OK, thanks for the info. Just most of the time i see that the year gets
also updated with changes if not done already.

Andreas



hg: jdk8/tl/langtools: 8024414: javac, should facilitate the use of the bootstrap compiler for debugging

2013-09-10 Thread vicente . romero
Changeset: d87f017ec217
Author:mcimadamore
Date:  2013-09-10 16:47 +0100
URL:   http://hg.openjdk.java.net/jdk8/tl/langtools/rev/d87f017ec217

8024414: javac, should facilitate the use of the bootstrap compiler for 
debugging
Reviewed-by: jjg

! make/netbeans/langtools/build.xml
! make/tools/anttasks/SelectToolTask.java



hg: jdk8/tl/jdk: 8024501: sun.security.mscapi.Key has no definition of serialVersionUID

2013-09-10 Thread xuelei . fan
Changeset: c9083205e6eb
Author:xuelei
Date:  2013-09-10 21:31 -0700
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c9083205e6eb

8024501: sun.security.mscapi.Key has no definition of serialVersionUID
Reviewed-by: weijun

! src/windows/classes/sun/security/mscapi/Key.java



hg: jdk8/tl/jdk: 8024141: Unexpected timezone display name

2013-09-10 Thread masayoshi . okutsu
Changeset: 13ee370ee8b3
Author:okutsu
Date:  2013-09-11 15:29 +0900
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/13ee370ee8b3

8024141: Unexpected timezone display name
Reviewed-by: peytoia

! src/share/classes/sun/util/locale/provider/LocaleResources.java
+ test/sun/util/locale/provider/Bug8024141.java