On Feb 19, 2008 5:55 AM, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: > > On Mon, 18 Feb 2008, Jay Soffian wrote: > > > > > if (!tracking.matches) { > > if (track != BRANCH_TRACK_ALWAYS) > > return 1; > > tracking.matches = 1; > > tracking.src = xstrdup(orig_ref); > > } > > Ah, yes. But I still maintain that xstrdup()ing orig_ref only to free it > later is ugly. Why not have the "tracking.src ? tracking.src : orig_ref" > as I suggested? Sorry, I'd missed that. Good idea, will do. j. - 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