You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2018/08/19 12:24:27 UTC
groovy git commit: Add a test and workaround for GROOVY-8409
Repository: groovy
Updated Branches:
refs/heads/master b5a8e6ad3 -> a70c7635c
Add a test and workaround for GROOVY-8409
We should refine `GenericsTypeName` to distinguish different GenericsType with same name
GROOVY-8409 is not fixed yet...
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/a70c7635
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/a70c7635
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/a70c7635
Branch: refs/heads/master
Commit: a70c7635ced04a831d362ecd2dab0f1602e86d42
Parents: b5a8e6a
Author: Daniel Sun <su...@apache.org>
Authored: Sun Aug 19 20:24:02 2018 +0800
Committer: Daniel Sun <su...@apache.org>
Committed: Sun Aug 19 20:24:25 2018 +0800
----------------------------------------------------------------------
src/test/groovy/bugs/Groovy8409Bug.groovy | 69 ++++++++++++++++++++++++++
1 file changed, 69 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/a70c7635/src/test/groovy/bugs/Groovy8409Bug.groovy
----------------------------------------------------------------------
diff --git a/src/test/groovy/bugs/Groovy8409Bug.groovy b/src/test/groovy/bugs/Groovy8409Bug.groovy
new file mode 100644
index 0000000..c4de8b7
--- /dev/null
+++ b/src/test/groovy/bugs/Groovy8409Bug.groovy
@@ -0,0 +1,69 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package groovy.bugs
+
+import gls.CompilableTestSupport
+import groovy.transform.NotYetImplemented
+
+class Groovy8409Bug extends CompilableTestSupport {
+ @NotYetImplemented
+ void test() {
+ assertScript '''
+ import groovy.transform.CompileStatic
+ import java.util.function.BiFunction
+
+ @CompileStatic
+ class Groovy8409Bug {
+
+ static <T> T actionWrapperT(BiFunction<Date, URL, T> action) {
+ T result = action.apply(new Date(), new URL('http://www.example.com'))
+ // do something else here
+ return result
+ }
+
+ static void main(String[] args) {
+ Groovy8409Bug t = actionWrapperT { Date date, URL url -> new Groovy8409Bug() }
+ }
+
+ }
+ '''
+ }
+
+ void testWorkaround() {
+ assertScript '''
+ import groovy.transform.CompileStatic
+ import java.util.function.BiFunction
+
+ @CompileStatic
+ class Groovy8409Bug {
+
+ static <X> X actionWrapperT(BiFunction<Date, URL, X> action) {
+ X result = action.apply(new Date(), new URL('http://www.example.com'))
+ // do something else here
+ return result
+ }
+
+ static void main(String[] args) {
+ Groovy8409Bug t = actionWrapperT { Date date, URL url -> new Groovy8409Bug() }
+ }
+
+ }
+ '''
+ }
+}