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

Re: [PATCH 1/2] cat-file: pass expand_data to print_object_or_die

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Dec 11, 2013, 20:11 UTC
Message-ID
<20131211201112.GM2311@google.com>
In-Reply-To
<20131211115642.GA10594@sigill.intra.peff.net>
Hi,
Jeff King wrote:
>                                        However, if we pass
> the whole struct, the called function can make more
> intelligent decisions about which fields were actualled
> filled by sha1_object_info.
Thanks.
s/actualled/actually/, I think.

At first I thought this patch was going to be about making those intelligent decisions. Maybe s/the called function can/a future patch can teach the called function/ or something?

[...]
> There should be no functional change to this patch.
The patch itself looks straightforward, yep. :)
With the typofix mentioned above,
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Previous: Jeff KingNext: Jeff King
Message 4 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.