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

Catch and fix more warnings in Java sources #2419

Merged
merged 3 commits into from
Oct 23, 2020
Merged

Conversation

ennru
Copy link
Member

@ennru ennru commented Sep 21, 2020

Enable more Java linting flags andfix a few warnings detected by it.

@seglo
Copy link
Member

seglo commented Sep 22, 2020

Needs some javafmtAll

@seglo
Copy link
Member

seglo commented Sep 22, 2020

Doesn't like some of your flags, -Xlint:removal https://travis-ci.com/github/akka/alpakka/jobs/389305854#L575

@ennru
Copy link
Member Author

ennru commented Sep 22, 2020

Oh, I only tried it on JDK 11.

Copy link
Member

@raboof raboof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

neat

public static Source<Pair<Path, DirectoryChange>, NotUsed> create(
Path directoryPath, FiniteDuration pollInterval, int maxBufferSize) {
return Source.fromGraph(
new DirectoryChangesSource(directoryPath, pollInterval, maxBufferSize, Pair::apply));
new DirectoryChangesSource<>(directoryPath, pollInterval, maxBufferSize, Pair::apply));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@ennru ennru merged commit 5620bfd into akka:master Oct 23, 2020
@ennru ennru deleted the java-warnings branch October 23, 2020 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants