You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-commits@db.apache.org by dj...@apache.org on 2006/09/22 16:30:54 UTC

svn commit: r448948 - /db/derby/code/trunk/java/engine/org/apache/derby/impl/jdbc/EmbedResultSet.java

Author: djd
Date: Fri Sep 22 07:30:53 2006
New Revision: 448948

URL: http://svn.apache.org/viewvc?view=rev&rev=448948
Log:
Minor cleanup in EmbedResultSet.java, code to set warning in constructor can use the addWarning() method
rather than repeating code.

Modified:
    db/derby/code/trunk/java/engine/org/apache/derby/impl/jdbc/EmbedResultSet.java

Modified: db/derby/code/trunk/java/engine/org/apache/derby/impl/jdbc/EmbedResultSet.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/engine/org/apache/derby/impl/jdbc/EmbedResultSet.java?view=diff&rev=448948&r1=448947&r2=448948
==============================================================================
--- db/derby/code/trunk/java/engine/org/apache/derby/impl/jdbc/EmbedResultSet.java (original)
+++ db/derby/code/trunk/java/engine/org/apache/derby/impl/jdbc/EmbedResultSet.java Fri Sep 22 07:30:53 2006
@@ -249,10 +249,7 @@
 			if (!isForUpdate()) { //language resultset not updatable
 				concurrencyOfThisResultSet = JDBC20Translation.CONCUR_READ_ONLY;
 				SQLWarning w = StandardException.newWarning(SQLState.QUERY_NOT_QUALIFIED_FOR_UPDATABLE_RESULTSET);
-				if (topWarning == null)
-					topWarning = w;
-				else
-					topWarning.setNextWarning(w);
+				addWarning(w);
 			} else
 					concurrencyOfThisResultSet = JDBC20Translation.CONCUR_UPDATABLE;
 		}