emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: trunk r115265: * lisp/vc/vc-dispatcher.el (vc-log-edit): Setup the S


From: Dmitry Gutov
Subject: Re: trunk r115265: * lisp/vc/vc-dispatcher.el (vc-log-edit): Setup the Summary&Author headers.
Date: Fri, 29 Nov 2013 06:39:50 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux)

Stefan Monnier <address@hidden> writes:

> revno: 115265
>   * lisp/vc/vc-dispatcher.el (vc-log-edit): Setup the Summary&Author
> headers.

This looks like a fix of previously-implemented functionality, but I'm
not sure I like the result.

Pros: `log-edit-insert-changelog' and `log-edit-show-files' are nifty.

Cons:

1. The "Author" header will be almost always unused in other projects,
aside from Emacs, I'm contributing to. So it'll stick out like a sore
thumb. And even with Emacs, when the ChangeLog entry has the proper
author mentioned, it's auto-filled anyway.

2. The "Summary" doesn't add anything in the way of features, I can just
as well write it in the first line of the message.

When the buffer is displayed initially, the point is before "Summary:",
so now I have to move it first before I can write the commit message.

`vc-git-log-edit-toggle-amend' is not aware of this header and inserts
the whole message, summary and the rest, below the headers, leaving
"Summary" empty. Although this should be fixable.

3. If I decide I don't want to commit now, and I kill the log-edit
buffer (is there any other way to dismiss it?), the window and buffer
created by `log-edit-show-files', doesn't go anywhere, and I have to
take care of it separately. Also fixable.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]