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

[PATCH] promisor-remote.h: fix a 'hdr-check' warning

From
Ramsay Jones <ramsay@ramsayjones.plus.com>
Date
Mar 13, 2019, 19:03 UTC
Message-ID
<88fdcc85-559a-7213-2da8-d4e641ce979a@ramsayjones.plus.com>
Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
---
Hi Christian,

If you need to re-roll your 'cc/multi-promisor' branch, could you please squash this into the relevant patch (commit 9e8a7cf4be ("promisor-remote: implement promisor_remote_get_direct()", 2019-03-12)).

Also, I note that the promisor_remote_new() function is not called outside of 'promisor-remote.c' and so could be a file-local symbol. Are there any plans for future calls to this function (outside of this file)? If not, could you please mark this function as static and remove the extern declaration.

Thanks!

ATB, Ramsay Jones

 promisor-remote.h | 2 ++
 1 file changed, 2 insertions(+)
diff --git a/promisor-remote.h b/promisor-remote.h
index f0d609a3f5..d90df11996 100644
--- a/promisor-remote.h
+++ b/promisor-remote.h
@@ -1,6 +1,8 @@
 #ifndef PROMISOR_REMOTE_H
 #define PROMISOR_REMOTE_H
 
+struct object_id;
+
 /*
  * Promisor remote linked list
  *
-- 
2.21.0
Message 1 of 1 in “promisor-remote.h: fix a 'hdr-check' warning”
  1. promisor-remote.h: fix a 'hdr-check' warningRamsay Jones, Mar 13, 2019

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.