You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by yl...@apache.org on 2015/03/13 23:49:24 UTC

svn commit: r1666601 - /apr/apr/branches/1.5.x/test/testskiplist.c

Author: ylavic
Date: Fri Mar 13 22:49:23 2015
New Revision: 1666601

URL: http://svn.apache.org/r1666601
Log:
testskiplist: fix apr_skiplist_add() function pointer dereference.

Modified:
    apr/apr/branches/1.5.x/test/testskiplist.c

Modified: apr/apr/branches/1.5.x/test/testskiplist.c
URL: http://svn.apache.org/viewvc/apr/apr/branches/1.5.x/test/testskiplist.c?rev=1666601&r1=1666600&r2=1666601&view=diff
==============================================================================
--- apr/apr/branches/1.5.x/test/testskiplist.c (original)
+++ apr/apr/branches/1.5.x/test/testskiplist.c Fri Mar 13 22:49:23 2015
@@ -49,10 +49,10 @@ static apr_skiplistnode *apr_skiplist_ad
 }
 static apr_skiplistnode *apr_skiplist_add(apr_skiplist *sl, void *data)
 {
-    /* Ugly, really, but should work *while* the compare function is the first
-     * field of the (opaque) skiplist struct, which is the case for now :p
+    /* Ugly, really, but should work *as long as* the compare function is the
+     * first field of the (opaque) skiplist struct, this is the case for now :p
      */
-    return apr_skiplist_add_compare(sl, data, *(apr_skiplist_compare**)sl);
+    return apr_skiplist_add_compare(sl, data, *(apr_skiplist_compare*)sl);
 }
 
 static int skiplist_get_size(abts_case *tc, apr_skiplist *sl)