Re: [PR] GROOVY-11260: ASTMatcher should support matching a var-arg placeholder [groovy]

2024-01-01 Thread via GitHub
paulk-asert merged PR #2022: URL: https://github.com/apache/groovy/pull/2022 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@g

Re: [PR] GROOVY-11260: ASTMatcher should support matching a var-arg placeholder [groovy]

2023-12-29 Thread via GitHub
paulk-asert commented on PR #2022: URL: https://github.com/apache/groovy/pull/2022#issuecomment-1872397053 It currently doesn't check whether the placeholder names and varargPlaceholder names are distinct. It may not matter but code with such name clashes might become harder to understand,