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 2019/12/11 10:44:41 UTC

[groovy] 06/06: Add `serialVersionUID` field only for serializable lambda

This is an automated email from the ASF dual-hosted git repository.

sunlan pushed a commit to branch GROOVY_3_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit 72dcff2051e976ebe5cccf58faa262a8130c2732
Author: Daniel Sun <su...@apache.org>
AuthorDate: Wed Dec 11 18:00:36 2019 +0800

    Add `serialVersionUID` field only for serializable lambda
    
    (cherry picked from commit 3113667be88ef15a53d4e724022bba3edf3481bb)
---
 .../org/codehaus/groovy/classgen/asm/sc/StaticTypesLambdaWriter.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesLambdaWriter.java b/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesLambdaWriter.java
index a80c5c8..8f2fe0c 100644
--- a/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesLambdaWriter.java
+++ b/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesLambdaWriter.java
@@ -282,7 +282,9 @@ public class StaticTypesLambdaWriter extends LambdaWriter implements AbstractFun
         answer.setUsingGenerics(outerClass.isUsingGenerics());
         answer.setSourcePosition(expression);
 
-        addSerialVersionUIDField(answer);
+        if (expression.isSerializable()) {
+            addSerialVersionUIDField(answer);
+        }
 
         if (staticMethodOrInStaticClass) {
             answer.setStaticClass(true);