diff options
| author | Phillip Wood <phillip.wood@dunelm.org.uk> | 2026-01-26 10:48:51 +0000 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2026-01-26 08:38:29 -0800 |
| commit | c27afcbfd0f440f410758432e2fe11a16fb2b360 (patch) | |
| tree | 1ca40888eaa253bdb49f468ec119206b51d58967 /contrib/persistent-https | |
| parent | 22ce0cb6397d3d15c21c217696f262c4b8eb44b3 (diff) | |
| download | git-c27afcbfd0f440f410758432e2fe11a16fb2b360.tar.gz git-c27afcbfd0f440f410758432e2fe11a16fb2b360.zip | |
xdiff: remove "line_hash" field from xrecord_t
Prior to commit 6a26019c81 (xdiff: split xrecord_t.ha into line_hash
and minimal_perfect_hash, 2025-11-18) the "ha" field of xrecord_t
initially held the "line_hash" value and once the line had been
interned that field was updated to hold the "minimal_perfect_hash". The
"line_hash" is only used to intern the line so there is no point in
storing it after all the input lines have been interned.
Removing the "line_hash" field from xrecord_t and storing it in
xdlclass_t where it is actually used makes it clearer that it is a
temporary value and it should not be used once we're calculated the
"minimal_perfect_hash". This also reduces the size of xrecord_t by 25%
on 64-bit platforms and 40% on 32-bit platforms. While the struct is
small we create one instance per input line so any saving is welcome.
Signed-off-by: Phillip Wood <phillip.wood@dunelm.org.uk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'contrib/persistent-https')
0 files changed, 0 insertions, 0 deletions
