bug#15148: 24.3.50; cache-long-scans t breaks dired

bug#15148: 24.3.50; cache-long-scans t breaks dired

[Michael Heerdegen]

Hello,

here, cache-long-scans t in dired does speed up moving by lines a lot,
but it also breaks dired.  Every time a file's line is updated, stuff
like

//DIRED// 45 51
//DIRED-OPTIONS// --quoting-style=literal

is inserted into the buffer.  File operations raise errors.

A minimal example from emacs -Q:

- dired a directory
- there, M-: (setq cache-long-scans t)
- on a file, hit l (maybe twice)

Then I get additional lines like above.


Thanks,

Michael.



In GNU Emacs 24.3.50.1 (x86_64-pc-linux-gnu, GTK+ Version 3.8.2)
 of 2013-08-11 on dex, modified by Debian
 (emacs-snapshot package, version 2:20130810-1)
Windowing system distributor `The X.Org Foundation', version 11.0.11204000
System Description:	Debian GNU/Linux testing (jessie)

Configured using:
 `configure --build x86_64-linux-gnu --host x86_64-linux-gnu
 --prefix=/usr --sharedstatedir=/var/lib --libexecdir=/usr/lib
 --localstatedir=/var --infodir=/usr/share/info --mandir=/usr/share/man
 --with-pop=yes
 --enable-locallisppath=/etc/emacs-snapshot:/etc/emacs:/usr/local/share/emacs/24.3.50/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/24.3.50/site-lisp:/usr/share/emacs/site-lisp
 --without-compress-info --with-crt-dir=/usr/lib/x86_64-linux-gnu/
 --with-x=yes --with-x-toolkit=gtk3 --with-imagemagick=yes
 CFLAGS='-DDEBIAN -DSITELOAD_PURESIZE_EXTRA=5000 -g -O2'
 CPPFLAGS='-D_FORTIFY_SOURCE=2' LDFLAGS='-g -Wl,--as-needed
 -znocombreloc''

bug#15148: 24.3.50; cache-long-scans t breaks dired

[Eli Zaretskii]

> From: Michael Heerdegen <michael_heerdegen@web.de>
> Date: Wed, 21 Aug 2013 00:59:21 +0200
> 
> here, cache-long-scans t in dired does speed up moving by lines a lot,
> but it also breaks dired.  Every time a file's line is updated, stuff
> like
> 
> //DIRED// 45 51
> //DIRED-OPTIONS// --quoting-style=literal
> 
> is inserted into the buffer.  File operations raise errors.
> 
> A minimal example from emacs -Q:
> 
> - dired a directory
> - there, M-: (setq cache-long-scans t)
> - on a file, hit l (maybe twice)
> 
> Then I get additional lines like above.

Thanks, this is now fixed (as of trunk revision 114981).

It was a tip of an iceberg, btw, and affected any command that invokes
call-process with a non-trivial value of coding-system-for-read
(i.e. when the text read from the sub-process needed to be decoded).