On Fri, Feb 08, 2013 at 04:22:15PM -0800, Junio C Hamano wrote: > Junio C Hamano <gitster@xxxxxxxxx> writes: > > > Thomas Haller <thom311@xxxxxxxxx> writes: > > > >> it happens in file revision.c:2306 because "commit->buffer" is zero: > >> > >> retval = grep_buffer(&opt->grep_filter, > >> commit->buffer, strlen(commit->buffer)); > > > > I think this has been fixed at be5c9fb9049e (logmsg_reencode: lazily > > load missing commit buffers, 2013-01-26); I haven't merged it to any > > of the maintenance releases, but the tip of 'master' should have it > > already. > > Ah, no, this shares the same roots as the breakage the said commit > fixed, and the solution should use the same idea, but not fixed. Yeah, I think this needs separate treatment. However, this is a perfect example of the "case-by-case" I mention in the final two paragraphs there. What's the right encoding to be grepping in? The original, what we will output in, or even something else? IOW, I wonder if this should be using logmsg_reencode in the first place (the obvious reason not to want to do so is speed, but logmsg_reencode is written to only have an impact in the case that we do indeed need to reencode). -Peff -- To unsubscribe from this list: 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