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

Re: [PATCH] index-pack: correctly initialize appended objects

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jul 25, 2008, 10:24 UTC
Message-ID
<alpine.DEB.1.00.0807251219530.11976@eeepc-johanness>
In-Reply-To
<7vy73q4jzp.fsf@gitster.siamese.dyndns.org>
Hi,
On Thu, 24 Jul 2008, Junio C Hamano wrote:
Show 6 quoted lines
> The function does not seem to use type (which the patch is also setting) 
> nor real_type (which the patch does not set).
> 
> However, the code checks objects[nth].real_type all over the place in 
> the code.  Doesn't the lack of real_type assignment in 
> append_obj_to_pack() affect them in any way?
>From staring at the code, I thought that real_type was set in 
resolve_delta(), but I may be wrong.

The safer thing would be to set it, but I am not quite sure if we can use "type" directly, or if type can be "delta" for an object that is used to complete the pack, and therefore stored as a non-delta.

Ciao, Dscho

Previous: Junio C HamanoNext: Nicolas Pitre
Message 4 of 15 in “index-pack: correctly initialize appended objects”
  1. index-pack: correctly initialize appended objectsJohannes Schindelin, Jul 24, 2008
  2. Björn SteinbrinkJul 24, 2008
  3. Junio C HamanoJul 25, 2008
  4. Johannes SchindelinJul 25, 2008
  5. Nicolas PitreJul 25, 2008
  6. Björn SteinbrinkJul 25, 2008
  7. Nicolas PitreJul 25, 2008
  8. Junio C HamanoJul 25, 2008
  9. Björn SteinbrinkJul 25, 2008
  10. Johannes SchindelinJul 25, 2008
  11. Shawn O. PearceJul 25, 2008
  12. index-pack: correctly initialize appended objectsBjörn Steinbrink, Jul 25, 2008
  13. Shawn O. PearceJul 25, 2008
  14. Johannes SchindelinJul 26, 2008
  15. Nicolas PitreJul 25, 2008

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.