diff options
| author |  Stefan Beller <sbeller@google.com> | 2017-06-30 12:14:06 -0700 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2017-06-30 13:11:54 -0700 |
| commit | 3da492f8088413c9db2da6c489e117d4808859e9 (patch) | |
| tree | f7c8b8d1600efe3e16e6cd326b5b2cd43295f5a5 /builtin/var.c | |
| parent | hashmap.h: compare function has access to a data field (diff) | |
| download | tgif-3da492f8088413c9db2da6c489e117d4808859e9.tar.xz | |
patch-ids.c: use hashmap correctly
As alluded to in the previous patch, the code in patch-ids.c is
using the hashmaps API wrong.
Luckily we do not have a bug, as all hashmap functionality that we use
here (hashmap_get) passes through the keydata. If hashmap_get_next were
to be used, a bug would occur as that passes NULL for the key_data.
So instead use the hashmap API correctly and provide the caller required
data in the compare function via the first argument that always gets
passed and was setup via the hashmap_init function.
Signed-off-by: Stefan Beller <sbeller@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/var.c')
0 files changed, 0 insertions, 0 deletions