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

Re: [PATCH] remove delta-against-self bit

From
Junio C Hamano <junkio@cox.net>
Date
Feb 9, 2006, 23:53 UTC
Message-ID
<7v7j846qco.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<Pine.LNX.4.64.0602091736500.5397@localhost.localdomain>
Nicolas Pitre <nico@cam.org> writes:
Show 9 quoted lines
> Said bit would in fact be more useful to allow for encoding the copying 
> of chunks larger than 64KB providing more savings with large files.  
> This will correspond to packs version 3.
>
> While the current code still produces packs version 2, it is made future 
> proof so pack versions 2 and 3 are accepted.  Any pack version 2 are 
> compatible with version 3 since the redefined bit was never used before.  
> When enough time has passed, code to use that bit to produce version 3 
> packs could be added.

I agree with the general direction and this futureproofing is a good thing to have before 1.2.0, I think.

The bit is however _already_ looked at by the count_delta(), to assess the extent of damage, IIRC. Should we be futureproofing that bit as well?

Previous: Nicolas PitreNext: Junio C Hamano
Message 2 of 6 in “remove delta-against-self bit”
  1. remove delta-against-self bitNicolas Pitre, Feb 9, 2006
  2. Junio C HamanoFeb 9, 2006
  3. count-delta.c: Match the delta data semantics change in version 3.Junio C Hamano, Feb 10, 2006
  4. Nicolas PitreFeb 10, 2006
  5. Nicolas PitreFeb 10, 2006
  6. Junio C HamanoFeb 10, 2006

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.