Skip to content

Conversation

@siddharthkarn2212
Copy link

This PR clarifies the Javadoc of StringUtils.isAnyBlank to explicitly describe
the behavior difference between a null CharSequence argument and a null array
of CharSequences.

No functional changes are made; this update aligns the documentation with
existing behavior and tests.

Copy link
Member

@garydgregory garydgregory left a comment

Choose a reason for hiding this comment

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

-1 Our documentation for this type of information is repeated in many places, over and over. It doesn't make sense to change one random sample. To me, this looks like change for the sake of change. The current documentation is brief and simple. IMO, this doesn't need to be more verbose. As it stands, I think this PR can be closed.

@siddharthkarn2212
Copy link
Author

Thanks for the review, Gary.

I understand your point about avoiding redundant documentation and agree that the existing Javadoc is sufficient. I’m happy to close this PR.

Thanks for your time and feedback.

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