* How are VC switches supposed to work?
@ 2008-12-02 8:00 Glenn Morris
2008-12-02 15:29 ` Stefan Monnier
0 siblings, 1 reply; 3+ messages in thread
From: Glenn Morris @ 2008-12-02 8:00 UTC (permalink / raw)
To: emacs-devel
VC switches don't work as they are documented to work.
If someone can confirm that they work correctly, I will try to update
the many incorrect pieces of documentation. If on the other hand the
documentation is correct then the code needs fixing.
The documentation says that switches are appended. For example:
C-h v vc-diff-switches
C-h f vc-cvs-register
The Emacs manual "Examining And Comparing Old Revisions"
all say that it works, eg, as: vc-cvs-diff-switches +
vc-diff-switches + diff-switches.
In practice, (almost) everywhere uses the vc-switches function to
process switches. This does no appending, it simply returns the first
non-nil element.
For example:
(setq vc-cvs-diff-switches "-a"
vc-diff-switches "-b"
diff-switches "-c")
(vc-switches 'CVS 'diff)
-> '("-a")
The situation was basically the same (ie inconsistent) in Emacs 22 AFAICS.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: How are VC switches supposed to work?
2008-12-02 8:00 How are VC switches supposed to work? Glenn Morris
@ 2008-12-02 15:29 ` Stefan Monnier
2008-12-02 19:24 ` Glenn Morris
0 siblings, 1 reply; 3+ messages in thread
From: Stefan Monnier @ 2008-12-02 15:29 UTC (permalink / raw)
To: Glenn Morris; +Cc: emacs-devel
> VC switches don't work as they are documented to work.
> If someone can confirm that they work correctly, I will try to update
> the many incorrect pieces of documentation. If on the other hand the
> documentation is correct then the code needs fixing.
> The documentation says that switches are appended. For example:
> C-h v vc-diff-switches
> C-h f vc-cvs-register
> The Emacs manual "Examining And Comparing Old Revisions"
> all say that it works, eg, as: vc-cvs-diff-switches +
> vc-diff-switches + diff-switches.
> In practice, (almost) everywhere uses the vc-switches function to
> process switches. This does no appending, it simply returns the first
> non-nil element.
> For example:
> (setq vc-cvs-diff-switches "-a"
> vc-diff-switches "-b"
> diff-switches "-c")
> (vc-switches 'CVS 'diff)
> -> '("-a")
> The situation was basically the same (ie inconsistent) in Emacs 22 AFAICS.
I can't remember all the details, but I do remember that appending is
not good enough because you may have switches in diff-switches that are
not accepted by some backends.
Stefan
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: How are VC switches supposed to work?
2008-12-02 15:29 ` Stefan Monnier
@ 2008-12-02 19:24 ` Glenn Morris
0 siblings, 0 replies; 3+ messages in thread
From: Glenn Morris @ 2008-12-02 19:24 UTC (permalink / raw)
To: Stefan Monnier; +Cc: emacs-devel
Stefan Monnier wrote:
> I can't remember all the details, but I do remember that appending is
> not good enough because you may have switches in diff-switches that are
> not accepted by some backends.
Yes; now that I think about it, that is probably enough reason to kill
the "append" approach. Thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2008-12-02 19:24 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-12-02 8:00 How are VC switches supposed to work? Glenn Morris
2008-12-02 15:29 ` Stefan Monnier
2008-12-02 19:24 ` Glenn Morris
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).