unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#5393: Patch for lookaround assertion in regexp
       [not found] <lctyun2on2.fsf@fencepost.gnu.org>
@ 2019-06-27 19:17 ` Mattias Engdegård
  2020-08-14 13:15   ` Lars Ingebrigtsen
  0 siblings, 1 reply; 2+ messages in thread
From: Mattias Engdegård @ 2019-06-27 19:17 UTC (permalink / raw)
  To: Lars Ingebrigtsen, stephen.berman, rgm, 5393, t.matsuyama.pub

>That looks pretty cool to me, but there was next to no discussion about this at the time (2009), which is a bit odd... 

Didn't see it until now myself, and it looks useful, but perhaps it's unwise to add features that couldn't easily make it into a DFA-based engine. Positive lookahead maybe, the rest probably not. (The old engine would need to be kept around for backrefs, but those are fairly rare in actual use.)






^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#5393: Patch for lookaround assertion in regexp
  2019-06-27 19:17 ` bug#5393: Patch for lookaround assertion in regexp Mattias Engdegård
@ 2020-08-14 13:15   ` Lars Ingebrigtsen
  0 siblings, 0 replies; 2+ messages in thread
From: Lars Ingebrigtsen @ 2020-08-14 13:15 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: rgm, 5393, stephen.berman, t.matsuyama.pub

Mattias Engdegård <mattiase@acm.org> writes:

>>That looks pretty cool to me, but there was next to no discussion
> about this at the time (2009), which is a bit odd...
>
> Didn't see it until now myself, and it looks useful, but perhaps it's
> unwise to add features that couldn't easily make it into a DFA-based
> engine. Positive lookahead maybe, the rest probably not. (The old
> engine would need to be kept around for backrefs, but those are fairly
> rare in actual use.)

Right.  Well, there doesn't seem to be much enthusiasm for this patch,
and it doesn't apply (not only because the file names have changed, but
also other changes in the code), so I'm closing this bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-08-14 13:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <lctyun2on2.fsf@fencepost.gnu.org>
2019-06-27 19:17 ` bug#5393: Patch for lookaround assertion in regexp Mattias Engdegård
2020-08-14 13:15   ` Lars Ingebrigtsen

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).