polygene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Niclas Hedhman <nic...@hedhman.org>
Subject Massive bug...
Date Mon, 15 May 2017 12:57:53 GMT
Gang,

By mistake I found a big bug...


@Concerns( DefaultMethodsGenericSideEffect.class )
public interface DefaultMethodsGenericSideEffects extends DefaultMethods
{
    @Override
    default String sayHello( String name )
    {
        return greeting().get() + ", " + name + '!';
    }
}

@AppliesTo( DefaultMethodsFilter.class )
public static class DefaultMethodsGenericSideEffect extends
SideEffectOf<InvocationHandler>
    implements InvocationHandler
{



Do you see it?? Point is that there is no warning or anything, but the
SideEffect is applied as a Concern, and the value returned from the
SideEffect is the result of the method call.

The description of the bug should be something like; "Concerns lacking
@ConcernFor injection field is not rejected as a Concern" and the same
thing is like for SideEffects, but I haven't tested that yet.


Cheers
-- 
Niclas Hedhman, Software Developer
http://polygene.apache.org - New Energy for Java

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message