all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Alan Mackenzie <acm@muc.de>
To: "Herman, Géza" <geza.herman@gmail.com>
Cc: emacs-devel@gnu.org
Subject: Re: Parenthesis matching should consider the kind of parenthesis during pair-search
Date: Mon, 12 Feb 2024 15:57:32 +0000	[thread overview]
Message-ID: <Zco_7Ahu1_HNSztV@ACM> (raw)
In-Reply-To: <87wmr9hh8e.fsf@gmail.com>

Hello, Géza.

On Mon, Feb 12, 2024 at 15:10:12 +0100, Herman, Géza wrote:
> >> The only gain with the current way is that Emacs will point out the
> >> mismatch (considering my Makefile example), if the cursor is on the
> >> opening '('.  With my suggested method, it's not the mismatch which
> >> would be shown, but the pair of '(' would (which is actually a
> >> correct behavior in this case, the pair of '(' is indeed the ')' ).

> > OK, I don't agree with some of the things you're saying

> I think I just wrote facts here.

Well, you wrote "All the other interactions would be the same.", which is
not yet an established fact.

> If the matching algorithm were modified in the way I described, that
> would be the result.  Which parts do you not agree with?

> > but why don't you implement your idea so that we can try it out?

> If it's an easy thing to do for someone who doesn't really know Emacs's
> codebase too much, I can do that.  But, tbh, it shouldn't be too hard
> to imagine how it worked.  Also, all the other editors I know works
> similarly like I described.  So it's easy to try this out with other
> editors.

I would say the exercise is moderately difficult for somebody who hasn't
worked with the Emacs codebase before.  But experience indicates that
none of the other readers of emacs-devel is likely to be enthused enough
by your suggestion actually to take it up and implement it.  If you want
to see it in Emacs, you're going to have to implement it yourself.  But
help would be available on the list.

> >> If you put the cursor on the '}', Emacs would still show that it is
> >> a paren without a pair, just like now (but instead of saying that
> >> its "pair" is the '(', it wouldn't show anything, because '}'
> >> doesn't have a pair.

> > What if earlier on in the file there were a

> >     ({)

> > ?  Should your new way of matching find that {?  Or would the
> > enclosing ( and ) prevent this being a match?  If so, what are the
> > exact criteria?

> It would find it.  My suggested algorithm is simple: when 
> searching for a pair of '{', the algorithm should only consider 
> '{' and '}'. All the other kind of parens should be ignored. 
> That's it.

OK, I understand.

> >> Why is it a desired thing that Emacs should show some mismatching
> >> paren if the cursor is on a completely unrelated paren?

> > Emacs doesn't have the facilities to determine that the mismatching
> > paren is "completely unrelated".

> Emacs already tells that '(' and '}' are unrelated, because it 
> shows them as mismatching.  I don't think that there should be any 
> sophisticated facilities to determine unrelatedness.  If the paren 
> Emacs find is a mismatching one, then it should continue finding 
> the proper pair, that's it.

OK.

> > It's fairly likely to be a typo.

> Yes, that is an example that I can accept.  But I don't think that this
> is a very common thing.  At least, for me, this Emacs behavior was
> never useful.

For editing files written in C or C++, all of (), {}, [], and (in C++)
<> are used as paren pairs.  They've got to match properly.  After
killing and yanking a large chunk of text, the current paren matching is
useful to indicate any "missing" parens at the start or end of the region
just yanked, for example.

> But it annoys me that sometimes I cannot move to the pair of a paren
> because of this behavior.

To be honest, things like that annoy me a bit, too, like when typing in a
regexp into the minibuffer.  But it would annoy me more not to have any
mismatches highlighted in a C buffer.

> But thinking about it further, maybe this problem shouldn't be solved
> like this, because there is a better solution: makefile-mode should be
> more clever - if it is possible - to not mark } as a paren in my
> example case, because from the semantics of the Makefile viewpoint, it
> is not actually a paren, '}' should behave like if it were in a string.

Yes, this would be possible too, but likely quite a bit more complicated.

-- 
Alan Mackenzie (Nuremberg, Germany).



  reply	other threads:[~2024-02-12 15:57 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-12 11:49 Parenthesis matching should consider the kind of parenthesis during pair-search Herman, Géza
2024-02-12 12:34 ` Joost Kremers
2024-02-12 12:58 ` Alan Mackenzie
2024-02-12 13:10   ` Herman, Géza
2024-02-12 14:07     ` Alan Mackenzie
2024-02-12 14:10       ` Herman, Géza
2024-02-12 15:57         ` Alan Mackenzie [this message]
2024-02-12 16:05           ` Herman, Géza

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Zco_7Ahu1_HNSztV@ACM \
    --to=acm@muc.de \
    --cc=emacs-devel@gnu.org \
    --cc=geza.herman@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.