From 0b82f36a9ca5d1ec455633a57e2d996699726fbc Mon Sep 17 00:00:00 2001 From: eadler Date: Tue, 22 Nov 2011 02:50:24 +0000 Subject: - fix some style(9) nits with my last commit - add a comment explaining why I used '|' instead of '||' Submitted by: danfe@ Approved by: emaste@ --- lib/libc/string/strcasecmp.c | 7 ++++--- lib/libc/string/strncmp.c | 1 + 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/libc/string/strcasecmp.c b/lib/libc/string/strcasecmp.c index dae91d3..582399d 100644 --- a/lib/libc/string/strcasecmp.c +++ b/lib/libc/string/strcasecmp.c @@ -49,7 +49,7 @@ strcasecmp_l(const char *s1, const char *s2, locale_t locale) *us1 = (const u_char *)s1, *us2 = (const u_char *)s2; if (s1 == s2) - return (0); + return (0); FIX_LOCALE(locale); @@ -73,8 +73,9 @@ strncasecmp_l(const char *s1, const char *s2, size_t n, locale_t locale) *us1 = (const u_char *)s1, *us2 = (const u_char *)s2; - if (( s1 == s2) | (n == 0)) - return (0); + /* use a bitwise or to avoid an additional branch instruction */ + if ((s1 == s2) | (n == 0)) + return (0); do { diff --git a/lib/libc/string/strncmp.c b/lib/libc/string/strncmp.c index 46db3f9..8da2c43 100644 --- a/lib/libc/string/strncmp.c +++ b/lib/libc/string/strncmp.c @@ -39,6 +39,7 @@ int strncmp(const char *s1, const char *s2, size_t n) { + /* use a bitwise or to avoid an additional branch instruction */ if ((n == 0) | (s1 == s2)) return (0); -- cgit v1.1