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

[PATCH 1/2] advice: extract function to print messages with prefix

From
Ralf Thielow <ralf.thielow@gmail.com>
Date
Sep 7, 2012, 19:05 UTC
Message-ID
<1347044705-17268-2-git-send-email-ralf.thielow@gmail.com>
In-Reply-To
<1347044705-17268-1-git-send-email-ralf.thielow@gmail.com>

Extract a function that allows to print messages with a prefix.

Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com>
---
 advice.c | 14 ++++++++++----
 advice.h |  1 +
 2 Dateien geändert, 11 Zeilen hinzugefügt(+), 4 Zeilen entfernt(-)
diff --git a/advice.c b/advice.c
index edfbd4a..e73d53b 100644
--- a/advice.c
+++ b/advice.c
@@ -25,25 +25,31 @@ static struct {
 	{ "detachedhead", &advice_detached_head },
 };
 
-void advise(const char *advice, ...)
+void print_with_prefix(const char *prefix, const char *msg, ...)
 {
 	struct strbuf buf = STRBUF_INIT;
 	va_list params;
 	const char *cp, *np;
 
-	va_start(params, advice);
-	strbuf_vaddf(&buf, advice, params);
+	va_start(params, msg);
+	strbuf_vaddf(&buf, msg, params);
 	va_end(params);
 
 	for (cp = buf.buf; *cp; cp = np) {
 		np = strchrnul(cp, '\n');
-		fprintf(stderr,	_("hint: %.*s\n"), (int)(np - cp), cp);
+		fprintf(stderr,	 "%s %.*s\n", prefix, (int)(np - cp), cp);
 		if (*np)
 			np++;
 	}
 	strbuf_release(&buf);
 }
 
+void advise(const char *advice, ...)
+{
+	va_list vl;
+	print_with_prefix(_("hint:"), advice, vl);
+}
+
 int git_default_advice_config(const char *var, const char *value)
 {
 	const char *k = skip_prefix(var, "advice.");
diff --git a/advice.h b/advice.h
index f3cdbbf..328e255 100644
--- a/advice.h
+++ b/advice.h
@@ -14,6 +14,7 @@ extern int advice_implicit_identity;
 extern int advice_detached_head;
 
 int git_default_advice_config(const char *var, const char *value);
+void print_with_prefix(const char *prefix, const char *msg, ...);
 void advise(const char *advice, ...);
 int error_resolve_conflict(const char *me);
 extern void NORETURN die_resolve_conflict(const char *me);
-- 
1.7.12.176.g3fc0e4c.dirty
Previous: Ralf ThielowNext: Junio C Hamano
Message 11 of 18 in “l10n: de.po: translate 2 new messages”
  1. l10n: de.po: translate 2 new messagesRalf Thielow, Sep 5, 2012
  2. Ralf ThielowSep 5, 2012
  3. Thomas RastSep 6, 2012
  4. Ralf ThielowSep 6, 2012
  5. Jiang XinSep 7, 2012
  6. Ralf ThielowSep 7, 2012
  7. Ralf ThielowSep 7, 2012
  8. Ralf ThielowSep 7, 2012
  9. Junio C HamanoSep 7, 2012
  10. 0/2 multiline prefixes on messagesRalf Thielow, Sep 7, 2012
  11. 1/2 advice: extract function to print messages with prefixRalf Thielow, Sep 7, 2012
  12. Junio C HamanoSep 7, 2012
  13. Ralf ThielowSep 7, 2012
  14. Junio C HamanoSep 7, 2012
  15. Ralf ThielowSep 7, 2012
  16. Junio C HamanoSep 7, 2012
  17. Junio C HamanoSep 7, 2012
  18. 2/2 i18n: mark prefix "warning:" for translationRalf Thielow, Sep 7, 2012

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.