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)