Github user paulk-asert commented on a diff in the pull request:
https://github.com/apache/groovy/pull/282#discussion_r107062503
--- Diff: src/main/groovy/lang/GroovyCodeSource.java ---
@@ -164,7 +164,7 @@ public GroovyCodeSource(URI uri) throws IOException {
this(uri.toURL());
}
- public GroovyCodeSource(URL url) throws IOException {
--- End diff --
I only just spotted this now. While "unused" in the sense that this method
didn't throw this exception anymore (I presume it did in the past), removing
the exception breaks contract/source code compatibility. :-( I know it is a
result of how checked exceptions are handled on the JVM but we should have not
done this in a point release - all good to know in hindsight. Anyway, it isn't
clear that reverting after a year is going to help things so I'm thinking of
just documenting for now with a Jira issue. Apologies if this has already been
raised elsewhere.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
