diff options
| author |  Jonathan Nieder <jrnieder@gmail.com> | 2014-09-02 14:10:52 -0700 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2014-09-03 10:04:19 -0700 |
| commit | ab5ac95725f7a4f36f72be48f340a84e46996e90 (patch) | |
| tree | a3496136ea02fecc20b771a7128dfb8845861ee8 /builtin/grep.c | |
| parent | refs.c: make delete_ref use a transaction (diff) | |
| download | tgif-ab5ac95725f7a4f36f72be48f340a84e46996e90.tar.xz | |
update-ref --stdin: narrow scope of err strbuf
Making the strbuf local in each function that needs to print errors
saves the reader from having to think about action at a distance,
such as
* errors piling up and being concatenated with no newline between
them
* errors unhandled in one function, to be later handled in another
* concurrency issues, if this code starts using threads some day
No functional change intended.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Reviewed-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/grep.c')
0 files changed, 0 insertions, 0 deletions