describe-repeat-maps: Possible print bug:

describe-repeat-maps: Possible print bug:

[T.V Raman]

I use C-; as my Hyper-key and have commands bound to it used as a prefix-key.

I also use repeat-mode and  have a repeat property on one of those commands.

As an example:
emacspeak-media-relaxation is on C-; 1, C-x @ h 1
However, M-x describe-repeat-maps does not show that key in the help
buffer, it appears to render it as a  space.

So the corresponding line  for that afore-mentioned command shows up
like so in the Help Buffer:

‘emacspeak-media-relaxation’ (bound to )
-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1  🦮

-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1  🦮

Re: describe-repeat-maps: Possible print bug:

[Juri Linkov]

> I use C-; as my Hyper-key and have commands bound to it used as a prefix-key.
>
> I also use repeat-mode and  have a repeat property on one of those commands.
>
> As an example:
> emacspeak-media-relaxation is on C-; 1, C-x @ h 1
> However, M-x describe-repeat-maps does not show that key in the help
> buffer, it appears to render it as a  space.
>
> So the corresponding line  for that afore-mentioned command shows up
> like so in the Help Buffer:
>
> ‘emacspeak-media-relaxation’ (bound to )

I can't reproduce this problem with the following test case:

  (repeat-mode 1)
  (keymap-set global-map "C-; 1" 'next-line)
  (defvar-keymap test-repeat-map
    "1" 'next-line
    "C-; 1" 'next-line
    "C-x @ h 1" 'next-line)
  (put 'next-line 'repeat-map 'test-repeat-map)

Then 'M-x describe-repeat-maps' shows such output:

  ‘test-repeat-map’ keymap is repeatable by these commands:
   ‘next-line’ (bound to 1, C-; 1, C-x @ h 1)

Re: describe-repeat-maps: Possible print bug:

[T.V Raman]

Interesting, evaluating your test case shows the expected and desired
output C-; 1 is shown  as the key.

But sadly it doesn't work for my code.

My media player module is here: (apologies in advance -- it's a large
module, but the repeat-mode code is only a few lines.

https://github.com/tvraman/emacspeak/blob/master/lisp/emacspeak-m-player.el#L1442

Juri Linkov writes:
 > > I use C-; as my Hyper-key and have commands bound to it used as a prefix-key.
 > >
 > > I also use repeat-mode and  have a repeat property on one of those commands.
 > >
 > > As an example:
 > > emacspeak-media-relaxation is on C-; 1, C-x @ h 1
 > > However, M-x describe-repeat-maps does not show that key in the help
 > > buffer, it appears to render it as a  space.
 > >
 > > So the corresponding line  for that afore-mentioned command shows up
 > > like so in the Help Buffer:
 > >
 > > ‘emacspeak-media-relaxation’ (bound to )
 > 
 > I can't reproduce this problem with the following test case:
 > 
 >   (repeat-mode 1)
 >   (keymap-set global-map "C-; 1" 'next-line)
 >   (defvar-keymap test-repeat-map
 >     "1" 'next-line
 >     "C-; 1" 'next-line
 >     "C-x @ h 1" 'next-line)
 >   (put 'next-line 'repeat-map 'test-repeat-map)
 > 
 > Then 'M-x describe-repeat-maps' shows such output:
 > 
 >   ‘test-repeat-map’ keymap is repeatable by these commands:
 >    ‘next-line’ (bound to 1, C-; 1, C-x @ h 1)

-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1  🦮

--

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1  🦮

Re: describe-repeat-maps: Possible print bug:

[Juri Linkov]

> Interesting, evaluating your test case shows the expected and desired
> output C-; 1 is shown  as the key.
>
> But sadly it doesn't work for my code.
>
> My media player module is here: (apologies in advance -- it's a large
> module, but the repeat-mode code is only a few lines.
>
> https://github.com/tvraman/emacspeak/blob/master/lisp/emacspeak-m-player.el#L1442

Thanks, this reference helped to understand that
emacspeak-m-player-bind-accelerator binds only global keys,
not keys in the repeat map.  Here is a short test case
that confirms this:

  (repeat-mode 1)
  (keymap-set global-map "C-; 1" (defun test-repeat-next-line () (interactive) (next-line)))
  (defvar-keymap test-repeat-map)
  (put 'test-repeat-next-line 'repeat-map 'test-repeat-map)

And indeed 'describe-repeat-maps' outputs empty "(bound to )".
This is because the keys in the global map used to enter the
repeatable sequence currently are not handled by 'describe-repeat-maps'.
This should be fixed before the next release.

Also it was suggested to allow specifying the keys that enter
the repeatable sequence and exit it, by an additional keyword
in defvar-keymap, or by adding a new macro defvar-repeat-keymap.
This could be implemented as well.

Re: describe-repeat-maps: Possible print bug:

[T.V Raman]

Thanks for looking at my code and chasing it down, much apreciated.

Juri Linkov writes:
 > > Interesting, evaluating your test case shows the expected and desired
 > > output C-; 1 is shown  as the key.
 > >
 > > But sadly it doesn't work for my code.
 > >
 > > My media player module is here: (apologies in advance -- it's a large
 > > module, but the repeat-mode code is only a few lines.
 > >
 > > https://github.com/tvraman/emacspeak/blob/master/lisp/emacspeak-m-player.el#L1442
 > 
 > Thanks, this reference helped to understand that
 > emacspeak-m-player-bind-accelerator binds only global keys,
 > not keys in the repeat map.  Here is a short test case
 > that confirms this:
 > 
 >   (repeat-mode 1)
 >   (keymap-set global-map "C-; 1" (defun test-repeat-next-line () (interactive) (next-line)))
 >   (defvar-keymap test-repeat-map)
 >   (put 'test-repeat-next-line 'repeat-map 'test-repeat-map)
 > 
 > And indeed 'describe-repeat-maps' outputs empty "(bound to )".
 > This is because the keys in the global map used to enter the
 > repeatable sequence currently are not handled by 'describe-repeat-maps'.
 > This should be fixed before the next release.
 > 
 > Also it was suggested to allow specifying the keys that enter
 > the repeatable sequence and exit it, by an additional keyword
 > in defvar-keymap, or by adding a new macro defvar-repeat-keymap.
 > This could be implemented as well.

-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1  🦮

--

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1  🦮

Re: describe-repeat-maps: Possible print bug:

[Juri Linkov]

[-- Attachment #1: Type: text/plain, Size: 1373 bytes --]

> Thanks, this reference helped to understand that
> emacspeak-m-player-bind-accelerator binds only global keys,
> not keys in the repeat map.  Here is a short test case
> that confirms this:
>
>   (repeat-mode 1)
>   (keymap-set global-map "C-; 1" (defun test-repeat-next-line () (interactive) (next-line)))
>   (defvar-keymap test-repeat-map)
>   (put 'test-repeat-next-line 'repeat-map 'test-repeat-map)
>
> And indeed 'describe-repeat-maps' outputs empty "(bound to )".
> This is because the keys in the global map used to enter the
> repeatable sequence currently are not handled by 'describe-repeat-maps'.
> This should be fixed before the next release.

Here is a patch that uses outlines, and outputs the commands
that enter and exit the repeat-map.  For example,

  * ‘buffer-navigation-repeat-map’

  Entered with: ‘test-enter-next-line’
  Exited with: ‘test-exit-next-line’

  Key             Binding
  <left>          previous-buffer
  <right>         next-buffer

These two lines from the patch show the difference between the
commands that enter and exit the repeat-map:

    (setq commands-enter (seq-difference repeat-commands map-commands))
    (setq commands-exit  (seq-difference map-commands repeat-commands))

where 'repeat-commands' are commands with the symbol property 'repeat-map',
and 'map-commands' are commands existing in the map.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: describe-repeat-maps.patch --]
[-- Type: text/x-diff, Size: 2806 bytes --]

diff --git a/lisp/repeat.el b/lisp/repeat.el
index 0ae68d6024d..5f19479a1e8 100644
--- a/lisp/repeat.el
+++ b/lisp/repeat.el
@@ -588,21 +588,32 @@ describe-repeat-maps
                                           (when (and (symbolp (car a))
                                                      (symbolp (car b)))
                                             (string-lessp (car a) (car b))))))
-            (insert (format-message
-                     "`%s' keymap is repeatable by these commands:\n"
-                     (car keymap)))
-            (dolist (command (sort (cdr keymap) #'string-lessp))
-              (let* ((info (help-fns--analyze-function command))
-                     (map (list (if (symbolp (car keymap))
-                                    (symbol-value (car keymap))
-                                  (car keymap))))
-                     (desc (mapconcat (lambda (key)
-                                        (propertize (key-description key)
-                                                    'face 'help-key-binding))
-                                      (or (where-is-internal command map)
-                                          (where-is-internal (nth 3 info) map))
-                                      ", ")))
-                (insert (format-message " `%s' (bound to %s)\n" command desc))))
+            (insert (format-message "* `%s'\n" (car keymap)))
+
+            (let* ((map (if (symbolp (car keymap))
+                            (symbol-value (car keymap))
+                          (car keymap)))
+                   (repeat-commands (cdr keymap))
+                   map-commands commands-enter commands-exit)
+              (map-keymap (lambda (_key cmd) (when (symbolp cmd) (push cmd map-commands))) map)
+              (setq map-commands (seq-uniq map-commands))
+              (setq commands-enter (seq-difference repeat-commands map-commands))
+              (setq commands-exit  (seq-difference map-commands repeat-commands))
+
+              (when (or commands-enter commands-exit) (insert "\n"))
+              (when commands-enter
+                (insert (concat "Entered with: "
+                                (mapconcat (lambda (cmd) (format-message "`%s'" cmd))
+                                           commands-enter ", ")
+                                "\n")))
+              (when commands-exit
+                (insert (concat "Exited with: "
+                                (mapconcat (lambda (cmd) (format-message "`%s'" cmd))
+                                           commands-exit ", ")
+                                "\n"))))
+
+            (when (symbolp (car keymap))
+              (insert (substitute-command-keys (format-message "\\{%s}" (car keymap)))))
             (insert "\n")))))))
 
 (provide 'repeat)

[-- Attachment #3: Type: text/plain, Size: 280 bytes --]


> Also it was suggested to allow specifying the keys that enter
> the repeatable sequence and exit it, by an additional keyword
> in defvar-keymap, or by adding a new macro defvar-repeat-keymap.
> This could be implemented as well.

This is implemented now in a separate patch.

Re: describe-repeat-maps: Possible print bug:

[T.V Raman]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=gb18030, Size: 8976 bytes --]

Juri Linkov <juri@linkov.net> writes:

I'm still seeing some bindings displayed incorrectly:

Source Code:
https://github.com/tvraman/emacspeak/blob/master/lisp/emacspeak-m-player.el#L320
Example of problem:

In the below extract from the help buffer for describe-repeat-maps, note
that some keys are not printed, e.g.M-;
I'll annotate the problem lines with "#<key>" where <key> is what should
have been displayed.


¡®emacspeak-m-player-mode-map¡¯ keymap is repeatable by these commands:
 ¡®emacspeak-eww-play-media-at-point¡¯ (bound to ) #<M-;>
 ¡®emacspeak-m-player-add-autopan¡¯ (bound to a)
 ¡®emacspeak-m-player-add-autosat¡¯ (bound to z)
 ¡®emacspeak-m-player-add-equalizer¡¯ (bound to E)
 ¡®emacspeak-m-player-add-filter¡¯ (bound to f)
 ¡®emacspeak-m-player-amark-add¡¯ (bound to A)
 ¡®emacspeak-m-player-amark-jump¡¯ (bound to j)
 ¡®emacspeak-m-player-amark-save¡¯ (bound to S)
 ¡®emacspeak-m-player-apply-reverb-preset¡¯ (bound to P)
 ¡®emacspeak-m-player-backward-10min¡¯ (bound to <prior>)
 ¡®emacspeak-m-player-backward-10s¡¯ (bound to <left>, ,)
 ¡®emacspeak-m-player-backward-1min¡¯ (bound to <up>, <)
 ¡®emacspeak-m-player-beginning-of-track¡¯ (bound to <home>)
 ¡®emacspeak-m-player-clear-filters¡¯ (bound to C)
 ¡®emacspeak-m-player-customize-options¡¯ (bound to o)
 ¡®emacspeak-m-player-delete-filter¡¯ (bound to d)
 ¡®emacspeak-m-player-display-metadata¡¯ (bound to M)
 ¡®emacspeak-m-player-display-percent¡¯ (bound to %)
 ¡®emacspeak-m-player-display-position¡¯ (bound to ?)
 ¡®emacspeak-m-player-double-speed¡¯ (bound to })
 ¡®emacspeak-m-player-edit-reverb¡¯ (bound to R)
 ¡®emacspeak-m-player-end-of-track¡¯ (bound to <end>)
 ¡®emacspeak-m-player-equalizer-preset¡¯ (bound to e)
 ¡®emacspeak-m-player-faster¡¯ (bound to ])
 ¡®emacspeak-m-player-forward-10min¡¯ (bound to <next>)
 ¡®emacspeak-m-player-forward-10s¡¯ (bound to <right>, .)
 ¡®emacspeak-m-player-forward-1min¡¯ (bound to >, <down>)
 ¡®emacspeak-m-player-from-history¡¯ (bound to h)
 ¡®emacspeak-m-player-get-length¡¯ (bound to L)
 ¡®emacspeak-m-player-half-speed¡¯ (bound to {)
 ¡®emacspeak-m-player-left-channel¡¯ (bound to ()
 ¡®emacspeak-m-player-load¡¯ (bound to RET)
 ¡®emacspeak-m-player-loop¡¯ (bound to )#<C-e x ;>
 ¡®emacspeak-m-player-mode-line¡¯ (bound to m)
 ¡®emacspeak-m-player-next-track¡¯ (bound to n)
 ¡®emacspeak-m-player-pan¡¯ (bound to x)
 ¡®emacspeak-m-player-pause¡¯ (bound to SPC)
 ¡®emacspeak-m-player-persist-process¡¯ (bound to \)
 ¡®emacspeak-m-player-play-tracks-jump¡¯ (bound to t)
 ¡®emacspeak-m-player-pop-to-player¡¯ (bound to ;)
 ¡®emacspeak-m-player-previous-track¡¯ (bound to p)
 ¡®emacspeak-m-player-reset-options¡¯ (bound to O)
 ¡®emacspeak-m-player-reset-speed¡¯ (bound to DEL)
 ¡®emacspeak-m-player-restore-process¡¯ (bound to /)
 ¡®emacspeak-m-player-right-channel¡¯ (bound to ))
 ¡®emacspeak-m-player-run-jump¡¯ (bound to J)
 ¡®emacspeak-m-player-scale-speed¡¯ (bound to s)
 ¡®emacspeak-m-player-seek-absolute¡¯ (bound to g)
 ¡®emacspeak-m-player-seek-percentage¡¯ (bound to G)
 ¡®emacspeak-m-player-seek-relative¡¯ (bound to r)
 ¡®emacspeak-m-player-shuffle¡¯ (bound to ) #<:>
 ¡®emacspeak-m-player-slave-command¡¯ (bound to c)
 ¡®emacspeak-m-player-slower¡¯ (bound to [)
 ¡®emacspeak-m-player-store-link¡¯ (bound to l)
 ¡®emacspeak-m-player-stream-info¡¯ (bound to i)
 ¡®emacspeak-m-player-url¡¯ (bound to u)
 ¡®emacspeak-m-player-using-openal¡¯ (bound to ) #<:>
 ¡®emacspeak-m-player-volume-change¡¯ (bound to v)
 ¡®emacspeak-m-player-volume-down¡¯ (bound to -)
 ¡®emacspeak-m-player-volume-set¡¯ (bound to 9, 8, 7, 6, 5, 4, 3, 2, 1)
 ¡®emacspeak-m-player-volume-up¡¯ (bound to =, +)
 ¡®emacspeak-m-player-write-clip¡¯ (bound to w)
 ¡®emacspeak-m-player-youtube-live¡¯ (bound to ) #<C-e xl>
 ¡®emacspeak-media-38-dc-books¡¯ (bound to )
 ¡®emacspeak-media-brain-sync¡¯ (bound to ) # <C-; 7>
 ¡®emacspeak-media-classical¡¯ (bound to ) #<C-; 4>
 ¡®emacspeak-media-devotional¡¯ (bound to )
 ¡®emacspeak-media-indian¡¯ (bound to )
 ¡®emacspeak-media-mp3¡¯ (bound to )
 ¡®emacspeak-media-pop¡¯ (bound to )
 ¡®emacspeak-multimedia¡¯ (bound to )
>> Thanks, this reference helped to understand that
>> emacspeak-m-player-bind-accelerator binds only global keys,
>> not keys in the repeat map.  Here is a short test case
>> that confirms this:
>>
>>   (repeat-mode 1)
>>   (keymap-set global-map "C-; 1" (defun test-repeat-next-line () (interactive) (next-line)))
>>   (defvar-keymap test-repeat-map)
>>   (put 'test-repeat-next-line 'repeat-map 'test-repeat-map)
>>
>> And indeed 'describe-repeat-maps' outputs empty "(bound to )".
>> This is because the keys in the global map used to enter the
>> repeatable sequence currently are not handled by 'describe-repeat-maps'.
>> This should be fixed before the next release.
>
> Here is a patch that uses outlines, and outputs the commands
> that enter and exit the repeat-map.  For example,
>
>   * ¡®buffer-navigation-repeat-map¡¯
>
>   Entered with: ¡®test-enter-next-line¡¯
>   Exited with: ¡®test-exit-next-line¡¯
>
>   Key             Binding
>   <left>          previous-buffer
>   <right>         next-buffer
>
> These two lines from the patch show the difference between the
> commands that enter and exit the repeat-map:
>
>     (setq commands-enter (seq-difference repeat-commands map-commands))
>     (setq commands-exit  (seq-difference map-commands repeat-commands))
>
> where 'repeat-commands' are commands with the symbol property 'repeat-map',
> and 'map-commands' are commands existing in the map.
>
> diff --git a/lisp/repeat.el b/lisp/repeat.el
> index 0ae68d6024d..5f19479a1e8 100644
> --- a/lisp/repeat.el
> +++ b/lisp/repeat.el
> @@ -588,21 +588,32 @@ describe-repeat-maps
>                                            (when (and (symbolp (car a))
>                                                       (symbolp (car b)))
>                                              (string-lessp (car a) (car b))))))
> -            (insert (format-message
> -                     "`%s' keymap is repeatable by these commands:\n"
> -                     (car keymap)))
> -            (dolist (command (sort (cdr keymap) #'string-lessp))
> -              (let* ((info (help-fns--analyze-function command))
> -                     (map (list (if (symbolp (car keymap))
> -                                    (symbol-value (car keymap))
> -                                  (car keymap))))
> -                     (desc (mapconcat (lambda (key)
> -                                        (propertize (key-description key)
> -                                                    'face 'help-key-binding))
> -                                      (or (where-is-internal command map)
> -                                          (where-is-internal (nth 3 info) map))
> -                                      ", ")))
> -                (insert (format-message " `%s' (bound to %s)\n" command desc))))
> +            (insert (format-message "* `%s'\n" (car keymap)))
> +
> +            (let* ((map (if (symbolp (car keymap))
> +                            (symbol-value (car keymap))
> +                          (car keymap)))
> +                   (repeat-commands (cdr keymap))
> +                   map-commands commands-enter commands-exit)
> +              (map-keymap (lambda (_key cmd) (when (symbolp cmd) (push cmd map-commands))) map)
> +              (setq map-commands (seq-uniq map-commands))
> +              (setq commands-enter (seq-difference repeat-commands map-commands))
> +              (setq commands-exit  (seq-difference map-commands repeat-commands))
> +
> +              (when (or commands-enter commands-exit) (insert "\n"))
> +              (when commands-enter
> +                (insert (concat "Entered with: "
> +                                (mapconcat (lambda (cmd) (format-message "`%s'" cmd))
> +                                           commands-enter ", ")
> +                                "\n")))
> +              (when commands-exit
> +                (insert (concat "Exited with: "
> +                                (mapconcat (lambda (cmd) (format-message "`%s'" cmd))
> +                                           commands-exit ", ")
> +                                "\n"))))
> +
> +            (when (symbolp (car keymap))
> +              (insert (substitute-command-keys (format-message "\\{%s}" (car keymap)))))
>              (insert "\n")))))))
>  
>  (provide 'repeat)
>
>
>> Also it was suggested to allow specifying the keys that enter
>> the repeatable sequence and exit it, by an additional keyword
>> in defvar-keymap, or by adding a new macro defvar-repeat-keymap.
>> This could be implemented as well.
>
> This is implemented now in a separate patch.
>

-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
7©4 Id: kg:/m/0285kf1  •0Ü8

Re: describe-repeat-maps: Possible print bug:

[Juri Linkov]

> I'm still seeing some bindings displayed incorrectly:

Now the fix is pushed in the commit d6c1c76ba4.
Please try again.

Re: describe-repeat-maps: Possible print bug:

[T.V Raman]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=gb18030, Size: 180 bytes --]

