mutant icon indicating copy to clipboard operation
mutant copied to clipboard

Add mutation `/(foo)/` → `/(?>foo)/`

Open backus opened this issue 9 years ago • 1 comments

backus avatar May 23 '16 08:05 backus

@mbj I'm undecided on how useful I think this mutation is. We already mutate from /(foo)/ to /(?:foo)/ so we already get the lack of capture that this would achieve. I think atomic groups probably only make sense in fairly niche situations so I'm worried it may be confusing/misleading to generate atomic group mutations when we already have the mutation to non-capturing. I'm assuming we also don't want to go from atomic -> non-capturing or non-capturing -> atomic, but wanted to get your opinion here.

I have an implementation locally (it is extremely simple) but I am waiting to push until there is a chance for discussion.

dgollahon avatar Jan 02 '21 17:01 dgollahon