summary refs log tree commit diff
diff options
context:
space:
mode:
authorJune McEnroe <june@causal.agency>2019-11-10 23:02:59 -0500
committerJune McEnroe <june@causal.agency>2019-11-10 23:02:59 -0500
commit8fe37cf890168ec043199098326d5bb327f10617 (patch)
tree9c79eace33d938b2cd72608b13af63932486b171
parentSeparate tags from all target (diff)
downloadpounce-8fe37cf890168ec043199098326d5bb327f10617.tar.gz
pounce-8fe37cf890168ec043199098326d5bb327f10617.zip
Compare words without copying in filters
-rw-r--r--client.c25
1 files changed, 13 insertions, 12 deletions
diff --git a/client.c b/client.c
index 22106e3..201566e 100644
--- a/client.c
+++ b/client.c
@@ -277,44 +277,45 @@ size_t clientDiff(const struct Client *client) {
 
 typedef const char *Filter(const char *line);
 
-static const char *word(size_t i, const char *line) {
-	if (*line != ':') i--;
+static int wordcmp(const char *line, size_t i, const char *word) {
 	for (; i; --i) {
 		line += strcspn(line, " ");
 		if (*line) line++;
 	}
-	static char buf[512];
-	snprintf(buf, sizeof(buf), "%.*s", (int)strcspn(line, " "), line);
-	return buf;
+	size_t len = strcspn(line, " ");
+	if (len < strlen(word)) return -1;
+	if (len > strlen(word)) return +1;
+	return strncmp(line, word, len);
 }
 
 static const char *filterAccountNotify(const char *line) {
-	return (strcmp(word(1, line), "ACCOUNT") ? line : NULL);
+	return (wordcmp(line, 1, "ACCOUNT") ? line : NULL);
 }
 
 static const char *filterAwayNotify(const char *line) {
-	return (strcmp(word(1, line), "AWAY") ? line : NULL);
+	return (wordcmp(line, 1, "AWAY") ? line : NULL);
 }
 
 static const char *filterChghost(const char *line) {
-	return (strcmp(word(1, line), "CHGHOST") ? line : NULL);
+	return (wordcmp(line, 1, "CHGHOST") ? line : NULL);
 }
 
 static const char *filterExtendedJoin(const char *line) {
-	if (strcmp(word(1, line), "JOIN")) return line;
+	if (wordcmp(line, 1, "JOIN")) return line;
 	size_t len = 0;
 	for (int i = 0; i < 3; ++i) {
 		len += strcspn(&line[len], " ");
 		if (line[len]) len++;
 	}
 	static char buf[512];
-	snprintf(buf, sizeof(buf), "%.*s", (int)len, line);
+	assert(len < sizeof(buf));
+	strncpy(buf, line, len);
 	return buf;
 }
 
 static const char *filterInviteNotify(const char *line) {
-	if (strcmp(word(1, line), "INVITE")) return line;
-	return (strcmp(word(2, line), stateNick()) ? NULL : line);
+	if (wordcmp(line, 1, "INVITE")) return line;
+	return (wordcmp(line, 2, stateNick()) ? NULL : line);
 }
 
 static Filter *Filters[] = {