Skip to content
This repository has been archived by the owner on Sep 17, 2020. It is now read-only.

Add $deserializeLambda$ to a set of known serialization-related methods #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jerrinot
Copy link

@jerrinot jerrinot commented Nov 7, 2016

Fixes #31

I'm not sure how to add a test. I tried to add

    @AssertNoWarning("UncalledPrivateMethod")
    private static class SerializedClass implements Serializable {
        private int foo;

        private Runnable foo() {
            return () -> {
                System.out.println(foo);
            };
        }
    }

but that's passing even without the patch. I guess I'd need to annotate the $deserializeLambda$ method, but the method is generated by Java compiler.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.21% when pulling bb28c60 on jerrinot:fixes/lambda-deserialization-method/master into 3fff9d6 on amaembo:master.

@amaembo
Copy link
Owner

amaembo commented Nov 8, 2016

What about return (Runnable & Serializable) () -> ...? Will it still pass without patch?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants