* bug#14787: 24.3, M-x query-replace - useless warning
@ 2013-07-04 8:22 Andreas Röhler
2013-07-04 8:36 ` Andreas Schwab
0 siblings, 1 reply; 14+ messages in thread
From: Andreas Röhler @ 2013-07-04 8:22 UTC (permalink / raw)
To: 14787
M-x query-replace RET ] RET
sends a warning
"No matching parenthesis found"
which seems useful only WRT to regexps.
See also
http://stackoverflow.com/questions/17464676/emacs-replace-right-square-bracket/17465200#17465200
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-04 8:22 bug#14787: 24.3, M-x query-replace - useless warning Andreas Röhler
@ 2013-07-04 8:36 ` Andreas Schwab
2013-07-04 13:18 ` Andreas Röhler
0 siblings, 1 reply; 14+ messages in thread
From: Andreas Schwab @ 2013-07-04 8:36 UTC (permalink / raw)
To: Andreas Röhler; +Cc: 14787
Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
> M-x query-replace RET ] RET
>
> sends a warning
>
> "No matching parenthesis found"
>
> which seems useful only WRT to regexps.
This has nothing to do with regexps. Show Paren mode is a global minor
mode that is enabled independent of the context.
Andreas.
--
Andreas Schwab, SUSE Labs, schwab@suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-04 8:36 ` Andreas Schwab
@ 2013-07-04 13:18 ` Andreas Röhler
2013-07-04 14:55 ` Tassilo Horn
0 siblings, 1 reply; 14+ messages in thread
From: Andreas Röhler @ 2013-07-04 13:18 UTC (permalink / raw)
To: Andreas Schwab; +Cc: 14787
Am 04.07.2013 10:36, schrieb Andreas Schwab:
> Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
>
>> M-x query-replace RET ] RET
>>
>> sends a warning
>>
>> "No matching parenthesis found"
>>
>> which seems useful only WRT to regexps.
>
> This has nothing to do with regexps. Show Paren mode is a global minor
> mode that is enabled independent of the context.
>
> Andreas.
>
It also happens if show-paren-mode is nil.
Don't see it related.
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-04 13:18 ` Andreas Röhler
@ 2013-07-04 14:55 ` Tassilo Horn
2013-07-04 19:21 ` Andreas Röhler
0 siblings, 1 reply; 14+ messages in thread
From: Tassilo Horn @ 2013-07-04 14:55 UTC (permalink / raw)
To: Andreas Röhler; +Cc: Andreas Schwab, 14787
Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
>>> M-x query-replace RET ] RET
>>>
>>> sends a warning
>>>
>>> "No matching parenthesis found"
>>>
>>> which seems useful only WRT to regexps.
>>
>> This has nothing to do with regexps. Show Paren mode is a global minor
>> mode that is enabled independent of the context.
>
> It also happens if show-paren-mode is nil.
> Don't see it related.
It actually depends on `blink-matching-paren', not `show-paren-mode'.
Bye,
Tassilo
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-04 14:55 ` Tassilo Horn
@ 2013-07-04 19:21 ` Andreas Röhler
2013-07-05 7:45 ` Tassilo Horn
0 siblings, 1 reply; 14+ messages in thread
From: Andreas Röhler @ 2013-07-04 19:21 UTC (permalink / raw)
To: Tassilo Horn; +Cc: Andreas Schwab, 14787
Am 04.07.2013 16:55, schrieb Tassilo Horn:
> Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
>
>>>> M-x query-replace RET ] RET
>>>>
>>>> sends a warning
>>>>
>>>> "No matching parenthesis found"
>>>>
>>>> which seems useful only WRT to regexps.
>>>
>>> This has nothing to do with regexps. Show Paren mode is a global minor
>>> mode that is enabled independent of the context.
>>
>> It also happens if show-paren-mode is nil.
>> Don't see it related.
>
> It actually depends on `blink-matching-paren', not `show-paren-mode'.
>
> Bye,
> Tassilo
>
Okay, thanks. IMO it's a bug nonetheless, it's a warning which might confuse users.
Cheers,
Andreas
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-04 19:21 ` Andreas Röhler
@ 2013-07-05 7:45 ` Tassilo Horn
2013-07-05 7:54 ` martin rudalics
2013-07-05 19:38 ` Andreas Röhler
0 siblings, 2 replies; 14+ messages in thread
From: Tassilo Horn @ 2013-07-05 7:45 UTC (permalink / raw)
To: Andreas Röhler; +Cc: Andreas Schwab, 14787
Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
>>>> This has nothing to do with regexps. Show Paren mode is a global
>>>> minor mode that is enabled independent of the context.
>>>
>>> It also happens if show-paren-mode is nil. Don't see it related.
>>
>> It actually depends on `blink-matching-paren', not `show-paren-mode'.
>
> Okay, thanks. IMO it's a bug nonetheless, it's a warning which might
> confuse users.
Not sure, IMHO "balanced" replacements like \(foo\|bar\) -> [\1] are
more common, and then `blink-matching-paren' makes sense. I mean, what
are the alternatives? We could disable `blink-matching-paren' when
reading from the minibuffer altogether, or have another defcustom
`blink-matching-paren-in-minibuffer' for controlling that behavior, and
then, what should its default value be?
Bye,
Tassilo
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-05 7:45 ` Tassilo Horn
@ 2013-07-05 7:54 ` martin rudalics
2013-07-05 19:38 ` Andreas Röhler
1 sibling, 0 replies; 14+ messages in thread
From: martin rudalics @ 2013-07-05 7:54 UTC (permalink / raw)
To: Tassilo Horn; +Cc: Andreas Schwab, 14787
> Not sure, IMHO "balanced" replacements like \(foo\|bar\) -> [\1] are
> more common, and then `blink-matching-paren' makes sense. I mean, what
> are the alternatives? We could disable `blink-matching-paren' when
> reading from the minibuffer altogether, or have another defcustom
> `blink-matching-paren-in-minibuffer' for controlling that behavior, and
> then, what should its default value be?
We could disable `blink-matching-paren' by default ;-)
martin
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-05 7:45 ` Tassilo Horn
2013-07-05 7:54 ` martin rudalics
@ 2013-07-05 19:38 ` Andreas Röhler
2013-07-05 22:10 ` Juri Linkov
2013-07-08 8:30 ` Tassilo Horn
1 sibling, 2 replies; 14+ messages in thread
From: Andreas Röhler @ 2013-07-05 19:38 UTC (permalink / raw)
To: Tassilo Horn; +Cc: Andreas Schwab, 14787
Am 05.07.2013 09:45, schrieb Tassilo Horn:
> Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
>
>>>>> This has nothing to do with regexps. Show Paren mode is a global
>>>>> minor mode that is enabled independent of the context.
>>>>
>>>> It also happens if show-paren-mode is nil. Don't see it related.
>>>
>>> It actually depends on `blink-matching-paren', not `show-paren-mode'.
>>
>> Okay, thanks. IMO it's a bug nonetheless, it's a warning which might
>> confuse users.
>
> Not sure, IMHO "balanced" replacements like \(foo\|bar\) -> [\1] are
> more common,
It's about query-replace, not query-replace-regexp
Cheers,
Andreas
and then `blink-matching-paren' makes sense. I mean, what
> are the alternatives? We could disable `blink-matching-paren' when
> reading from the minibuffer altogether, or have another defcustom
> `blink-matching-paren-in-minibuffer' for controlling that behavior, and
> then, what should its default value be?
>
> Bye,
> Tassilo
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-05 19:38 ` Andreas Röhler
@ 2013-07-05 22:10 ` Juri Linkov
2013-07-06 6:17 ` Andreas Röhler
2013-07-08 8:30 ` Tassilo Horn
1 sibling, 1 reply; 14+ messages in thread
From: Juri Linkov @ 2013-07-05 22:10 UTC (permalink / raw)
To: Andreas Röhler; +Cc: Andreas Schwab, 14787, Tassilo Horn
>>>> It actually depends on `blink-matching-paren', not `show-paren-mode'.
>>>
>>> Okay, thanks. IMO it's a bug nonetheless, it's a warning which might
>>> confuse users.
>>
>> Not sure, IMHO "balanced" replacements like \(foo\|bar\) -> [\1] are
>> more common,
>
> It's about query-replace, not query-replace-regexp
You can make balanced non-regexp replacements in query-replace like
(foo (bar)) -> (bar (foo))
The warning "No matching parenthesis found" helps the users to write
correctly balanced expressions everywhere, including minibuffers.
When you sometimes write an unbalanced expression intentionally,
you can ignore this warning.
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-05 22:10 ` Juri Linkov
@ 2013-07-06 6:17 ` Andreas Röhler
2013-07-06 7:20 ` Andreas Schwab
0 siblings, 1 reply; 14+ messages in thread
From: Andreas Röhler @ 2013-07-06 6:17 UTC (permalink / raw)
To: Juri Linkov; +Cc: Andreas Schwab, 14787, Tassilo Horn
Am 06.07.2013 00:10, schrieb Juri Linkov:
>>>>> It actually depends on `blink-matching-paren', not `show-paren-mode'.
>>>>
>>>> Okay, thanks. IMO it's a bug nonetheless, it's a warning which might
>>>> confuse users.
>>>
>>> Not sure, IMHO "balanced" replacements like \(foo\|bar\) -> [\1] are
>>> more common,
>>
>> It's about query-replace, not query-replace-regexp
>
> You can make balanced non-regexp replacements in query-replace like
> (foo (bar)) -> (bar (foo))
>
> The warning "No matching parenthesis found" helps the users to write
> correctly balanced expressions everywhere, including minibuffers.
> When you sometimes write an unbalanced expression intentionally,
> you can ignore this warning.
>
A help should not go in the way. When writing code, this help is useful.
When replacing single chars, not.
Andreas
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-06 6:17 ` Andreas Röhler
@ 2013-07-06 7:20 ` Andreas Schwab
2013-07-06 10:24 ` Andreas Röhler
0 siblings, 1 reply; 14+ messages in thread
From: Andreas Schwab @ 2013-07-06 7:20 UTC (permalink / raw)
To: Andreas Röhler; +Cc: 14787, Tassilo Horn
Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
> A help should not go in the way. When writing code, this help is useful.
> When replacing single chars, not.
How will the minibuffer know that you intent to input only a single
character?
Andreas.
--
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-06 7:20 ` Andreas Schwab
@ 2013-07-06 10:24 ` Andreas Röhler
0 siblings, 0 replies; 14+ messages in thread
From: Andreas Röhler @ 2013-07-06 10:24 UTC (permalink / raw)
To: Andreas Schwab; +Cc: 14787, Tassilo Horn
Am 06.07.2013 09:20, schrieb Andreas Schwab:
> Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
>
>> A help should not go in the way. When writing code, this help is useful.
>> When replacing single chars, not.
>
> How will the minibuffer know that you intent to input only a single
> character?
>
> Andreas.
>
So let's extend the single character to a string, which should make the issue more obvious.
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-05 19:38 ` Andreas Röhler
2013-07-05 22:10 ` Juri Linkov
@ 2013-07-08 8:30 ` Tassilo Horn
2013-07-08 14:20 ` Andreas Röhler
1 sibling, 1 reply; 14+ messages in thread
From: Tassilo Horn @ 2013-07-08 8:30 UTC (permalink / raw)
To: Andreas Röhler; +Cc: Andreas Schwab, 14787
Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
>> Not sure, IMHO "balanced" replacements like \(foo\|bar\) -> [\1] are
>> more common,
>
> It's about query-replace, not query-replace-regexp
You get that warning nearly everywhere something is read from the
minibuffer, e.g., also with `search-forward',
`execute-extended-command', `find-file', etc.
Well, at least for the latter two, `blink-matching-paren' doesn't really
make much sense, but then again, it's unlikely a user enters just a
closing paren when he wants to execute a command or find a file.
Bye,
Tassilo
^ permalink raw reply [flat|nested] 14+ messages in thread
* bug#14787: 24.3, M-x query-replace - useless warning
2013-07-08 8:30 ` Tassilo Horn
@ 2013-07-08 14:20 ` Andreas Röhler
0 siblings, 0 replies; 14+ messages in thread
From: Andreas Röhler @ 2013-07-08 14:20 UTC (permalink / raw)
To: Tassilo Horn; +Cc: Andreas Schwab, 14787
Am 08.07.2013 10:30, schrieb Tassilo Horn:
> Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
>
>>> Not sure, IMHO "balanced" replacements like \(foo\|bar\) -> [\1] are
>>> more common,
>>
>> It's about query-replace, not query-replace-regexp
>
> You get that warning nearly everywhere something is read from the
> minibuffer, e.g., also with `search-forward',
> `execute-extended-command', `find-file', etc.
>
> Well, at least for the latter two, `blink-matching-paren' doesn't really
> make much sense, but then again, it's unlikely a user enters just a
> closing paren when he wants to execute a command or find a file.
>
> Bye,
> Tassilo
>
Indeed, bug exists also WRT search-forward.
Cheers,
Andreas
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2013-07-08 14:20 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-04 8:22 bug#14787: 24.3, M-x query-replace - useless warning Andreas Röhler
2013-07-04 8:36 ` Andreas Schwab
2013-07-04 13:18 ` Andreas Röhler
2013-07-04 14:55 ` Tassilo Horn
2013-07-04 19:21 ` Andreas Röhler
2013-07-05 7:45 ` Tassilo Horn
2013-07-05 7:54 ` martin rudalics
2013-07-05 19:38 ` Andreas Röhler
2013-07-05 22:10 ` Juri Linkov
2013-07-06 6:17 ` Andreas Röhler
2013-07-06 7:20 ` Andreas Schwab
2013-07-06 10:24 ` Andreas Röhler
2013-07-08 8:30 ` Tassilo Horn
2013-07-08 14:20 ` Andreas Röhler
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).