[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