bug#38222: describe-mode shows (fn...)

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

bug#38222: describe-mode shows (fn...)

Juanma Barranquero
Package: emacs
Severity: minor
Tags: patch

Currently, the output of `describe-mode' includes the (fn...) calling convention specifiers from the docstrings of major and minor modes.

Assuming that's not intended, here's a patch to fix it.

OK to install?


0001-Strip-fn.-from-output-of-describe-mode.patch (2K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

bug#38222: describe-mode shows (fn...)

Lars Ingebrigtsen
Juanma Barranquero <[hidden email]> writes:

> Currently, the output of `describe-mode' includes the (fn...) calling
> convention specifiers from the docstrings of major and minor modes.
>
> Assuming that's not intended, here's a patch to fix it.
>
> OK to install?

[...]

> +(defun help--doc-w/o-fn (mode)

We generally avoid using abbreviations in function names, I think.
Otherwise, makes sense to me.

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



Reply | Threaded
Open this post in threaded view
|

bug#38222: describe-mode shows (fn...)

Juanma Barranquero


On Sat, Nov 16, 2019 at 6:14 AM Lars Ingebrigtsen <[hidden email]> wrote:

> We generally avoid using abbreviations in function names, I think.

Yes, though w/o is used in two function names in uniquify.el.

I'll change it to help--doc-without-fn.

> Otherwise, makes sense to me.

Thanks.

Reply | Threaded
Open this post in threaded view
|

bug#38222: describe-mode shows (fn...)

Juanma Barranquero

Committed asĀ 49192e9510fe3c491b8c759a639bbe8bccf35856.