git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 2/2] cat-file: handle --batch format with missing type/size

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 12, 2013, 03:05 UTC
Message-ID
<7viouuu5th.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20131211231549.GB16606@sigill.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 6 quoted lines
> Yes. The loss of the optimization was a small thing compared to being
> totally broken. :)
> ...
>> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
>
> Thanks. Updated patch is below.
;-)

I like it when I see patches are polished between the submitter and reviewer(s) fully, before the maintainer has a chance to pick an intermediate version (only to later replace and requeue).

Thanks, both.
Previous: Jonathan Nieder
Message 11 of 11 in “[BUG] "echo HEAD | git cat-file --batch=''" fails catastrophically”
  1. Samuel BronsonDec 11, 2013
  2. Jeff KingDec 11, 2013
  3. 1/2 cat-file: pass expand_data to print_object_or_dieJeff King, Dec 11, 2013
  4. Jonathan NiederDec 11, 2013
  5. Jeff KingDec 11, 2013
  6. Junio C HamanoDec 12, 2013
  7. 2/2 cat-file: handle --batch format with missing type/sizeJeff King, Dec 11, 2013
  8. Jonathan NiederDec 11, 2013
  9. Jeff KingDec 11, 2013
  10. Jonathan NiederDec 11, 2013
  11. Junio C HamanoDec 12, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.