Hi Jonathan, Jonathan Nieder writes: > Hmm, this seems to be trying to have it both ways. It would be simpler to > either: > > static int merge_summary; > static int log_limit = 20; > > providing independent internal "enabled" and "limit" knobs, so one could use, > say, > > [merge] > log = 2 > log = false > log = true > > with the result being be a log_limit of 2, or > > static int log_limit; > > where 0 means disabled, so in that example the result would be a log_limit > of 20. > [...] > What happened to the merge_summary argument? This patch is roughly equivalent to my original log_limit patch. Yes, I'm trying to have it both ways in this patch :) merge_summary is set by both the command-line option and the config option, while log_limit is set by the config option. As you noticed, the next patch removes merge_summary. -- Ram -- 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