bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#65470: 29.1.50; js-ts-mode: regex pattern can cause incorrect parent


From: Eli Zaretskii
Subject: bug#65470: 29.1.50; js-ts-mode: regex pattern can cause incorrect parenthesis matching
Date: Sun, 17 Sep 2023 08:22:00 +0300

> Date: Sat, 16 Sep 2023 23:05:35 +0300
> Cc: jostein@secure.kjonigsen.net, casouri@gmail.com, theo@thornhill.no,
>  jostein@kjonigsen.net, btuin@mailo.com, 65470@debbugs.gnu.org
> From: Dmitry Gutov <dmitry@gutov.dev>
> 
> On 16/09/2023 16:59, Stefan Monnier wrote:
> >>> No problem, pushed that to emacs-29, thanks.
> >> This seems to cause:
> >>
> >>      INFO     Scraping files for loaddefs...85%
> >>    typescript-ts-mode:0: Warning: Not registering prefix "ts".  Affects: 
> >> ("tsx-ts-mode--indent-compatibility-b893426" 
> >> "tsx-ts-mode--font-lock-compatibility-bb1f97b" "tsx-ts--s-p-query" 
> >> "tsx-ts--syntax-propertize" "ts-ts--s-p-query" "ts-ts--syntax-propertize" 
> >> "ts-ts--syntax-propertize-captures")
> >>
> >> Stefan, what do we usually do with such warnings?
> > We usually ignored them?
> > [ They only affect the features controlled by
> >    `help-enable-(completion-)autoload`.  ]
> > 
> > A better option, is to try and use fewer namespace prefixes.
> > e.g. make sure all the definitions start with `ts-`, `ts-ts-`,
> > `tsts-`, or somesuch (except for the autoloaded ones which can break
> > that rule).
> > 
> > Maybe rename `typescript-ts-mode-*` to `ts-ts-mode-*` (and
> > make `typescript-ts-mode` an alias of `ts-ts-mode`).
> 
> Thanks. I haven't seen the warnings myself, but I sorted the new 
> addition into two existing prefixes in that file: typescript-ts and tsx-ts.

Thanks, compiles cleanly now.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]