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

[PATCH 1/6] shift end_url_with_slash() from http.[ch] to url.[ch]

From
Tay Ray Chuan <rctay89@gmail.com>
Date
Nov 19, 2010, 12:46 UTC
Message-ID
<1290170790-2200-2-git-send-email-rctay89@gmail.com>
In-Reply-To
<1290170790-2200-1-git-send-email-rctay89@gmail.com>
This allows non-http/curl users to access it too (eg. http-backend.c).
Update include headers in end_url_with_slash() users too.
Signed-off-by: Tay Ray Chuan <rctay89@gmail.com>
---
 Makefile |    2 +-
 http.c   |    8 +-------
 http.h   |    1 -
 url.c    |    7 +++++++
 url.h    |    2 ++
 5 files changed, 11 insertions(+), 9 deletions(-)
diff --git a/Makefile b/Makefile
index ca1d1fe..e02b0f4 100644
--- a/Makefile
+++ b/Makefile
@@ -1922,7 +1922,7 @@ builtin/tar-tree.o archive-tar.o: tar.h
 builtin/pack-objects.o: thread-utils.h
 connect.o transport.o http-backend.o: url.h
 http-fetch.o http-walker.o remote-curl.o transport.o walker.o: walker.h
-http.o http-walker.o http-push.o http-fetch.o remote-curl.o: http.h
+http.o http-walker.o http-push.o http-fetch.o remote-curl.o: http.h url.h
 
 xdiff-interface.o $(XDIFF_OBJS): \
 	xdiff/xinclude.h xdiff/xmacros.h xdiff/xdiff.h xdiff/xtypes.h \
diff --git a/http.c b/http.c
index 17bcf19..310dd86 100644
--- a/http.c
+++ b/http.c
@@ -2,6 +2,7 @@
 #include "pack.h"
 #include "sideband.h"
 #include "run-command.h"
+#include "url.h"
 
 int data_received;
 int active_requests;
@@ -733,13 +734,6 @@ static inline int hex(int v)
 		return 'A' + v - 10;
 }
 
-void end_url_with_slash(struct strbuf *buf, const char *url)
-{
-	strbuf_addstr(buf, url);
-	if (buf->len && buf->buf[buf->len - 1] != '/')
-		strbuf_addstr(buf, "/");
-}
-
 static char *quote_ref_url(const char *base, const char *ref)
 {
 	struct strbuf buf = STRBUF_INIT;
diff --git a/http.h b/http.h
index 173f74c..10de0a5 100644
--- a/http.h
+++ b/http.h
@@ -117,7 +117,6 @@ extern void append_remote_object_url(struct strbuf *buf, const char *url,
 				     int only_two_digit_prefix);
 extern char *get_remote_object_url(const char *url, const char *hex,
 				   int only_two_digit_prefix);
-extern void end_url_with_slash(struct strbuf *buf, const char *url);
 
 /* Options for http_request_*() */
 #define HTTP_NO_CACHE		1
diff --git a/url.c b/url.c
index cd8f74f..7cebc64 100644
--- a/url.c
+++ b/url.c
@@ -125,3 +125,10 @@ char *url_decode_parameter_value(const char **query)
 	struct strbuf out = STRBUF_INIT;
 	return url_decode_internal(query, "&", &out, 1);
 }
+
+void end_url_with_slash(struct strbuf *buf, const char *url)
+{
+	strbuf_addstr(buf, url);
+	if (buf->len && buf->buf[buf->len - 1] != '/')
+		strbuf_addstr(buf, "/");
+}
diff --git a/url.h b/url.h
index 15817f8..8cb74d4 100644
--- a/url.h
+++ b/url.h
@@ -7,4 +7,6 @@ extern char *url_decode(const char *url);
 extern char *url_decode_parameter_name(const char **query);
 extern char *url_decode_parameter_value(const char **query);
 
+extern void end_url_with_slash(struct strbuf *buf, const char *url);
+
 #endif /* URL_H */
-- 
1.7.3.67.g2a10b
Previous: Tay Ray ChuanNext: Tay Ray Chuan
Message 5 of 15 in “Normalise directory names when pushing to some WebDAV servers”
  1. Normalise directory names when pushing to some WebDAV serversGabriel Corona, Nov 19, 2010
  2. Tay Ray ChuanNov 19, 2010
  3. Jonathan NiederNov 19, 2010
  4. 0/6 http (dumb): end urls with slashTay Ray Chuan, Nov 19, 2010
  5. 1/6 shift end_url_with_slash() from http.[ch] to url.[ch]Tay Ray Chuan, Nov 19, 2010
  6. 2/6 url: add str wrapper for end_url_with_slash()Tay Ray Chuan, Nov 19, 2010
  7. 3/6 http-push: Normalise directory names when pushing to some WebDAV serversTay Ray Chuan, Nov 19, 2010
  8. 4/6 http-backend: use end_url_with_slash()Tay Ray Chuan, Nov 19, 2010
  9. 5/6 http-push: check path length before using itTay Ray Chuan, Nov 19, 2010
  10. 6/6 http-push: add trailing slash at arg-parse time, instead of later onTay Ray Chuan, Nov 19, 2010
  11. Jonathan NiederNov 19, 2010
  12. Tay Ray ChuanNov 20, 2010
  13. Jonathan NiederNov 19, 2010
  14. Junio C HamanoNov 19, 2010
  15. Tay Ray ChuanNov 20, 2010

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.