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

[PATCH 0/2] multiline prefixes on messages

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

This is a short implementation of an idea i have, based on Junio's comment. It might not follows the coding style, the comments can be adverse and so on... Just want to know if it's the right direction.

This series introduces a function that allows to add a prefix to multilined messages. The function itself is an extraction of the code of "advice.c::advice()". Extract this as a separate function has the advantage that we can mark these prefixes for translation and can produce a nicer output.

Junio said that we should probably avoid memory allocation on "die()" codepaths, but "warning" messages are acceptable, I think. If the series is basically acceptable, the "usage:" messages are also a cancidate for migration to this.

Ralf Thielow (2):
  advice: extract function to print messages with prefix
  i18n: mark prefix "warning:" for translation
 advice.c | 14 ++++++++++----
 advice.h |  1 +
 usage.c  |  2 +-
 3 Dateien geändert, 12 Zeilen hinzugefügt(+), 5 Zeilen entfernt(-)
-- 
1.7.12.176.g3fc0e4c.dirty
Previous: Junio C HamanoNext: Ralf Thielow
Message 10 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.