]> git.ipfire.org Git - thirdparty/git.git/commit
rev-list-options: fix off-by-one in '--filter=blob:limit=<n>' explainer
authorNikolay Edigaryev <edigaryev@gmail.com>
Sun, 14 Jan 2024 19:50:50 +0000 (19:50 +0000)
committerJunio C Hamano <gitster@pobox.com>
Tue, 16 Jan 2024 16:53:13 +0000 (08:53 -0800)
commit8f50984cf4e1c7562cd96a4064dcdfef181d6b97
treefba234e46e8ee370c6eea6515adf86268f4ac146
parentec583449067bab5b800ecc63926f35c9dae96fa1
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>
Documentation/rev-list-options.txt