git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 1/5] path-list: add functions to work with unsorted lists

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Feb 29, 2008, 01:44 UTC
Message-ID
<alpine.LSU.1.00.0802290144410.22527@racer.site>
In-Reply-To
<alpine.LSU.1.00.0802290144130.22527@racer.site>

Up to now, path-lists were sorted at all times. But sometimes it is much more convenient to build the list and sort it at the end, or sort it not at all.

Add path_list_append() and sort_path_list() to allow that.

Also, add the unsorted_path_list_has_path() function, to do a linear search.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
	This is probably useful to a number of existing non-users of
	path_list.
 path-list.c |   30 ++++++++++++++++++++++++++++++
 path-list.h |    8 +++++++-
 2 files changed, 37 insertions(+), 1 deletions(-)
diff --git a/path-list.c b/path-list.c
index 3d83b7b..92e5cf2 100644
--- a/path-list.c
+++ b/path-list.c
@@ -102,3 +102,33 @@ void print_path_list(const char *text, const struct path_list *p)
 	for (i = 0; i < p->nr; i++)
 		printf("%s:%p\n", p->items[i].path, p->items[i].util);
 }
+
+struct path_list_item *path_list_append(const char *path, struct path_list *list)
+{
+	ALLOC_GROW(list->items, list->nr + 1, list->alloc);
+	list->items[list->nr].path =
+		list->strdup_paths ? xstrdup(path) : (char *)path;
+	return list->items + list->nr++;
+}
+
+static int cmp_items(const void *a, const void *b)
+{
+	const struct path_list_item *one = a;
+	const struct path_list_item *two = b;
+	return strcmp(one->path, two->path);
+}
+
+void sort_path_list(struct path_list *list)
+{
+	qsort(list->items, list->nr, sizeof(*list->items), cmp_items);
+}
+
+int unsorted_path_list_has_path(struct path_list *list, const char *path)
+{
+	int i;
+	for (i = 0; i < list->nr; i++)
+		if (!strcmp(path, list->items[i].path))
+			return 1;
+	return 0;
+}
+
diff --git a/path-list.h b/path-list.h
index 5931e2c..ca2cbba 100644
--- a/path-list.h
+++ b/path-list.h
@@ -13,10 +13,16 @@ struct path_list
 };
 
 void print_path_list(const char *text, const struct path_list *p);
+void path_list_clear(struct path_list *list, int free_util);
 
+/* Use these functions only on sorted lists: */
 int path_list_has_path(const struct path_list *list, const char *path);
-void path_list_clear(struct path_list *list, int free_util);
 struct path_list_item *path_list_insert(const char *path, struct path_list *list);
 struct path_list_item *path_list_lookup(const char *path, struct path_list *list);
 
+/* Use these functions only on unsorted lists: */
+struct path_list_item *path_list_append(const char *path, struct path_list *list);
+void sort_path_list(struct path_list *list);
+int unsorted_path_list_has_path(struct path_list *list, const char *path);
+
 #endif /* PATH_LIST_H */
-- 
1.5.4.3.431.g066fa
Previous: Johannes SchindelinNext: Junio C Hamano
Message 2 of 29 in “Builtin "git remote"”
  1. 0/5 Builtin "git remote"Johannes Schindelin, Feb 29, 2008
  2. 1/5 path-list: add functions to work with unsorted listsJohannes Schindelin, Feb 29, 2008
  3. Junio C HamanoFeb 29, 2008
  4. Johannes SchindelinFeb 29, 2008
  5. 0/6 Builtin remote, v2Johannes Schindelin, Mar 1, 2008
  6. 1/6 Rename path_list to string_listJohannes Schindelin, Mar 1, 2008
  7. David KågedalMar 11, 2008
  8. Johannes SchindelinMar 11, 2008
  9. David KågedalMar 11, 2008
  10. 2/6 string-list: add functions to work with unsorted listsJohannes Schindelin, Mar 1, 2008
  11. 3/6 parseopt: add flag to stop on first non optionJohannes Schindelin, Mar 1, 2008
  12. 4/6 Test "git remote show" and "git remote prune"Johannes Schindelin, Mar 1, 2008
  13. 5/6 Make git-remote a builtinJohannes Schindelin, Mar 1, 2008
  14. 6/6 builtin-remote: prune remotes correctly that were added with --mirrorJohannes Schindelin, Mar 1, 2008
  15. 2/5 parseopt: add flag to stop on first non optionJohannes Schindelin, Feb 29, 2008
  16. 3/5 Test "git remote show" and "git remote prune"Johannes Schindelin, Feb 29, 2008
  17. 4/5 Make git-remote a builtinJohannes Schindelin, Feb 29, 2008
  18. 5/5 builtin-remote: prune remotes correctly that were added with --mirrorJohannes Schindelin, Feb 29, 2008
  19. Junio C HamanoFeb 29, 2008
  20. Johannes SchindelinFeb 29, 2008
  21. Daniel BarkalowFeb 29, 2008
  22. Johannes SchindelinFeb 29, 2008
  23. Johannes SchindelinMar 2, 2008
  24. Johannes SchindelinMar 2, 2008
  25. 0/2 format-patch --cover-letter improvementsJohannes Schindelin, Mar 2, 2008
  26. 1/2 format-patch: use the diff options for the cover letter, tooJohannes Schindelin, Mar 2, 2008
  27. 2/2 format-patch: wrap cover-letter's shortlog sensiblyJohannes Schindelin, Mar 2, 2008
  28. Junio C HamanoMar 2, 2008
  29. Johannes SchindelinMar 2, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.