You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@groovy.apache.org by "Stanislav Poslavsky (JIRA)" <ji...@apache.org> on 2016/06/20 11:03:05 UTC
[jira] [Commented] (GROOVY-7869) Groovy category does now work with
abstract iterable classes
[ https://issues.apache.org/jira/browse/GROOVY-7869?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15339346#comment-15339346 ]
Stanislav Poslavsky commented on GROOVY-7869:
---------------------------------------------
I will very appreciate any workaround, since this bug breaks my DSL and I can't force the users to use old Groovy versions.
> Groovy category does now work with abstract iterable classes
> ------------------------------------------------------------
>
> Key: GROOVY-7869
> URL: https://issues.apache.org/jira/browse/GROOVY-7869
> Project: Groovy
> Issue Type: Bug
> Components: groovy-runtime
> Affects Versions: 2.4.0, 2.4.1, 2.4.2, 2.4.3, 2.4.4, 2.4.5, 2.4.6, 2.4.7
> Environment: OS X/Linux/Windows
> Reporter: Stanislav Poslavsky
> Priority: Critical
>
> If one has a simple abstract iterable class and want to overload *plus* operator using category, Groovy will use different implementation of 'plus' from that defined in the category.
> Abstract iterable class:
> {code:java}
> import java.util.Iterator;
> public abstract class MyAbstractClass implements Iterable<Number> {
> public final int num;
> public MyAbstractClass(int num) { this.num = num; }
> @Override
> public String toString() { return String.valueOf(num); }
> @Override
> public Iterator<Number> iterator() {
> //just some dummy empty iterator
> return new Iterator<Number>() {
> @Override
> public boolean hasNext() { return false; }
> @Override
> public Number next() { return null; }
> };
> }
> }
> {code}
> A single implementation:
> {code:java}
> public class MyAbstractClassImpl extends MyAbstractClass {
> public MyAbstractClassImpl(int a) { super(a); }
> }
> {code}
> Category that overloads *plus* operator:
> {code:java}
> class MyCategory {
> public static MyAbstractClass plus(MyAbstractClass a, MyAbstractClass b) {
> return new MyAbstractClassImpl(a.num + b.num);
> }
> }
> {code}
> Test case:
> {code:java}
> use(MyCategory) {
> def a = new MyAbstractClassImpl(1)
> def b = new MyAbstractClassImpl(2)
> println a + b //expected 3, but received []
> assert (a+b).num == 3 // so here we see exception
> }
> {code}
> Expected to see *3*, but instead Groovy returns empty list, since it uses plus operator defined for iterable, but not that is defined in MyCategory.
> There is no bug in version 2.2.x.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)