]> granicus.if.org Git - graphviz/commitdiff
colorcmpf: remove strcmp micro-optimization
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 29 Aug 2021 23:19:57 +0000 (16:19 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 4 Sep 2021 02:26:35 +0000 (19:26 -0700)
This is unnecessary when using a modern compiler.

cmd/tools/colxlate.c

index 2139bdc9839dffdde6e6a86629342d1dc77072ab..3604c63929db20d15b523aba21aa1d3476025a47 100644 (file)
@@ -61,8 +61,7 @@ static int colorcmpf(const void *a0, const void *a1)
 {
     const hsbcolor_t *p0 = a0;
     const hsbcolor_t *p1 = a1;
-    int i = (p0->name[0] - p1->name[0]);
-    return (i ? i : strcmp(p0->name, p1->name));
+    return strcmp(p0->name, p1->name);
 }
 
 char *colorxlate(char *str, char *buf)