works well,  dont see any more issues with displaying of keys.

Thanks,

-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
7©4 Id: kg:/m/0285kf1  •0Ü8

Re: describe-repeat-maps: Possible print bug:

[T.V Raman]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=gb18030, Size: 679 bytes --]


One more nit:

If you have a repeat command with many entry points, the help buffer
displays them all on one line-- see pathological example below:

Entered with: ¡®emacspeak-media-mp3¡¯, ¡®emacspeak-media-38-dc-books¡¯, ¡®emacspeak-m-player-loop¡¯, ¡®emacspeak-media-brain-sync¡¯, ¡®emacspeak-multimedia¡¯, ¡®emacspeak-m-player-youtube-live¡¯, ¡®emacspeak-media-devotional¡¯, ¡®emacspeak-media-classical¡¯, ¡®emacspeak-m-player-shuffle¡¯, ¡®emacspeak-eww-play-media-at-point¡¯, ¡®emacspeak-media-pop¡¯, ¡®emacspeak-media-indian¡¯, ¡®emacspeak-m-player-using-openal¡¯
-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
7©4 Id: kg:/m/0285kf1  •0Ü8

Re: describe-repeat-maps: Possible print bug:

[Juri Linkov]

> One more nit:
>
> If you have a repeat command with many entry points, the help buffer
> displays them all on one line-- see pathological example below:
>
> Entered with: ‘emacspeak-media-mp3’, ‘emacspeak-media-38-dc-books’,
> ‘emacspeak-m-player-loop’, ‘emacspeak-media-brain-sync’,
> ‘emacspeak-multimedia’, ‘emacspeak-m-player-youtube-live’,
> ‘emacspeak-media-devotional’, ‘emacspeak-media-classical’,
> ‘emacspeak-m-player-shuffle’, ‘emacspeak-eww-play-media-at-point’,
> ‘emacspeak-media-pop’, ‘emacspeak-media-indian’,
> ‘emacspeak-m-player-using-openal’

