[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#28998: 27.0.50; Man--sections is in reverse order
From: |
Noam Postavsky |
Subject: |
bug#28998: 27.0.50; Man--sections is in reverse order |
Date: |
Mon, 06 Nov 2017 19:50:22 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.90 (gnu/linux) |
"Basil L. Contovounesios" <contovob@tcd.ie> writes:
> -(defsubst Man-build-section-alist ()
> +(defsubst Man-build-section-list ()
Perhaps we should switch to defun while we're at it?
> - (let ((case-fold-search nil))
> + (let (case-fold-search)
Personally, I like the more explicit version here.
> AFAICT there is no purpose or benefit to keeping
> Man--sections reversed, other than avoiding a call to
> nreverse, which AFAIK is quite an efficient operation.
>
> Conversely, keeping the sections in their natural order
> (i.e. from (point-min) to (point-max)) both seems more
> natural and is more convenient for users of completion
> frontends like ivy.el.
Makes sense to me.
- bug#28998: 27.0.50; Man--sections is in reverse order,
Noam Postavsky <=