Re: [PATCH 37/38] pack v4: introduce "escape hatches" in the name and path indexes

From: Nicolas Pitre <nico@fluxnic.net>
Date: Sep 5, 2013, 21:48 UTC
Message-ID: <alpine.LFD.2.03.1309051744490.20709@syhkavp.arg>
In-Reply-To: <alpine.LFD.2.03.1309051445140.14472@syhkavp.arg>

On Thu, 5 Sep 2013, Nicolas Pitre wrote:

Show 11 quoted lines

> On Thu, 5 Sep 2013, Nicolas Pitre wrote:
> 
> > If the path or name index is zero, this means the entry data is to be
> > found inline rather than being located in the dictionary table. This is
> > there to allow easy completion of thin packs without having to add new
> > table entries which would have required a full rewrite of the pack data.
> > 
> > Signed-off-by: Nicolas Pitre <nico@fluxnic.net>
> 
> I'm now dropping this patch.  Please also remove this from your 
> documentation patch.

Well... I couldn't resist another little change that has been nagging me for a while.

Both the author and committer time stamps are very closely related most of the time. So the committer time stamp is now encoded as a difference against the author time stamp with the LSB indicating a negative difference.

On git.git this saves 0.3% on the pack size. Not much, but still impressive for only a time stamp.

Nicolas