diff options
| author | SZEDER Gábor <szeder.dev@gmail.com> | 2022-11-02 23:01:42 +0100 |
|---|---|---|
| committer | Taylor Blau <me@ttaylorr.com> | 2022-11-02 20:16:34 -0400 |
| commit | 586d8b5052f6b98c262c872f54216e39f3d56625 (patch) | |
| tree | 0753446283aeb4af3dff299127742becb632d759 /builtin/submodule--helper.c | |
| parent | ef84222fa9b3ed163bacd6dc59a1d855dfbfbd75 (diff) | |
| download | git-586d8b5052f6b98c262c872f54216e39f3d56625.tar.gz | |
diff.c: use diff_free_queue()
Use diff_free_queue() instead of open-coding it. This shortens the
code and make it less repetitive.
Note that the second hunk in diff_flush() is interesting, because the
'free_queue' label separates the loop freeing the queue's filepairs
from free()-ing the queue's internal array. This is somewhat
suspicious, but it was not an issue before: there is only one place
from where we jump to this label with a goto, and that is protected by
an 'if (!q->nr && ...)' condition, i.e. we only skipped the loop
freeing the filepairs when there were no filepairs in the queue to
begin with.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Diffstat (limited to 'builtin/submodule--helper.c')
0 files changed, 0 insertions, 0 deletions
