Hi, On Mon, 3 Jul 2006, Timo Hirvonen wrote: > Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: > > > I had in mind that I want to use path-list instead (which is cooking in > > the merge-recursive efforts ATM). And there, I would add a flag > > needs_payload. Opinions? > > This code is so simple that making the path_list more complex > (needs_payload special case?) is not worth it. I have not looked at the > code very closely though and have no idea what I'm talking about :) Okay. But I'd rather go back to work on merge-recursive, and just reuse the path_list struct. > > > free(NULL) is safe. > > > > Is it? I vaguely remember that I had problems with this on some obscure > > platform. > > I don't think so. Well, after a little Googling, I am more convinced than ever that it is a BAD thing to rely on free(NULL) being a NOP. Ciao, Dscho - : send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html