From: "Andreas Röhler" <andreas.roehler@easy-emacs.de>
To: 23443@debbugs.gnu.org
Subject: bug#23443: 25.0.93; Temporary change to syntax-table is in effect during syntax-propertize
Date: Fri, 13 May 2016 07:49:16 +0200 [thread overview]
Message-ID: <57356ADC.1060007@easy-emacs.de> (raw)
In-Reply-To: <CAM-tV-8OidLK6zZpPqC45Y+82rB+W-eKzfbjX1=KmBLdMU0FeA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1143 bytes --]
|reopen| bugnumber
The proposed solution, which has to bind syntax-propertize-function to
nil before calling scan-sexp, reads like a work around. These things
shouldn't be necessary.
Assume a fix must affect usage of syntax-propertize-function and probably is not a trivial one.
Its design seems worth further discussion.
For example reading:
"The specified function may call ‘syntax-ppss’ on any position
before END, but it should not call ‘syntax-ppss-flush-cache’,
which means that it should not call ‘syntax-ppss’ on some
position and later modify the buffer on some earlier position."
Emacs can do better.
On 12.05.2016 12:05, Noam Postavsky wrote:
> close 23443
> tags 23443 + notabug
> quit
>
> On Sun, May 8, 2016 at 3:12 PM, Dmitry Gutov <dgutov@yandex.ru> wrote:
>> Here's a possible stopgap:
>>
>> - Narrow the buffer to the snippet (just to be safe).
>> - Bind syntax-propertize-function to nil.
>> - Call scan-sexps.
>> - Call (syntax-ppss-flush-cache snippet-beginning-position).
> Thanks, this seems to be working well (implemented in [1]).
>
> [1]: https://github.com/capitaomorte/yasnippet/pull/695
>
>
>
[-- Attachment #2: Type: text/html, Size: 1854 bytes --]
next prev parent reply other threads:[~2016-05-13 5:49 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-04 10:41 bug#23443: 25.0.93; Temporary change to syntax-table is in effect during syntax-propertize Noam Postavsky
2016-05-08 0:58 ` Dmitry Gutov
2016-05-08 18:51 ` Noam Postavsky
2016-05-08 18:53 ` Dmitry Gutov
2016-05-08 18:58 ` Noam Postavsky
2016-05-08 19:12 ` Dmitry Gutov
2016-05-12 10:05 ` Noam Postavsky
2016-05-13 5:49 ` Andreas Röhler [this message]
2016-05-16 18:28 ` Dmitry Gutov
2016-05-13 5:55 ` Andreas Röhler
2016-05-10 17:34 ` Andreas Röhler
2016-05-10 20:42 ` Dmitry Gutov
2016-05-11 5:20 ` Andreas Röhler
2016-05-11 9:20 ` Dmitry Gutov
2016-05-11 12:13 ` Andreas Röhler
2016-05-11 12:31 ` Dmitry Gutov
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=57356ADC.1060007@easy-emacs.de \
--to=andreas.roehler@easy-emacs.de \
--cc=23443@debbugs.gnu.org \
/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.