-
Notifications
You must be signed in to change notification settings - Fork 49
Implement .as for Regex and Unify Match and AnyRegexOutput #376
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
592e377
to
7e1ab7d
Compare
@swift-ci please test |
@@ -17,17 +17,13 @@ extension Regex { | |||
/// providing direct access to captures. | |||
@dynamicMemberLookup | |||
public struct Match { | |||
let input: String | |||
let anyRegexOutput: AnyRegexOutput | |||
|
|||
/// The range of the overall match. | |||
public let range: Range<String.Index> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't ARO store this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ARO inside of Match does not store it, but match.output where Output == ARO does store it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wait, so sometimes ARO stores it and sometimes it doesn't? That seems like a footgun.
@swift-ci please test |
@milseman anything else for this? |
No description provided.