And a few more duplicates under jdk/src/share/classes/java/sql:
./NClob.java: * which *provides provides* methods for getting the
./Connection.java: * @exception *SQLException SQLException* if a
database access error occurs
./Connection.java: * @exception *SQLException SQLException* if a
database access error occurs
./Connection.java: * @exception *SQLException SQLException* if a
database access error occurs
./Connection.java: * You must set *the the* values for the
<code>TypeMap</code> prior to
./Connection.java: * can *be be* determined by invoking
./DatabaseMetaData.java: * <P>Each procedure description has *the
the* following columns:
./DatabaseMetaData.java: * <P>Each function description has *the
the* following columns:
On 08.08.2014 4:09, Ivan Gerasimov wrote:
Hi Lance!
I like to grep for the word duplicates :)
Under jdk/src/share/classes/javax/sql grep found:
./rowset/spi/SyncFactory.java: * Adds*the the* given
synchronization provider to the factory register. Guidelines
./rowset/spi/SyncFactory.java: * ROWSET properties and stores lazy
references into*the the* internal registry.
./rowset/spi/SyncProvider.java: * degrees *of of *care. The first
group of constants indicate how synchronization
./rowset/spi/package.html:For *the the* next level up, the
./rowset/BaseRowSet.java: * in the driver documentation. Although
there are guidelines*for for *how
./rowset/RowSetProvider.java: * in jars available to the runtime.
For example, to have*the the* RowSetFactory
./rowset/JoinRowSet.java: * A match column can *be be *set in two ways:
./rowset/JoinRowSet.java: * An ANSI-style <code>JOIN</code>
providing *a a* full JOIN. Specifies that all
./rowset/serial/package.html:that *the the* originating data source
manages. <br>
./rowset/serial/package.html:<code>wasNull</code> is used to determine
whether *the the* last value read was SQL NULL.
./rowset/CachedRowSet.java: * until*all all *components reading
this <code>CachedRowSet</code> object
./rowset/CachedRowSet.java: * row *in in* this
<code>CachedRowSet</code> object
./rowset/CachedRowSet.java: * on *the the* rowset reader
(<code>RowSetReader</code> object) from the
./rowset/CachedRowSet.java: * was instantiated, or if none
*was was* set, the default provider
./rowset/FilteredRowSet.java: * <code>FilteredRowSet</code> object are
detected. A new filter*has has* an
./rowset/RowSetMetaDataImpl.java: * Retrieves*the the* suggested
column title for the designated
./rowset/RowSetMetaDataImpl.java: * The field that holds *the
the* type name used by this particular data source
./rowset/package.html:standard interfaces specified in this package
*and and* <b>may</b> extend the
Would you please include these typo fixes in your patch?
Sincerely yours,
Ivan
On 08.08.2014 3:57, Lance @ Oracle wrote:
Hi joe,
Thank you for the quick review.
I will change those to {@code Inputstream}
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
Sent from my iPad
On Aug 7, 2014, at 7:49 PM, Joe Darcy <joe.da...@oracle.com> wrote:
On 08/07/2014 03:44 PM, Lance Andersen wrote:
Hi,
Looking for a reviewer for corrections to various javadoc typos
The webrev can be found at
http://cr.openjdk.java.net/~lancea/8054555/webrev.00/
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
Hi Lance,
In src/share/classes/java/sql/CallableStatement.java
@@ -1899,7 +1900,7 @@
* @param length the number of bytes in the parameter data.
* @throws SQLException if parameterName does not correspond
to a named
* parameter; if the length specified
- * is less than zero; if the number of bytes in the inputstream
does not match
+ * is less than zero; if the number of bytes in the Inputstream
does not match
* the specified length; if a database access error occurs or
* this method is called on a closed
<code>CallableStatement</code>
* @exception SQLFeatureNotSupportedException if the JDBC
driver does not support
I think the new line would be better using either "input stream" or
"{@code InputStream}".
Similar comment in src/share/classes/javax/sql/RowSet.java
@@ -1420,7 +1420,7 @@
void setBlob (int i, Blob x) throws SQLException;
/**
- * Sets the designated parameter to a <code>InputStream</code>
object. The inputstream must contain the number
+ * Sets the designated parameter to a <code>InputStream</code>
object. The <code>InputStream</code> must contain the number
* of characters specified by length otherwise a
<code>SQLException</code> will be
* generated when the <code>PreparedStatement</code> is executed.
* This method differs from the <code>setBinaryStream (int,
InputStream, int)</code>
@@ -1474,7 +1474,8 @@
throws SQLException;
/**
- * Sets the designated parameter to a <code>InputStream</code>
object. The <code>inputstream</code> must contain the number
+ * Sets the designated parameter to a <code>InputStream</code>
object.
+ * The <code>inputstream</code> must contain the number
* of characters specified by length, otherwise a
<code>SQLException</code> will be
* generated when the <code>CallableStatement</code> is executed.
* This method differs from the <code>setBinaryStream (int,
InputStream, int)</code>
Otherwise, the changes look okay.
Cheers,
-Joe
