]> granicus.if.org Git - graphviz/commitdiff
cgraph strview_str_contains: abbreviate prefix check
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 15 Oct 2022 21:25:22 +0000 (14:25 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 3 Nov 2022 00:13:35 +0000 (17:13 -0700)
After this function was written, the `startswith` helper was introduced for
abbreviating the idiom `strncmp(foo, bar, strlen(bar)) == 0`. We can use it to
more concisely and clearly express the intent of this code.

lib/cgraph/strview.h

index 8e4d593adeea440df5485ba46b74f1b8aba0bfb6..28c78af589b15f99590a717a110af2fbfc2acbfd 100644 (file)
@@ -9,6 +9,7 @@
 
 #include <assert.h>
 #include <cgraph/alloc.h>
+#include <cgraph/startswith.h>
 #include <cgraph/strcasecmp.h>
 #include <stdbool.h>
 #include <stddef.h>
@@ -128,7 +129,7 @@ static inline bool strview_str_contains(strview_t haystack,
     }
 
     // is it a match?
-    if (strncmp(candidate, needle, strlen(needle)) == 0) {
+    if (startswith(candidate, needle)) {
       return true;
     }