Subject: RE: [PATCH] `dired-next-line' go to meaningful line

From: Drew Adams <[email-masked]>
Date: Wed, 30 Aug 2023 14:54:39 +0000

Be sure to consider also the behavior when 
subdir listings are inserted in the buffer.
(Didn't try your patch; just sayin'.)
___

FWIW, I think that the right way to deal
with this is what's done in `dired+.el':

1. There's an option, `diredp-wrap-around-flag',
which controls wrapping around to the buffer
beginning/end when using `n'/`p'.

That is, it says whether moving down when
there's no file/dired/header line further down
wraps to the beginning (first such line in the
buffer), and similarly when moving up when
there's no such line above.

2. Commands `dired-(next|previous)-line' have
their keys remapped to commands that respect
the option (`diredp-(next|previous)-line').

Simple, useful, IMO.  (Likely suggested
before to `emacs-devel' but rejected.)
___

Commands `diredp-(next|previous)-line' also
have the improvement that they respect var
`goal-column': If non-nil then they put the
cursor at that column.