Skip to content

Simplify implementation of FilteredIterator #33902

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

Closed
wants to merge 1 commit into from

Conversation

polyglot-k
Copy link
Contributor

In my opinion, there are unnecessary 'else' phrases and if phrases.

This can be good in terms of readability, but deepening depth can actually be out of sight.

That's why I modified the same format to be used a little more concisely.

@pivotal-cla
Copy link

@KNU-K 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-cla
Copy link

@KNU-K Thank you for signing the Contributor License Agreement!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Nov 17, 2024
@sbrannen sbrannen self-assigned this Nov 17, 2024
@sbrannen sbrannen added type: task A general task in: core Issues in core modules (aop, beans, core, context, expression) and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Nov 17, 2024
@sbrannen sbrannen added this to the 6.2.1 milestone Nov 17, 2024
@sbrannen sbrannen changed the title refactor: Reduce unnecessary code depth for improved readability Simplify implementation of FilteredIterator Nov 17, 2024
sbrannen added a commit that referenced this pull request Nov 17, 2024
@sbrannen sbrannen closed this in e0e96c4 Nov 17, 2024
@polyglot-k
Copy link
Contributor Author

@sbrannen
i have one question.
Is my pr contributed?

@sbrannen
Copy link
Member

Hi @KNU-K,

Congratulations on submitting your first PR for the Spring Framework! 👍

This has been merged into 6.2.x and main in e0e96c4 and revised in 6544698.

Thanks

@sbrannen
Copy link
Member

sbrannen commented Nov 17, 2024

i have one question. Is my pr contributed?

Yes, it has.

Please note, however, that there is no need to ask that. You can see that yourself by looking at the "events" in the issue, for example: #33902 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: core Issues in core modules (aop, beans, core, context, expression) type: task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants