Skip to content

ISSUE #655: Use Lambda expression when possible#852

Closed
pasha-kuznetsov wants to merge 3 commits into
apache:masterfrom
pasha-kuznetsov:lambdas
Closed

ISSUE #655: Use Lambda expression when possible#852
pasha-kuznetsov wants to merge 3 commits into
apache:masterfrom
pasha-kuznetsov:lambdas

Conversation

@pasha-kuznetsov
Copy link
Copy Markdown
Contributor

@pasha-kuznetsov pasha-kuznetsov commented Dec 14, 2017

Notes:

  1. This only includes lambda changes, w/o the potential CompletableFuture changes discussed in Issue Use Lambda expression when possible #655.
  2. It assumes that PR ISSUE #668: Use try-with-resources blocks whenever is possible #845 is committed first.
  3. Note the 6a7e238 sync issue (flagged by the spotbugs check).
  4. Adding ?w=1 (or &w=1) to the Github PR URL ignores whitespace diffs to simplify PR review.

@merlimat
Copy link
Copy Markdown
Contributor

I'm all for this, though since we are in the process of merging a lot of changes, this would lead to a number of additional conflicts. Can we wait few more weeks before converting everything to lambda?

@pasha-kuznetsov
Copy link
Copy Markdown
Contributor Author

Sure, I'll stash these for now. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants