diff options
| author | David Barr <davidbarr@google.com> | 2012-05-31 14:41:26 (GMT) |
|---|---|---|
| committer | Jonathan Nieder <jrnieder@gmail.com> | 2012-07-06 04:26:52 (GMT) |
| commit | 4a1613194af218afb99be0e14af449e86852d06e (patch) | |
| tree | 62d97f9ad6046361f81eeb99ca8ad072367858e9 /strbuf.c | |
| parent | 4a5de8dd79879d737e172a5d7e4196c6bf99e57d (diff) | |
| download | git-4a1613194af218afb99be0e14af449e86852d06e.zip git-4a1613194af218afb99be0e14af449e86852d06e.tar.gz git-4a1613194af218afb99be0e14af449e86852d06e.tar.bz2 | |
vcs-svn: simplify cleanup in apply_one_window
Currently the cleanup code looks like this:
free resources
return 0;
error_out:
free resources
return -1;
Avoid duplicating the "free resources" part by keeping the return
value in a variable and sharing code between the success and
exceptional case:
ret = 0;
out:
free resources
return ret;
Noticed in the svn-dump-fast-export project, where using the error()
macro in void context produces a warning.
Signed-off-by: David Barr <davidbarr@google.com>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Diffstat (limited to 'strbuf.c')
0 files changed, 0 insertions, 0 deletions