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 my...@apache.org on 2008/08/15 13:24:20 UTC

svn commit: r686193 - /db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/TableScanResultSet.java

Author: myrnavl
Date: Fri Aug 15 04:24:19 2008
New Revision: 686193

URL: http://svn.apache.org/viewvc?rev=686193&view=rev
Log:
DERBY-3798: remove unnecessary call to Hashtable.get()
  Patch contributed by Junjie Peng

Modified:
    db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/TableScanResultSet.java

Modified: db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/TableScanResultSet.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/TableScanResultSet.java?rev=686193&r1=686192&r2=686193&view=diff
==============================================================================
--- db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/TableScanResultSet.java (original)
+++ db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/TableScanResultSet.java Fri Aug 15 04:24:19 2008
@@ -706,10 +706,8 @@
 					if (past2FutureTbl != null)
 					{
 						RowLocation rowLoc = (RowLocation) currentRow.getColumn(currentRow.nColumns());
-						if (past2FutureTbl.get(rowLoc) != null)
-						{
-							past2FutureTbl.remove(rowLoc);
-							continue;
+						if (past2FutureTbl.remove(rowLoc) != null){
+						    continue;
 						}
 					}
 
@@ -1260,7 +1258,6 @@
 								 GeneratedMethod positionGetter,
 								 ExecIndexRow positioner)
 	{
-                String idt = "";
 		String output = "";
 		if (positionGetter == null)
 		{