Hi Junio, On Mon, Apr 19, 2010 at 10:49 PM, Shawn O. Pearce <spearce@xxxxxxxxxxx> wrote: > Tay Ray Chuan <rctay89@xxxxxxxxx> wrote: >> the small patch below could also be applied to the rebased topic branch. >> >> -->8-- >> From: Tay Ray Chuan <rctay89@xxxxxxxxx> >> Subject: [PATCH] http.c::new_http_pack_request: do away with the temp variable filename >> >> Now that the temporary variable char *filename is only used in one >> place, do away with it and just call sha1_pack_name() directly. >> >> Signed-off-by: Tay Ray Chuan <rctay89@xxxxxxxxx> > > FWIW, Acked-by: Shawn O. Pearce <spearce@xxxxxxxxxxx> I noticed that the inlined patch (in message <20100419224643.00001ff1@unknown>) being acked here wasn't applied to the topic branch 'sp/maint-dumb-http-pack-reidx' in pu; just a heads-up in case you've missed something. -- Cheers, Ray Chuan -- 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