* bug#5410: Parenthesis Matching Bug!!
@ 2010-01-18 12:20 Matrix
2016-08-04 2:18 ` Andrew Hyatt
2022-05-02 9:02 ` bug#5410: bug#5411: show-paren-mode and parens spread over multi-line comments Lars Ingebrigtsen
0 siblings, 2 replies; 8+ messages in thread
From: Matrix @ 2010-01-18 12:20 UTC (permalink / raw
To: bug-gnu-emacs
Hello , developer! I am a student of china . I like emacs very much .
I use it to read txt-format book(because the color-theme of my emacs is
very good for eye!^^) . I also use emacs to coding!good~!!
But now I find a bug in emacs :( . May be it`s not a bug , I want to
believe this.Now The-Bug as follow:
// Print a message and dump core (equivalent to ASSERT(FALSE)
without
// making the compiler whine).
The Parenthesis-Matching-highlight is work with the second '(' .
yeah , When my cursor at the second '(' the first ')' highlight . But
when my cursor at first '(' , the second ')' doesn`t highlight and show
unmatch Parenthesis....I think this is wrong , It match as: ( ( ) ).
^^ , I like emacs , I want it to be more good tool for me and other
programer!!
Thanks developers!!!
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#5410: Parenthesis Matching Bug!!
2010-01-18 12:20 bug#5410: Parenthesis Matching Bug!! Matrix
@ 2016-08-04 2:18 ` Andrew Hyatt
2016-08-04 2:55 ` Michael Heerdegen
2022-05-02 9:02 ` bug#5410: bug#5411: show-paren-mode and parens spread over multi-line comments Lars Ingebrigtsen
1 sibling, 1 reply; 8+ messages in thread
From: Andrew Hyatt @ 2016-08-04 2:18 UTC (permalink / raw
To: Matrix; +Cc: 5410
Matrix <min.matrix@gmail.com> writes:
> Hello , developer! I am a student of china . I like emacs very much .
> I use it to read txt-format book(because the color-theme of my emacs is
> very good for eye!^^) . I also use emacs to coding!good~!!
> But now I find a bug in emacs :( . May be it`s not a bug , I want to
> believe this.Now The-Bug as follow:
>
> // Print a message and dump core (equivalent to ASSERT(FALSE)
> without
> // making the compiler whine).
>
> The Parenthesis-Matching-highlight is work with the second '(' .
> yeah , When my cursor at the second '(' the first ')' highlight . But
> when my cursor at first '(' , the second ')' doesn`t highlight and show
> unmatch Parenthesis....I think this is wrong , It match as: ( ( ) ).
>
> ^^ , I like emacs , I want it to be more good tool for me and other
> programer!!
> Thanks developers!!!
Hi, sorry for the late reply.
Your example reproduces for me in Emacs 25. My guess is that emacs
doesn't really assume that comment lines should be treated as a
contiguous text block. Which is sometimes right, sometimes not, in my
experience. For example, you could have comments that are something
like:
// This function does three things:
// 1) Reverse the string
// 2) Uuencode the string
// 3) Rot-13 the string
Which certainly wouldn't make a lot of sense treated contiguously.
Does anyone have any opinions whether this is a bug or a wishlish item?
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#5410: Parenthesis Matching Bug!!
2016-08-04 2:18 ` Andrew Hyatt
@ 2016-08-04 2:55 ` Michael Heerdegen
2016-08-04 20:17 ` Andrew Hyatt
0 siblings, 1 reply; 8+ messages in thread
From: Michael Heerdegen @ 2016-08-04 2:55 UTC (permalink / raw
To: Andrew Hyatt; +Cc: Matrix, 5410
Andrew Hyatt <ahyatt@gmail.com> writes:
> Your example reproduces for me in Emacs 25. My guess is that emacs
> doesn't really assume that comment lines should be treated as a
> contiguous text block. Which is sometimes right, sometimes not, in my
> experience.
We are talking about `show-paren-mode', right? AFAICT this just calls
`scan-sexps' and looks if that errors or not, without doing syntactical
analysis itself.
`parse-sexp-ignore-comments' influences `scan-sexps's behavior in this
regard, and setting it to nil does change the behavior in this case for
me. I guess we could use this for a solution.
> For example, you could have comments that are something
> like:
>
> // This function does three things:
> // 1) Reverse the string
> // 2) Uuencode the string
> // 3) Rot-13 the string
>
> Which certainly wouldn't make a lot of sense treated contiguously.
Well, we would not loose anything in this case: parens are already shown
as mismatches now.
Michael.
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#5410: Parenthesis Matching Bug!!
2016-08-04 2:55 ` Michael Heerdegen
@ 2016-08-04 20:17 ` Andrew Hyatt
2016-08-04 21:47 ` Michael Heerdegen
0 siblings, 1 reply; 8+ messages in thread
From: Andrew Hyatt @ 2016-08-04 20:17 UTC (permalink / raw
To: Michael Heerdegen; +Cc: Matrix, 5410
[-- Attachment #1: Type: text/plain, Size: 1567 bytes --]
On Wed, Aug 3, 2016 at 10:55 PM Michael Heerdegen <michael_heerdegen@web.de>
wrote:
> Andrew Hyatt <ahyatt@gmail.com> writes:
>
> > Your example reproduces for me in Emacs 25. My guess is that emacs
> > doesn't really assume that comment lines should be treated as a
> > contiguous text block. Which is sometimes right, sometimes not, in my
> > experience.
>
> We are talking about `show-paren-mode', right? AFAICT this just calls
> `scan-sexps' and looks if that errors or not, without doing syntactical
> analysis itself.
>
> `parse-sexp-ignore-comments' influences `scan-sexps's behavior in this
> regard, and setting it to nil does change the behavior in this case for
> me. I guess we could use this for a solution.2
>
Yes, this works to resolve the issue for me too. Are you saying it should
default to nil always, or for cc-mode? It seems like setting this to nil
might cause other issues we wouldn't want; ignoring comments usually seems
like the right thing to do.
It'd be nice, I guess, if you could just turn it to nil when in a comment.
>
> > For example, you could have comments that are something
> > like:
> >
> > // This function does three things:
> > // 1) Reverse the string
> > // 2) Uuencode the string
> > // 3) Rot-13 the string
> >
> > Which certainly wouldn't make a lot of sense treated contiguously.
>
> Well, we would not loose anything in this case: parens are already shown
> as mismatches now.
>
>
True, I guess my point is that it'd be strange to consider this as 3
unmatched parens rather than 1 per line.
>
> Michael.
>
[-- Attachment #2: Type: text/html, Size: 2465 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#5410: Parenthesis Matching Bug!!
2016-08-04 20:17 ` Andrew Hyatt
@ 2016-08-04 21:47 ` Michael Heerdegen
2016-08-11 3:57 ` Andrew Hyatt
0 siblings, 1 reply; 8+ messages in thread
From: Michael Heerdegen @ 2016-08-04 21:47 UTC (permalink / raw
To: Andrew Hyatt; +Cc: Matrix, 5410
Andrew Hyatt <ahyatt@gmail.com> writes:
> It'd be nice, I guess, if you could just turn it to nil when in a
> comment.
Yes, I think we could bind `parse-sexp-ignore-comments' to nil around
the calls to `scan-sexps' in `show-paren--default' when point is inside
a comment. But I must admit that I don't understand the terse doc of
`parse-sexp-ignore-comments':
| Non-nil means `forward-sexp', etc., should treat comments as
| whitespace.
But what does nil mean, exactly? It seems that comments are then
treated as if they were indistinguishable from code. When I set
`parse-sexp-ignore-comments' to nil in emacs-lisp-mode, and have such a
file:
;; (
)
then show-paren-mode indicates the parens as matching, though one is
inside in a comment, and the other is not. `scan-sexps' behaves
accordingly. That would mean we would need to assure that the matching
paren position that `scan-sexps' has found is still inside the current
comment.
Michael.
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#5410: Parenthesis Matching Bug!!
2016-08-04 21:47 ` Michael Heerdegen
@ 2016-08-11 3:57 ` Andrew Hyatt
2016-08-23 15:02 ` Michael Heerdegen
0 siblings, 1 reply; 8+ messages in thread
From: Andrew Hyatt @ 2016-08-11 3:57 UTC (permalink / raw
To: Michael Heerdegen; +Cc: 5410, Matrix
Michael Heerdegen <michael_heerdegen@web.de> writes:
> Andrew Hyatt <ahyatt@gmail.com> writes:
>
>> It'd be nice, I guess, if you could just turn it to nil when in a
>> comment.
>
> Yes, I think we could bind `parse-sexp-ignore-comments' to nil around
> the calls to `scan-sexps' in `show-paren--default' when point is inside
> a comment. But I must admit that I don't understand the terse doc of
> `parse-sexp-ignore-comments':
>
> | Non-nil means `forward-sexp', etc., should treat comments as
> | whitespace.
>
> But what does nil mean, exactly? It seems that comments are then
> treated as if they were indistinguishable from code. When I set
> `parse-sexp-ignore-comments' to nil in emacs-lisp-mode, and have such a
> file:
>
> ;; (
> )
>
> then show-paren-mode indicates the parens as matching, though one is
> inside in a comment, and the other is not. `scan-sexps' behaves
> accordingly. That would mean we would need to assure that the matching
> paren position that `scan-sexps' has found is still inside the current
> comment.
Interesting point. The more I hear, the more this sounds like a
wishlist item - we just don't seem to have a way to treat paren matching
in comments separately yet with multiline capabilities. I'm going to
mark it as wishlist for now.
>
>
> Michael.
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#5410: Parenthesis Matching Bug!!
2016-08-11 3:57 ` Andrew Hyatt
@ 2016-08-23 15:02 ` Michael Heerdegen
0 siblings, 0 replies; 8+ messages in thread
From: Michael Heerdegen @ 2016-08-23 15:02 UTC (permalink / raw
To: Andrew Hyatt; +Cc: 5410, Matrix
Andrew Hyatt <ahyatt@gmail.com> writes:
> Interesting point. The more I hear, the more this sounds like a
> wishlist item - we just don't seem to have a way to treat paren
> matching in comments separately yet with multiline capabilities. I'm
> going to mark it as wishlist for now.
I agree - thanks.
FWIW I had filed
bug#24172: 25.1; Doc of parse-sexp-ignore-comments: what does a value
of nil mean?
When once it has been fixed, we'll know better how to fix this issue.
Michael.
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#5410: bug#5411: show-paren-mode and parens spread over multi-line comments
2010-01-18 12:20 bug#5410: Parenthesis Matching Bug!! Matrix
2016-08-04 2:18 ` Andrew Hyatt
@ 2022-05-02 9:02 ` Lars Ingebrigtsen
1 sibling, 0 replies; 8+ messages in thread
From: Lars Ingebrigtsen @ 2022-05-02 9:02 UTC (permalink / raw
To: Matrix; +Cc: 5410
Matrix <min.matrix@gmail.com> writes:
> But now I find a bug in emacs :( . May be it`s not a bug , I want to
> believe this.Now The-Bug as follow:
>
> // Print a message and dump core (equivalent to ASSERT(FALSE)
> without
> // making the compiler whine).
>
> The Parenthesis-Matching-highlight is work with the second '(' .
> yeah , When my cursor at the second '(' the first ')' highlight . But
> when my cursor at first '(' , the second ')' doesn`t highlight and show
> unmatch Parenthesis....I think this is wrong , It match as: ( ( ) ).
I've now fixed this in Emacs 29 (just 12 years after it was reported
😀).
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2022-05-02 9:02 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-18 12:20 bug#5410: Parenthesis Matching Bug!! Matrix
2016-08-04 2:18 ` Andrew Hyatt
2016-08-04 2:55 ` Michael Heerdegen
2016-08-04 20:17 ` Andrew Hyatt
2016-08-04 21:47 ` Michael Heerdegen
2016-08-11 3:57 ` Andrew Hyatt
2016-08-23 15:02 ` Michael Heerdegen
2022-05-02 9:02 ` bug#5410: bug#5411: show-paren-mode and parens spread over multi-line comments Lars Ingebrigtsen
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.