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

Tokenizer - removing unnecessary boxing #1695

Closed
wants to merge 1 commit into from

Conversation

vladad
Copy link

@vladad vladad commented Feb 20, 2018

Instead of "Integer.valueOf(ch)" can be simply "(int) ch"

Instead of "Integer.valueOf(ch)" can be simply "(int) ch"
@pivotal-issuemaster
Copy link

@vladad Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@vladad Thank you for signing the Contributor License Agreement!

@luyaor
Copy link

luyaor commented Jul 14, 2018

Dear Vladimír Domes,

We are researchers working on identifying redundant development and duplicated pull requests. We have found there is a pull request: #1817 which might be a potentially duplicate to this one. We would really appreciate if you could help us to validate and give us feedback.

Thank you very much for your time!

Sincerely,
Luyao

@snicoll
Copy link
Member

snicoll commented Jul 14, 2018

@FancyCoder0 good catch again, thank you.

@vladad thank you for opening your first pull request in Spring Framework and apologize for not triaging it properly. This has been fixed in the meantime.

@snicoll snicoll closed this Jul 14, 2018
@sbrannen sbrannen added the status: superseded An issue that has been superseded by another label May 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: superseded An issue that has been superseded by another
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants