From: "Kévin Le Gouguec" <kevin.legouguec@gmail.com>
To: "João Távora" <joaotavora@gmail.com>
Cc: 56135@debbugs.gnu.org, Stefan Monnier <monnier@iro.umontreal.ca>
Subject: bug#56135: 29.0.50; python.el - forward-sexp regression over triple-quoted strings
Date: Thu, 23 Jun 2022 08:49:31 +0200 [thread overview]
Message-ID: <87fsjv27t0.fsf@gmail.com> (raw)
In-Reply-To: <CALDnm53bj4rWzkP50L7TdwF3CRpy4rST3oy23bujWgC9YCJnAA@mail.gmail.com> ("João Távora"'s message of "Wed, 22 Jun 2022 23:19:40 +0100")
João Távora <joaotavora@gmail.com> writes:
> On Wed, Jun 22, 2022 at 10:48 PM Kévin Le Gouguec <kevin.legouguec@gmail.com> wrote:
>
> Kévin Le Gouguec <kevin.legouguec@gmail.com> writes:
>
> > João Távora <joaotavora@gmail.com> writes:
> >
> >> Why are you setting python-forward-sexp-function to nil?
> >
> > [wall of text follows]
>
> (
> Hadn't realized we had taken this off-list. FWIW if you found all
> my verbiage informative, don't hesitate to (tell me to) forward it
> back on the bug tracker.
> )
>
> Ups. That was my mistake. Wondering what to do now. Should we re-send to the
> tracker. Or let future generations wonder about the missing great wall of text.
😁 I'll forward it in a separate message, in case someone finds other
points to bounce off of.
> Anyway, I did find your wall of text informative, particularly this point.
>
>> Wondering what the best way forward would be. Maybe a third suggested
>> value for python-forward-sexp-function, which would handle triple-quotes
>> but not add those "implicit paired delimiters"?
>
> Yes, I think this makes sense. Let's see what other think, but a patch
> that extracts only that bit from the current functionality and puts it in
> a third value would be a good idea, I think.
Mm. I've stepped through python-nav--forward-sexp with Edebug, and
there wasn't any obvious "triple-quote handler" to extract. IIUC
* being on either end of a triple-quote block satisfies
python-info-{beginning,end}-of-statement-p, so 'context' becomes
'statement-{start,end},
* (then we dance a little gigue to compute next-sexp-{context,pos},
which we do not end up using in this scenario,)
* eventually we call python-nav-{end,beginning}-of-statement.
So the "magic" happens in those functions IIUC. E.g. in
python-nav-end-of-statement we either
* skip the whole cond form because the initial (goto-char
(line-end-position)) took us at EOL after a single-line string, or…
* go down the (setq string-start (python-syntax-context 'string)) path
for multiline strings, or…
* … oh! Mm, well, here's a bug maybe? Consider:
"""string 1"""
def foo():
"""string 2
(multiline).
"""
print("""string 3""")
^
So *with python-forward-sexp-function set to its default value*
(python-nav-forward-sexp), with point on the ^ mark (at the beginning of
"""string 3"""),
* before commit 0646c68, C-M-f moved to the end of """string 3""",
* now it just moves over one pair of quotes.
(Feel like I should note that single-line triple-quoted strings have
their uses FWIW, since they let one write single and double quotes into
the string without the need to backslash-escape them)
I see that the new commentary in python-syntax-stringify addresses this,
I guess?
> ;; FIXME: This makes sexp-movement a bit suboptimal since """a"""
> ;; is now treated as 3 strings.
So AFAICT while the change meant to improve the situation with
electric-pair-mode (which I'm grateful for, since I'm a happy user of
that minor mode), it did make sexp-movement a bit more "janky" (meaning
"hard to predict and rely on", no offense meant), *regardless* of
whether users (un)set (python-)forward-sexp-function 😕
next prev parent reply other threads:[~2022-06-23 6:49 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-22 6:50 bug#56135: 29.0.50; python.el - forward-sexp regression over triple-quoted strings Kévin Le Gouguec
[not found] ` <CALDnm50Mp21dyo899GC7oJHL+sSzF9JaUJty3xUgPAHm=k=_RA@mail.gmail.com>
[not found] ` <877d58juup.fsf@gmail.com>
[not found] ` <87y1xotlna.fsf@gmail.com>
2022-06-22 22:19 ` João Távora
2022-06-23 6:49 ` Kévin Le Gouguec [this message]
2022-06-23 6:53 ` bug#56135: [Kévin Le Gouguec] " Kévin Le Gouguec
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
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87fsjv27t0.fsf@gmail.com \
--to=kevin.legouguec@gmail.com \
--cc=56135@debbugs.gnu.org \
--cc=joaotavora@gmail.com \
--cc=monnier@iro.umontreal.ca \
/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 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).