]> granicus.if.org Git - graphviz/commitdiff
update legacy 'qsort_r' comment
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 15 Oct 2021 00:18:06 +0000 (17:18 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 16 Oct 2021 18:18:41 +0000 (11:18 -0700)
The problem of “call `qsort` but also access some external context” is still
relevant, but this functionality has been standardized into `qsort_s` in C11,
not `qsort_r`.

lib/neatogen/delaunay.c

index 61f8ea6c3f792477fee6d05e73b6ebb00aba0d7a..6f13c6b33c631ff8aa9dc4e2aa8b445bd9610b17 100644 (file)
@@ -303,9 +303,8 @@ static void addEdge (GtsSegment * e, estate* es)
     es->n += 1;
 }
 
-/* If qsort_r ever becomes standardized, this should be used
- * instead of having a global variable.
- */
+// when moving to C11, qsort_s should be used instead of having a global
+// variable
 static double* _vals;
 typedef int (*qsort_cmpf) (const void *, const void *);