diff options
| author | Nikolay Edigaryev <edigaryev@gmail.com> | 2024-01-14 19:50:50 +0000 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2024-01-16 08:53:13 -0800 |
| commit | 8f50984cf4e1c7562cd96a4064dcdfef181d6b97 (patch) | |
| tree | fba234e46e8ee370c6eea6515adf86268f4ac146 /commit-graph.c | |
| parent | Git 2.38.5 (diff) | |
| download | git-8f50984cf4e1c7562cd96a4064dcdfef181d6b97.tar.gz git-8f50984cf4e1c7562cd96a4064dcdfef181d6b97.zip | |
rev-list-options: fix off-by-one in '--filter=blob:limit=<n>' explainer
'--filter=blob:limit=<n>' was introduced in 25ec7bcac0 (list-objects:
filter objects in traverse_commit_list, 2017-11-21) and later expanded
to bitmaps in 84243da129 (pack-bitmap: implement BLOB_LIMIT filtering,
2020-02-14)
The logic that was introduced in these commits (and that still persists
to this day) omits blobs larger than _or equal_ to n bytes or units.
However, the documentation (Documentation/rev-list-options.txt) states:
>The form '--filter=blob:limit=<n>[kmg]' omits blobs larger than n
bytes or units. n may be zero.
Moreover, the t6113-rev-list-bitmap-filters.sh tests for exactly this
logic, so it seems it is the documentation that needs fixing, not the
code.
This changes the explanation to be similar to
Documentation/git-clone.txt, which is correct.
Signed-off-by: Nikolay Edigaryev <edigaryev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'commit-graph.c')
0 files changed, 0 insertions, 0 deletions