Review needed for 8000687

2012-10-10 Thread Lance Andersen - Oracle 
Need a reviewer for a simple typo in the DriverManager javadoc

new-host-2:sql lanceandersen$ hg diff
diff -r 036c55976cef src/share/classes/java/sql/DriverManager.java
--- a/src/share/classes/java/sql/DriverManager.java Tue Oct 09 08:58:27 
2012 -0400
+++ b/src/share/classes/java/sql/DriverManager.java Wed Oct 10 10:59:00 
2012 -0400
@@ -412,7 +412,7 @@
  * method throws a codejava.lang.SecurityException/code.
  *
  * @param out the new logging/tracing PrintStream; to disable, set to 
codenull/code
- * @deprecated Use {@code setLogWriter)
+ * @deprecated Use {@code setLogWriter}
  * @throws SecurityException if a security manager exists and its
  *codecheckPermission/code method denies setting the log stream
  *
@@ -439,7 +439,7 @@
  * and all drivers.
  *
  * @return the logging/tracing PrintStream; if disabled, is 
codenull/code
- * @deprecated  Use {@code getLogWriter)
+ * @deprecated  Use {@code getLogWriter}
  * @see #setLogStream
  */
 @Deprecated
new-host-2:sql lanceandersen$ 


Best
Lance

Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
lance.ander...@oracle.com

Re: Review needed for 8000687

2012-10-10 Thread Alan Bateman 

On 10/10/2012 16:08, Lance Andersen - Oracle wrote:

Need a reviewer for a simple typo in the DriverManager javadoc

new-host-2:sql lanceandersen$ hg diff
diff -r 036c55976cef src/share/classes/java/sql/DriverManager.java
--- a/src/share/classes/java/sql/DriverManager.java Tue Oct 09 08:58:27 
2012 -0400
+++ b/src/share/classes/java/sql/DriverManager.java Wed Oct 10 10:59:00 
2012 -0400
@@ -412,7 +412,7 @@
   * method throws acodejava.lang.SecurityException/code.
   *
   * @param out the new logging/tracing PrintStream; to disable, set 
tocodenull/code
- * @deprecated Use {@code setLogWriter)
+ * @deprecated Use {@code setLogWriter}
   * @throws SecurityException if a security manager exists and its
   *codecheckPermission/code  method denies setting the log stream
   *
@@ -439,7 +439,7 @@
   * and all drivers.
   *
   * @return the logging/tracing PrintStream; if disabled, 
iscodenull/code
- * @deprecated  Use {@code getLogWriter)
+ * @deprecated  Use {@code getLogWriter}
   * @see #setLogStream
   */
  @Deprecated
new-host-2:sql lanceandersen$

Thanks Lance, looks good to me. Also the javadoc build was warning free 
a few days ago and it's nice to get back to that state.


-Alan