bug#81295: 32.0.50; dired-auto-toggle-b-switch problem with subdirectory

Eli Zaretskii <[email protected]>
Newsgroups gmane.emacs.bugs
Message-ID <[email protected]>
> From: Filipp Gunbin <[email protected]>
> Cc: [email protected],  [email protected],  [email protected]
> Date: Wed, 29 Jul 2026 17:49:31 +0300
> 
> On 29/07/2026 17:00 +0300, Eli Zaretskii wrote:
> 
> >> From: Filipp Gunbin <[email protected]>
> >> Cc: Eli Zaretskii <[email protected]>,  [email protected],
> >>   [email protected]
> >> Date: Wed, 29 Jul 2026 16:50:52 +0300
> >> 
> >> On 29/07/2026 15:32 +0200, Stephen Berman wrote:
> >> 
> >> [...]
> >> 
> >> > So that's the current situation, and why I've turned to you and Sean for
> >> > a decision on how to proceed: (i) install my changes to handle file
> >> > names with newlines in Dired subdirs, including the change that "fixes"
> >> > the bug with `C-u i'; (ii) install the changes except for the one that
> >> > "fixes" the `C-u i' bug, thus leaving that unfixed for now; (iii) wait
> >> > for Filipp's alternative implementation, which will hopefully fix that
> >> > bug without circumventing validation of subdir switches.
> >> 
> >> As I wrote already, I do not object to (ii), we agree on that patch.  I
> >> object only to the fix of supposedly rare problem which we found in the
> >> course of discussing this bug, and (i) additionally includes that fix.
> >> We could create another bug report for that and see if anyone (me, or
> >> you, or somebody else) comes up with the patch that we all agree with.
> >> That is, (iii) is not an alternative to (ii) - we could install the
> >> patch in (ii) right now, and that will fix the problem which I created
> >> this bug about.
> >
> > Why not install (i) and _then_ wait for that one aspect to be fixed
> > separately?
> 
> My concerns are that (i):
> 
> - Breaks the current logic, and thus will introduce more subtle bugs.
> Given that the problem it fixes is minor, I don't see this as a good
> trade-off.
> 
> - Makes the code harder to read and change later, as it adds a
> non-obvious special case, and you need to read this discussion to
> understand why it was added.  I always favor cleaner code, that's why
> I'd prefer to create another bug, rather than fix it here "at all
> costs", but that's me.

Fair enough, let's wait for Sean to chime in.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.