Re: [PATCH 0/4] rev-list: introduce NUL-delimited output mode
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Mar 11, 2025, 23:44 UTC
- Message-ID
- <xmqq1pv32kvp.fsf@gitster.g>
- In-Reply-To
- <nfrfyg56nq7m2bcko4v4tsfdklrafpkybc6uhqgo25swdeebh3@a7aneoylg46u>
Justin Tobler <jltobler@gmail.com> writes:
Show 8 quoted lines
>> (4) The traditional "rev-list --missing" output loses the leading >> "?"; it is replaced by "missing" as the <attr> name, i.e. such >> a record may look like "<oid> NUL missing=yes NUL..." together >> with other "<token>=<value> NUL" pairs appended as needed at >> the end. > > I think this is good. Instead of prefixing missing OIDs with '?', we can > just append another token/value pair `missing=yes`.
And we may want to avoid excessive bloat of the output that is not primarily meant for human consumption, in which case perhaps a common things like "missing" and "path" can be given a shorter token, perhaps like "m=yes" and "p=Makefile"?