Update of /cvsroot/monetdb/sql/src/server
In directory sfp-cvsdas-1.v30.ch3.sourceforge.com:/tmp/cvs-serv24298

Modified Files:
      Tag: Nov2009
        rel_optimizer.mx 
Log Message:
Removed unnecessary test for n!=0.  Because of the way pos is
initialized, the value *must* occur.
Found by Coverity.


Index: rel_optimizer.mx
===================================================================
RCS file: /cvsroot/monetdb/sql/src/server/rel_optimizer.mx,v
retrieving revision 1.71.2.5
retrieving revision 1.71.2.6
diff -u -d -r1.71.2.5 -r1.71.2.6
--- rel_optimizer.mx    26 Nov 2009 21:55:53 -0000      1.71.2.5
+++ rel_optimizer.mx    2 Dec 2009 11:49:29 -0000       1.71.2.6
@@ -507,7 +507,7 @@
        /* sort descending */
        GDKqsort_rev(keys, pos, NULL, cnt, sizeof(int), sizeof(int), TYPE_int);
        for(j=0; j<cnt; j++) {
-               for(n = dje->h, i = 0; n && i != pos[j]; n = n->next, i++) 
+               for(n = dje->h, i = 0; i != pos[j]; n = n->next, i++) 
                        ;
                list_append(res, exp_dup(n->data));
        }


------------------------------------------------------------------------------
Join us December 9, 2009 for the Red Hat Virtual Experience,
a free event focused on virtualization and cloud computing. 
Attend in-depth sessions from your desk. Your couch. Anywhere.
http://p.sf.net/sfu/redhat-sfdev2dev
_______________________________________________
Monetdb-sql-checkins mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-sql-checkins

Reply via email to