query-replace doesn't match spaces with soft newlines

query-replace doesn't match spaces with soft newlines

[David Reitter]

[-- Attachment #1: Type: text/plain, Size: 763 bytes --]

It appears that `query-replace' (or whatever this runs) does not  
properly match spaces in the search string with soft newlines. It  
should do that just like isearch does now.

I encountered this when I had `longlines-mode' turned on and tried to  
replace a string in the following line(s):

\item Adjoin: Combine the \slot{New Type} with the {\sc Context
Type}   according to one of the combinatorial rules.

Note that this is one paragraph, with a soft newline in between  
"Context" and "Type". To reproduce, enter it as one line and have  
longlines-mode turned on (perhaps adjusting fill-column) so that it  
breaks right there.

Then do M-% {\s Context Type} RET bar RET

The string will not be replaced.

This is with an up-to-date Emacs 22 CVS build.


[-- Attachment #2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 2193 bytes --]

Re: query-replace doesn't match spaces with soft newlines

[Juri Linkov]

> It appears that `query-replace' (or whatever this runs) does not properly
> match spaces in the search string with soft newlines. It  should do that
> just like isearch does now.
>
> I encountered this when I had `longlines-mode' turned on and tried to
> replace a string in the following line(s):
>
> \item Adjoin: Combine the \slot{New Type} with the {\sc Context
> Type}   according to one of the combinatorial rules.
>
> Note that this is one paragraph, with a soft newline in between "Context"
> and "Type". To reproduce, enter it as one line and have  longlines-mode
> turned on (perhaps adjusting fill-column) so that it  breaks right there.
>
> Then do M-% {\s Context Type} RET bar RET
>
> The string will not be replaced.

Perhaps I missed a discussion about the design of longlines-mode,
so I don't understand why longlines-mode doesn't put a `display'
property with a newline on whitespace to display it as soft newlines.
This would avoid many such problems.

If this is not possible, and replacing spaces with newlines is unavoidable,
then I suggest changing the plain search to the regexp search and
using `search-spaces-regexp' to convert spaces to newlines to be able
to match them in search and replace commands.

-- 
Juri Linkov
http://www.jurta.org/emacs/

Re: query-replace doesn't match spaces with soft newlines

[Miles Bader]

Juri Linkov <juri@jurta.org> writes:
> Perhaps I missed a discussion about the design of longlines-mode,
> so I don't understand why longlines-mode doesn't put a `display'
> property with a newline on whitespace to display it as soft newlines.
> This would avoid many such problems.

Because doing that makes many important emacs commands work incorrectly
(from the user's viewpoint) -- e.g., C-e will move to the end of the
paragraph...

-Miles

-- 
Marriage, n. The state or condition of a community consisting of a master, a
mistress and two slaves, making in all, two.

Re: query-replace doesn't match spaces with soft newlines

[Juri Linkov]

>> Perhaps I missed a discussion about the design of longlines-mode,
>> so I don't understand why longlines-mode doesn't put a `display'
>> property with a newline on whitespace to display it as soft newlines.
>> This would avoid many such problems.
>
> Because doing that makes many important emacs commands work incorrectly
> (from the user's viewpoint) -- e.g., C-e will move to the end of the
> paragraph...

Ok, I see now.  Many commands are expected to work on the visual
representation, so only a few commands that incorrectly treat inserted
newlines should be fixed.  I see no problem in matching additional
newlines in `query-replace' and the regexp search, but I don't see
how to distinguish soft newlines inserted by longlines-mode and hard
newlines with the `hard' property using the regexp search, i.e. how
(re-search-forward "[ \t\n]") could find soft newlines and skip
hard newlines?

-- 
Juri Linkov
http://www.jurta.org/emacs/