Split a log range with git's string list helper

1 file changed, 8 insertions, 28 deletions

diff --git a/source/ui-log.c b/source/ui-log.cindex 3cbbe33..b7804c8 100644--- a/source/ui-log.c+++ b/source/ui-log.c@@ -351,28 +351,6 @@ static int valid_range_token(const char *arg) 	return ok; } -static char *next_token(char **src)-{-	char *token;--	if (!*src)-		return NULL;-	while (isspace((unsigned char)**src))-		(*src)++;-	if (!**src)-		return NULL;-	token = *src;-	while (**src) {-		if (isspace((unsigned char)**src)) {-			**src = '\0';-			(*src)++;-			break;-		}-		(*src)++;-	}-	return token;-}- /*  * Whether there is a page after this one is answered by pulling one more  * commit off the walk. The links carry the path as it was asked for, since a@@ -462,12 +440,13 @@ void cgit_print_commit_decorations(struct commit *commit) 	html("</span>"); } -void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern,+void cgit_print_log(const char *tip, int ofs, int cnt, const char *grep, const char *pattern, 	const char *path, int pager, int commit_graph, int commit_sort, int table_columns) { 	struct rev_info rev; 	struct commit *commit; 	struct strvec rev_argv = STRVEC_INIT;+	struct string_list tokens = STRING_LIST_INIT_DUP; 	int i, columns, filler = 0; 	int must_free_tip = 0; @@ -495,7 +474,6 @@ void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern 	}  	if (grep && pattern && *pattern) {-		pattern = xstrdup(pattern); 		if (!strcmp(grep, "grep") || !strcmp(grep, "author") || !strcmp(grep, "committer")) { 			// The search is literal and ignores case. setup_revisions 			// compiles the pattern, so the flags go in with it.@@ -503,18 +481,19 @@ void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern 			strvec_push(&rev_argv, "--regexp-ignore-case"); 			strvec_pushf(&rev_argv, "--%s=%s", grep, pattern); 		} else if (!strcmp(grep, "range")) {-			char *arg;+			struct string_list_item *token;  			// Each whitespace separated token has to pass the same 			// check as any other revision. The tip pushed above goes 			// away, since the range supersedes it. 			strvec_pop(&rev_argv);-			while ((arg = next_token(&pattern))) {-				if (!valid_range_token(arg)) {+			string_list_split_f(&tokens, pattern, " \t\n\r", -1, STRING_LIST_SPLIT_NONEMPTY);+			for_each_string_list_item(token, &tokens) {+				if (!valid_range_token(token->string)) { 					cgit_print_error_page(400, "Bad Request", "Invalid revision"); 					goto out; 				}-				strvec_push(&rev_argv, arg);+				strvec_push(&rev_argv, token->string); 			} 		} 	}@@ -644,4 +623,5 @@ out: 	if (must_free_tip) 		free((char *)tip); 	strvec_clear(&rev_argv);+	string_list_clear(&tokens, 0); }
diff --git a/source/ui-log.c b/source/ui-log.c
index 3cbbe33..b7804c8 100644
--- a/source/ui-log.c
+++ b/source/ui-log.c
@@ -351,28 +351,6 @@ static int valid_range_token(const char *arg)
 	return ok;
 }
 
-static char *next_token(char **src)
-{
-	char *token;
-
-	if (!*src)
-		return NULL;
-	while (isspace((unsigned char)**src))
-		(*src)++;
-	if (!**src)
-		return NULL;
-	token = *src;
-	while (**src) {
-		if (isspace((unsigned char)**src)) {
-			**src = '\0';
-			(*src)++;
-			break;
-		}
-		(*src)++;
-	}
-	return token;
-}
-
 /*
  * Whether there is a page after this one is answered by pulling one more
  * commit off the walk. The links carry the path as it was asked for, since a
@@ -462,12 +440,13 @@ void cgit_print_commit_decorations(struct commit *commit)
 	html("</span>");
 }
 
-void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern,
+void cgit_print_log(const char *tip, int ofs, int cnt, const char *grep, const char *pattern,
 	const char *path, int pager, int commit_graph, int commit_sort, int table_columns)
 {
 	struct rev_info rev;
 	struct commit *commit;
 	struct strvec rev_argv = STRVEC_INIT;
+	struct string_list tokens = STRING_LIST_INIT_DUP;
 	int i, columns, filler = 0;
 	int must_free_tip = 0;
 
@@ -495,7 +474,6 @@ void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern
 	}
 
 	if (grep && pattern && *pattern) {
-		pattern = xstrdup(pattern);
 		if (!strcmp(grep, "grep") || !strcmp(grep, "author") || !strcmp(grep, "committer")) {
 			// The search is literal and ignores case. setup_revisions
 			// compiles the pattern, so the flags go in with it.
@@ -503,18 +481,19 @@ void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern
 			strvec_push(&rev_argv, "--regexp-ignore-case");
 			strvec_pushf(&rev_argv, "--%s=%s", grep, pattern);
 		} else if (!strcmp(grep, "range")) {
-			char *arg;
+			struct string_list_item *token;
 
 			// Each whitespace separated token has to pass the same
 			// check as any other revision. The tip pushed above goes
 			// away, since the range supersedes it.
 			strvec_pop(&rev_argv);
-			while ((arg = next_token(&pattern))) {
-				if (!valid_range_token(arg)) {
+			string_list_split_f(&tokens, pattern, " \t\n\r", -1, STRING_LIST_SPLIT_NONEMPTY);
+			for_each_string_list_item(token, &tokens) {
+				if (!valid_range_token(token->string)) {
 					cgit_print_error_page(400, "Bad Request", "Invalid revision");
 					goto out;
 				}
-				strvec_push(&rev_argv, arg);
+				strvec_push(&rev_argv, token->string);
 			}
 		}
 	}
@@ -644,4 +623,5 @@ out:
 	if (must_free_tip)
 		free((char *)tip);
 	strvec_clear(&rev_argv);
+	string_list_clear(&tokens, 0);
 }