You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Getting the full match is a very common operation, and although the documetation for Regex::get says that getting the result at index 0 will always return Some, that property isn't reflected in the type system. And adding a method like this would expose that guarantee through the type of the public API.
I'm not sure full_match is the right name though. Hmmm. I'm not a huge fan of get_match either in this context. Probably match would be ideally, except for the obvious drawback that match is a Rust keyword.
Activity
BurntSushi commentedon Jan 9, 2024
I think this is probably a good idea. I added an analogous method (among others) to the lower level
regex-automata::util::captures::Captures
type.I'm not sure
full_match
is the right name though. Hmmm. I'm not a huge fan ofget_match
either in this context. Probablymatch
would be ideally, except for the obvious drawback thatmatch
is a Rust keyword.tmccombs commentedon Jan 9, 2024
Yeah, I'm not sure
full_match
is a great name either, but I couldn't think of a better one.ultrabear commentedon Jul 15, 2024
A potential other name could be
as_match
regex: add as_match method to Captures trait