[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#27465: 24.5; `dired-mark-unmarked-files' with non-nil UNFLAG-P

From: Drew Adams
Subject: bug#27465: 24.5; `dired-mark-unmarked-files' with non-nil UNFLAG-P
Date: Fri, 23 Jun 2017 15:44:21 -0700 (PDT)

AFAICT this could never have worked.

1. Prior to Emacs 24, the interactive spec was just "P", which could of
   course not work interactively.  In Emacs 24 it was changed to read
   the REGEXP and use the prefix arg for UNFLAG-P.  That "P" was an
   indication that the command never worked interactively and was
   probably ill-conceived for that.  And that "P" remained from Emacs
   20 (if not before) through 23.

2. There are no occurrences in the Elisp sources that use non-nil

3. Non-nil UNFLAG-P does nothing, AFAICT.  It _cannot_ do anything.
   Certainly it cannot unflag, which means change a deletion flag (`D')
   to a space char.  Why?  Because the replacement is applied only to
   lines that already have a space char (not a `D').

   Its effect in that case is indeed to post a space char.  But it does
   so only for lines that already have a space char!

   This is the call:

      (looking-at-p " ") ; <=== Mark (with a SPC) only if already there!
      (let ((fn (dired-get-filename localp t)))
        (and fn (string-match-p regexp fn))))

What to do?  I suggest NOT trying to have any "unflag" behavior here.
If it were fixed, so that it actually replaced `D' with ` ' when there
is a prefix arg, I think that behavior would be confusing.  IOW, better
not to fix the implementation to fix an awful design.  Mark-if-unmarked
should not mean unflag sometimes.

But if you really want to fix this so that it does what the doc says,
that's OK by me.  Something should be done about it, anyway.

(BTW, the comment "uninteresting" is backward wrt the comment "not
already marked".  The files that are checked are those that are not
already marked and are interesting, i.e, match.)

In GNU Emacs 24.5.1 (i686-pc-mingw32)
 of 2015-04-11 on LEG570
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --prefix=/c/usr --host=i686-pc-mingw32'

reply via email to

[Prev in Thread] Current Thread [Next in Thread]