Thanks for noticing this, I haven't tested with so long lines.
Now fixed by filling lines as a paragraph.

Re: describe-repeat-maps: Possible print bug:

[T.V Raman]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=gb18030, Size: 398 bytes --]

One more small nit:

Perhaps start enumerating the commands on the line after  "Entered
with:" and "Exitted with:", so that the functions all appear together as
a group? 

You could then also make the "Entered With:" and "Exitted With:"
headlines to facilitate outline style navigation.
-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
7©4 Id: kg:/m/0285kf1  •0Ü8

Re: describe-repeat-maps: Possible print bug:

[Juri Linkov]

> One more small nit:
>
> Perhaps start enumerating the commands on the line after  "Entered
> with:" and "Exitted with:", so that the functions all appear together as
> a group?
>
> You could then also make the "Entered With:" and "Exitted With:"
> headlines to facilitate outline style navigation.

Maybe started with two asterisks as the outline second level, e.g.:

  * emacspeak-m-player-mode-map

  ** Entered with:

  ‘emacspeak-media-mp3’, ‘emacspeak-media-38-dc-books’,
  ‘emacspeak-m-player-loop’, ‘emacspeak-media-brain-sync’,
  ‘emacspeak-multimedia’, ‘emacspeak-m-player-youtube-live’,

Re: describe-repeat-maps: Possible print bug:

[T.V Raman]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=gb18030, Size: 336 bytes --]

One more thought: Since the help buffer now binds 'n' and 'p' to page
navigation, perhaps we could also use page delimiters to section out the
describe-repeat-maps buffer?

The outline headers are nice but 'n' is easier to hit 
-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
7©4 Id: kg:/m/0285kf1  •0Ü8

Re: describe-repeat-maps: Possible print bug:

[Juri Linkov]

> One more thought: Since the help buffer now binds 'n' and 'p' to page
> navigation, perhaps we could also use page delimiters to section out the
> describe-repeat-maps buffer?

Thanks for the nice suggestions, now added page separators
and more outlines.