Skip to content

cleanup Syntax changes. #7940

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

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

cleanup Syntax changes. #7940

wants to merge 1 commit into from

Conversation

apps4u
Copy link

@apps4u apps4u commented Aug 28, 2018

I wanted to help and I think to get to know a project its good to do a clean up and learn the code.
So this is a heap of code changes but Ive not changed the behaviour of the program, I've only made Syntax changes eg:
used Lambda instead of anonymous classes.
cleaned up some of the stream api usage.
add added a few Objects.requireNonNull.
and added a few missing @Suppress annotations.

Signed-off-by: Jason Kristian jasonk@apps4u.com.au

Signed-off-by: Jason Kristian <jasonk@apps4u.com.au>
@apps4u apps4u changed the title cleanup Aug 28, 2018
@CLAassistant
Copy link

CLAassistant commented Apr 9, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Jason Kristian seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants