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

Re: [PATCH 1/2] Add `struct object_hash`

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Feb 18, 2007, 11:43 UTC
Message-ID
<Pine.LNX.4.63.0702181241090.22628@wbgn013.biozentrum.uni-wuerzburg.de>
In-Reply-To
<7vtzxjj24j.fsf@assigned-by-dhcp.cox.net>
Hi,
On Sun, 18 Feb 2007, Junio C Hamano wrote:
Show 22 quoted lines
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> 
> > Using object_hash, you can store interesting information about
> > objects in a private hash map. This makes up for the lack of a
> > `util` member of `struct object`.
> >
> > Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> > ---
> >
> > 	We should have done that earlier. Now, git.git already
> > 	contains two specialised versions of an object hash map:
> > 	obj_hash and refs_hash.
> >
> > 	obj_hash is not really a hash map, but rather a hash set,
> > 	but there is no excuse for refs_hash having its own little
> > 	private implementation nobody else can reuse.
> 
> Ok, having now looked at it more, I can say:
> 
>  - I hate it.
> 
> I dislike it intensely, because it's so _close_ to being usable.

Seeing your patch I know why. I could not think of a sane method to condense the two pointers into one. Thank you for pointing out one.

I have not had a close look at it, that will have to wait for tomorrow.

Ciao, Dscho

Previous: Junio C HamanoNext: Johannes Schindelin
Message 3 of 4 in “Add `struct object_hash`”
  1. 1/2 Add `struct object_hash`Johannes Schindelin, Feb 17, 2007
  2. Junio C HamanoFeb 18, 2007
  3. Johannes SchindelinFeb 18, 2007
  4. Johannes SchindelinFeb 18, 2007

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.