By letting strncmp() do its job and not helping it with a prior length

check, we can remove the hard-coded length of all escape patterns.  This
frees up a nice chunk of memory.

1 files changed, 8 insertions, 9 deletions

diff --git a/chars.c b/chars.c
index f7348626..c6d534d6 100644
--- a/chars.c
+++ b/chars.c
@@ -32,7 +32,6 @@
 struct	ln {
 	struct ln	 *next;
 	const char	 *code;
-	size_t		  codesz;
 	const char	 *ascii;
 	size_t		  asciisz;
 	int		  unicode;
@@ -44,12 +43,12 @@ struct	ln {
 
 #define	LINES_MAX	  370
 
-#define CHAR(in, insz, ch, chsz, code) \
-	{ NULL, (in), (insz), (ch), (chsz), (code), CHARS_CHAR },
-#define STRING(in, insz, ch, chsz, code) \
-	{ NULL, (in), (insz), (ch), (chsz), (code), CHARS_STRING },
-#define BOTH(in, insz, ch, chsz, code) \
-	{ NULL, (in), (insz), (ch), (chsz), (code), CHARS_BOTH },
+#define CHAR(in, ch, chsz, code) \
+	{ NULL, (in), (ch), (chsz), (code), CHARS_CHAR },
+#define STRING(in, ch, chsz, code) \
+	{ NULL, (in), (ch), (chsz), (code), CHARS_STRING },
+#define BOTH(in, ch, chsz, code) \
+	{ NULL, (in), (ch), (chsz), (code), CHARS_BOTH },
 
 #define	CHAR_TBL_START	  static struct ln lines[LINES_MAX] = {
 #define	CHAR_TBL_END	  };
@@ -238,7 +237,7 @@ match(const struct ln *ln, const char *p, size_t sz, int type)
 
 	if ( ! (ln->type & type))
 		return(0);
-	if (ln->codesz != sz)
+	if (strncmp(ln->code, p, sz))
 		return(0);
-	return(0 == strncmp(ln->code, p, sz));
+	return('\0' == ln->code[(int)sz]);
 }