strchr() and strrchr() are already present in the kernel, but with less

popular names. Hence:

- comment current index() and rindex() functions, as these serve the same
  functionality as, respectively, strchr() and strrchr() from userland;
- add inlined version of strchr() and strrchr(), as we tend to use them more
  often;
- remove str[r]chr() definitions from ZFS code;

Reviewed by:	pjd
Approved by:	cognet (mentor)

2 files changed, 2 insertions, 33 deletions

diff --git a/sys/compat/opensolaris/kern/opensolaris_string.c b/sys/compat/opensolaris/kern/opensolaris_string.c
index cae984f..4448f34 100644
--- a/sys/compat/opensolaris/kern/opensolaris_string.c
+++ b/sys/compat/opensolaris/kern/opensolaris_string.c
@@ -32,37 +32,6 @@
 	(((c) >= 'a' && (c) <= 'z') || ((c) >= 'A' && (c) <= 'Z'))
 
 char *
-strchr(const char *s, int c)
-{
-	char ch;
-
-	ch = c;
-	for (;; ++s) {
-		if (*s == ch)
-			return ((char *)s);
-		if (*s == '\0')
-			return (NULL);
-	}
-	/* NOTREACHED */
-}
-
-char *
-strrchr(const char *s, int c)
-{
-	char *save;
-	char ch;
-
-	ch = c;
-	for (save = NULL;; ++s) {
-		if (*s == ch)
-			save = (char *)s;
-		if (*s == '\0')
-			return (save);
-	}
-	/* NOTREACHED */
-}
-
-char *
 strpbrk(const char *s, const char *b)
 {
 	const char *p;
diff --git a/sys/compat/opensolaris/sys/string.h b/sys/compat/opensolaris/sys/string.h
index c802c0f..aeec929 100644
--- a/sys/compat/opensolaris/sys/string.h
+++ b/sys/compat/opensolaris/sys/string.h
@@ -29,8 +29,8 @@
 #ifndef _OPENSOLARIS_SYS_STRING_H_
 #define	_OPENSOLARIS_SYS_STRING_H_
 
-char	*strchr(const char *, int);
-char	*strrchr(const char *p, int c);
+#include <sys/libkern.h>
+
 char	*strpbrk(const char *, const char *);
 void	 strident_canon(char *s, size_t n);