Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace all usages of method references (aka ::) with lambdas #2848

Closed
markushi opened this issue Jul 19, 2023 · 0 comments · Fixed by #2857
Closed

Replace all usages of method references (aka ::) with lambdas #2848

markushi opened this issue Jul 19, 2023 · 0 comments · Fixed by #2857

Comments

@markushi
Copy link
Member

Description

As discussed here using method references might causes issues in combination with older version of the AGP.

Overall there are quite some usages of method references(>50), but since this issue is only relevant on Android, it only needs to be fixed for those parts.

E.g. JsonObjectSerializer.java

case STRING:
        done = handlePrimitive(reader::nextString);

should be replaced by

case STRING:
        done = handlePrimitive(() -> reader.nextString());
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants