unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: Dan Nicolaescu <dann@ics.uci.edu>
Cc: Tom Tromey <tromey@redhat.com>,
	esr@thyrsus.com, harsanyi@mac.com, rms@gnu.org,
	emacs-devel@gnu.org
Subject: Re: Introducing 'unrecognized and 'ignored
Date: Sat, 19 Jan 2008 14:40:20 -0500	[thread overview]
Message-ID: <jwv1w8d7gp2.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <200801191705.m0JH5WOU026943@sallyv1.ics.uci.edu> (Dan Nicolaescu's message of "Sat, 19 Jan 2008 09:05:24 -0800")

>> I'm not really sure about using generate-new-buffer-name here.
>> Presumably each vc-status buffer should have a single vc "work" buffer
>> as well.

> IMO yes. I did it that way for vc-hg. But it still needs some
> management: warn if the buffer already exist and refuse to start a new
> status command, remove the buffer after it has been processed.

Yes, maybe Tom is right, sorry for telling you to always use
a new buffer.  Part of the PCL-CVS could probably be reused here because
it shouldn't have any CVS dependency, but then again, it may not be
worth the trouble.

>> After trying this a couple times, I think the user needs an indication
>> of whether anything is running.  pcl-cvs puts this nicely in the
>> buffer... anyway, if you run vc-status on a directory with no changes,
>> it can be a little confusing as-is.

> Yep. vc-status-refresh and vc-update-vc-status-buffer can insert some
> start/end messages similar to PCL-CVS.

> It would be nice if we had something more general that is easily
> visible for buffers that use VC async commands. 

> Something like changing the background of a mode-line entry for VC
> buffers that are waiting for a command to compile. This should make it
> very easy to spot a buffer that is still executing some command.

There's the mode-line-process entry, but it's not visible enough.
Also the PCL-CVS way of putting the actual command in the buffer turned
out to be very useful to double-check what it is doing (and even more
so when several commands are running at the same time).

> log, diff and annotate could all use this functionality.

Indeed, they could use `mode-line-process'.

As for on-the-fly updating rather than updating in the end.  It's been
on my todo list for PCL-CVS, but I never got to it.  It'd probably be
a good idea to add this from the beginning because it substantially
changes the way things work: you have to use a process filter rather than
a sentinel, you have to check whether the partial output you have is
enough to parse it reliably or whether we need to way for more output,
and you have to be able to update the display incrementally.


        Stefan

  reply	other threads:[~2008-01-19 19:40 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-28 17:45 Introducing 'unrecognized and 'ignored Eric S. Raymond
2007-12-28 23:01 ` Dan Nicolaescu
2007-12-29  2:48 ` Alexandru Harsanyi
2007-12-29 11:45   ` Eric S. Raymond
2008-01-02  2:02     ` Stefan Monnier
2008-01-02  2:19       ` Eric S. Raymond
2008-01-02  4:16         ` Stefan Monnier
2008-01-02  4:45           ` Dan Nicolaescu
2008-01-02 11:50             ` Eric S. Raymond
2008-01-02 17:31               ` Dan Nicolaescu
2008-01-03  9:50             ` Richard Stallman
2008-01-03 18:05               ` Dan Nicolaescu
2008-01-03 18:19                 ` Eric S. Raymond
2008-01-05  5:54                 ` Richard Stallman
2008-01-05  9:01                   ` Dan Nicolaescu
2008-01-05 14:34                     ` Eric S. Raymond
2008-01-05 22:25                       ` Stefan Monnier
2008-01-06 10:37                       ` Dan Nicolaescu
2008-01-06 15:57                         ` Eric S. Raymond
2008-01-18 23:31                           ` Dan Nicolaescu
2008-01-06 20:00                         ` Tom Tromey
2008-01-06 21:03                           ` Stefan Monnier
2008-01-07  2:59                             ` Dan Nicolaescu
2008-01-07  3:26                               ` Eric S. Raymond
2008-01-07  3:36                                 ` Dan Nicolaescu
2008-01-07  3:59                                 ` Stefan Monnier
2008-01-07 12:56                                   ` Eric S. Raymond
2008-01-07 15:31                                     ` Stefan Monnier
2008-01-07 11:30                                 ` Richard Stallman
2008-01-07 12:54                                   ` Eric S. Raymond
2008-01-07 15:32                                     ` Stefan Monnier
2008-01-08 19:06                                     ` Richard Stallman
2008-01-08 19:34                                       ` Miles Bader
2008-01-07  3:22                           ` Dan Nicolaescu
2008-01-07  3:03                             ` Tom Tromey
2008-01-07  4:01                               ` Stefan Monnier
2008-01-07 21:15                                 ` PCL-CVS buffers (was: Introducing 'unrecognized and 'ignored) Reiner Steib
2008-01-08  2:33                                   ` PCL-CVS buffers Stefan Monnier
2008-01-18 23:46                           ` Introducing 'unrecognized and 'ignored Dan Nicolaescu
2008-01-19  0:10                             ` Tom Tromey
2008-01-19  1:20                               ` Dan Nicolaescu
2008-01-19  6:00                             ` Tom Tromey
2008-01-19 17:05                               ` Dan Nicolaescu
2008-01-19 19:40                                 ` Stefan Monnier [this message]
2008-01-19 21:01                                   ` Thien-Thi Nguyen
2008-01-20 17:18                                     ` Dan Nicolaescu
2008-01-20 20:24                                       ` Thien-Thi Nguyen
2008-01-20 17:08                                   ` Dan Nicolaescu
2008-01-20 19:08                                   ` Tom Tromey
2008-01-20 20:14                                     ` Stefan Monnier
2008-01-20 19:45                                       ` Tom Tromey
2008-01-22  1:35                                         ` Dan Nicolaescu
2008-01-19 20:03                                 ` Thien-Thi Nguyen
2008-01-20 19:24                                   ` Stefan Monnier
2008-01-20 20:30                                     ` Thien-Thi Nguyen
2008-01-21 15:18                                       ` Stefan Monnier
2008-01-21 15:30                                         ` Dan Nicolaescu
2008-01-21 15:56                                           ` Stefan Monnier
2008-01-21 15:34                                         ` Thien-Thi Nguyen
2008-01-06  8:09                     ` Richard Stallman
2008-01-02 11:46           ` Eric S. Raymond
2008-01-02 20:38             ` Stefan Monnier
2008-01-02 22:11               ` Eric S. Raymond
2008-01-02 23:06                 ` Stefan Monnier
2008-01-02 23:29                   ` Eric S. Raymond
2008-01-03 14:30                     ` Stefan Monnier
2008-01-03 17:41                       ` Eric S. Raymond
2008-01-05  5:54                         ` Richard Stallman
2008-01-02 23:13                 ` Dan Nicolaescu
2008-01-02 23:33                   ` Eric S. Raymond

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=jwv1w8d7gp2.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=dann@ics.uci.edu \
    --cc=emacs-devel@gnu.org \
    --cc=esr@thyrsus.com \
    --cc=harsanyi@mac.com \
    --cc=rms@gnu.org \
    --cc=tromey@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).