On Fri, Dec 04, 2009 at 11:30:56PM -0800, Junio C Hamano wrote: > Two points and a half: > > - Why does info_display need to be visible to everybody else by being in > cache.h and environment.c? Shouldn't its scope be similar to that of > template_file, that is static to builtin-commit.c? > > - A configuration variable that does not allow the users to override from > the command line is a no-no. We usually do command line option first > and then (or at the same time) configuration variable as a typesaver. I can make these changes in the next few days. > - While the amount of change necessary for this change doesn't look too > bad, is it really worth it? What is wrong with "1,$p" while using ed > as your editor? I tried a few variants of this, and shell script wrappers for ex (and gate, which is a specizalized text editor that actually works kind of well for commit message editing), and it worked fairly well. The real motivation is that this feature ditches all assumptions about the capabilities of the text editor. I am not sure where that leads yet, but I'd rather make it possible. James -- James P. Howard, II, MPA MBCS jh@xxxxxxxxxxxxxx -- 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