On Wed, May 29, 2013 at 8:25 AM, Duy Nguyen <[email protected]> wrote: > On Mon, May 27, 2013 at 11:52 PM, Felipe Contreras > <[email protected]> wrote: >> We should free objects before leaving. >> >> Signed-off-by: Felipe Contreras <[email protected]> > > Micronit: perhaps you should move the "free obejcts before leaving" > (in do_pick_commit) to the subject instead of "trivial fix", which > adds no value to the patch.
Perhaps. I prefer it this way because it's really a trivial fix not really worth much time thinking about it. So when somebody is browsing the history they can happily skip this one. The time save by not reading I think adds more value than any succinct description that would force each and every patch-reviewer/history-reader to read it. -- Felipe Contreras -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
