unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#14341: adding an auto toggle update keybinding to proced
@ 2013-05-03  9:00 Nic Ferrier
  2013-05-08  7:20 ` Glenn Morris
  0 siblings, 1 reply; 6+ messages in thread
From: Nic Ferrier @ 2013-05-03  9:00 UTC (permalink / raw)
  To: 14341

ChangeLog entry:

2013-05-03  Nic Ferrier  <nferrier@gnu.org>

	* proced.el: Add a keybinding for proced-toggle-auto-update


Patch:

diff -cp -L /home/nferrier/emacs-local-20130401/share/emacs/24.3/lisp/proced.el.gz.\~1\~ -L /home/nferrier/emacs-local-20130401/share/emacs/24.3/lisp/proced.el.gz /tmp/jka-com2557wvb /tmp/jka-com2557KEo
*** /home/nferrier/emacs-local-20130401/share/emacs/24.3/lisp/proced.el.gz.~1~
--- /home/nferrier/emacs-local-20130401/share/emacs/24.3/lisp/proced.el.gz
*************** Important: the match ends just after the
*** 494,499 ****
--- 494,500 ----
      (define-key km [header-line mouse-1] 'proced-sort-header)
      (define-key km [header-line mouse-2] 'proced-sort-header)
      (define-key km "T" 'proced-toggle-tree)
+     (define-key km "U" 'proced-toggle-auto-update)
      ;; formatting
      (define-key km "F"  'proced-format-interactive)
      ;; operate

Diff finished.  Fri May  3 09:51:13 2013





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#14341: adding an auto toggle update keybinding to proced
  2013-05-03  9:00 bug#14341: adding an auto toggle update keybinding to proced Nic Ferrier
@ 2013-05-08  7:20 ` Glenn Morris
  2013-05-08 10:57   ` Daimrod
  0 siblings, 1 reply; 6+ messages in thread
From: Glenn Morris @ 2013-05-08  7:20 UTC (permalink / raw)
  To: Nic Ferrier; +Cc: 14341, winkler

Nic Ferrier wrote:

> +     (define-key km "U" 'proced-toggle-auto-update)

The U key is already in use.

(I'm not sure if proced-toggle-auto-update needs a keybinding, but I
don't use proced.)





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#14341: adding an auto toggle update keybinding to proced
  2013-05-08  7:20 ` Glenn Morris
@ 2013-05-08 10:57   ` Daimrod
  2013-05-08 12:08     ` Nic Ferrier
  2013-05-15  3:34     ` Roland Winkler
  0 siblings, 2 replies; 6+ messages in thread
From: Daimrod @ 2013-05-08 10:57 UTC (permalink / raw)
  To: Glenn Morris; +Cc: 14341, winkler, Nic Ferrier

Glenn Morris <rgm@gnu.org> writes:

> Nic Ferrier wrote:
>
>> +     (define-key km "U" 'proced-toggle-auto-update)
>
> The U key is already in use.
>
> (I'm not sure if proced-toggle-auto-update needs a keybinding, but I
> don't use proced.)

It should at least been documented by `proced-mode'. Something like:

Type \\[proced-toggle-auto-update] to update automatically the process
list.  The time interval for updates can be configured via
`proced-auto-update-interval'.

-- 
Daimrod/Greg





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#14341: adding an auto toggle update keybinding to proced
  2013-05-08 10:57   ` Daimrod
@ 2013-05-08 12:08     ` Nic Ferrier
  2013-05-15  3:34     ` Roland Winkler
  1 sibling, 0 replies; 6+ messages in thread
From: Nic Ferrier @ 2013-05-08 12:08 UTC (permalink / raw)
  To: Daimrod; +Cc: 14341, winkler

Daimrod <daimrod@gmail.com> writes:

> Glenn Morris <rgm@gnu.org> writes:
>
>> Nic Ferrier wrote:
>>
>>> +     (define-key km "U" 'proced-toggle-auto-update)
>>
>> The U key is already in use.
>>
>> (I'm not sure if proced-toggle-auto-update needs a keybinding, but I
>> don't use proced.)
>
> It should at least been documented by `proced-mode'. Something like:
>
> Type \\[proced-toggle-auto-update] to update automatically the process
> list.  The time interval for updates can be configured via
> `proced-auto-update-interval'.

I'll probably send another that adds a new proced top level:

proced-top

that automatically turns on the auto-update.

As soon as I can get ridiculous bzr working over my slow Internet link.


Nic





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#14341: adding an auto toggle update keybinding to proced
  2013-05-08 10:57   ` Daimrod
  2013-05-08 12:08     ` Nic Ferrier
@ 2013-05-15  3:34     ` Roland Winkler
  2016-02-24  5:06       ` Lars Ingebrigtsen
  1 sibling, 1 reply; 6+ messages in thread
From: Roland Winkler @ 2013-05-15  3:34 UTC (permalink / raw)
  To: Daimrod; +Cc: 14341, Nic Ferrier

On Wed May 8 2013 Daimrod wrote:
> Glenn Morris <rgm@gnu.org> writes:
> > (I'm not sure if proced-toggle-auto-update needs a keybinding, but I
> > don't use proced.)

I agree that proced-toggle-auto-update is not the kind of command
that justifies a keybinding.

> It should at least been documented by `proced-mode'. Something like:
> 
> Type \\[proced-toggle-auto-update] to update automatically the process
> list.  The time interval for updates can be configured via
> `proced-auto-update-interval'.

Thank you, makes sense. I'll do that.

> I'll probably send another that adds a new proced top level:
> 
> proced-top
> 
> that automatically turns on the auto-update.

If you like the auto-update, you can also set the default of the
buffer-local variable proced-auto-update-flag to t.

So I guess that a separate command proced-top would be needed only
if one starts several proced sessions, some with auto-update and
others without it. Does this happen? Normally, I have exactly one
proced session running for as long as my emacs session. And this one
session gets updated as needed.





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#14341: adding an auto toggle update keybinding to proced
  2013-05-15  3:34     ` Roland Winkler
@ 2016-02-24  5:06       ` Lars Ingebrigtsen
  0 siblings, 0 replies; 6+ messages in thread
From: Lars Ingebrigtsen @ 2016-02-24  5:06 UTC (permalink / raw)
  To: Roland Winkler; +Cc: Daimrod, 14341, Nic Ferrier

"Roland Winkler" <winkler@gnu.org> writes:

>> It should at least been documented by `proced-mode'. Something like:
>> 
>> Type \\[proced-toggle-auto-update] to update automatically the process
>> list.  The time interval for updates can be configured via
>> `proced-auto-update-interval'.
>
> Thank you, makes sense. I'll do that.

This apparently wasn't done, so I've now added that to the doc string in
the Emacs trunk, and I'm closing the bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-02-24  5:06 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-05-03  9:00 bug#14341: adding an auto toggle update keybinding to proced Nic Ferrier
2013-05-08  7:20 ` Glenn Morris
2013-05-08 10:57   ` Daimrod
2013-05-08 12:08     ` Nic Ferrier
2013-05-15  3:34     ` Roland Winkler
2016-02-24  5:06       ` Lars Ingebrigtsen

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).