hg: lambda/lambda/langtools: Updated functional interface test as per latest spec draft (v0.4)

[Ali Ebrahimi]

Hi,
An another catch in latest push:

+++ b/src/share/classes/com/sun/tools/javac/comp/Attr.java	Wed Aug 10
17:48:09 2011 +0100
@@ -1840,8 +1840,8 @@ public class Attr extends JCTree.Visitor
                     identifyLambdaCandidate &&
                     clazztype.tag == CLASS &&
                     lambdaOrReferenceAllowed(tree) &&
-                    !types.findSAM(clazztype).isErroneous()) {
-                Types.SAMResult res = types.findSAM(clazztype);
+                    !types.isSam(clazztype.tsym)) { <==========================
+                MethodSymbol samDescriptor =
types.findDescriptor(clazztype.tsym);

Is this "!types.isSam(clazztype.tsym)" Or "types.isSam(clazztype.tsym)"?


Best Regards,
Ali Ebrahimi
On 8/10/11, maurizio.cimadamore at oracle.com
<maurizio.cimadamore at oracle.com> wrote:
> Changeset: b9e08af836ed
> Author:    mcimadamore
> Date:      2011-08-10 17:48 +0100
> URL:
> http://hg.openjdk.java.net/lambda/lambda/langtools/rev/b9e08af836ed
>
> Updated functional interface test as per latest spec draft (v0.4)
> *) Removed Types.SAMResult
> *) Revised implementation for merging signatures of override-equivalent
> methods
> *) Added support for caching result of sam-ness test
> Updated code-generation support
> *) Added support for method type constants as static args in indy calls
>
> ! src/share/classes/com/sun/tools/javac/code/Types.java
> ! src/share/classes/com/sun/tools/javac/comp/Attr.java
> ! src/share/classes/com/sun/tools/javac/comp/Check.java
> ! src/share/classes/com/sun/tools/javac/comp/Infer.java
> ! src/share/classes/com/sun/tools/javac/comp/LambdaToMethod.java
> ! src/share/classes/com/sun/tools/javac/comp/LambdaTranslator.java
> ! src/share/classes/com/sun/tools/javac/jvm/ClassWriter.java
> ! src/share/classes/com/sun/tools/javac/resources/compiler.properties
> ! test/tools/javac/diags/examples.not-yet.txt
> ! test/tools/javac/lambda/BadConv03.out
> + test/tools/javac/lambda/BadConv04.java
> + test/tools/javac/lambda/BadConv04.out
> ! test/tools/javac/lambda/BadLambdaPos.out
> ! test/tools/javac/lambda/BadTargetType.out
> ! test/tools/javac/lambda/LambdaConv09.out
> ! test/tools/javac/lambda/LambdaExpr10.out
> ! test/tools/javac/lambda/MethodReference04.out
> ! test/tools/javac/lambda/TargetType17.out
> ! test/tools/javac/lambda/sqe/SAM_types/Helper.java
> ! test/tools/javac/lambda/sqe/SAM_types/LambdaTest2_SAM3.java
> + test/tools/javac/lambda/sqe/SAM_types/LambdaTest2_neg1.java
> + test/tools/javac/lambda/sqe/SAM_types/LambdaTest2_neg1.out
> ! test/tools/javac/lambda/sqe/SAM_types/NonSAM1.out
> ! test/tools/javac/lambda/sqe/SAM_types/NonSAM3.out
> ! test/tools/javac/lambda/sqe/lambdaExpression/AbstractClass_neg.out
> ! test/tools/javac/lambda/sqe/lambdaExpression/InvalidExpression5.out
>
>
>