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.

Sign up for GitHub

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

Jump to bottom

removed all occurences of "".equals(...) except in Utils #65

Merged
merged 1 commit into from
Jul 13, 2011
Merged

removed all occurences of "".equals(...) except in Utils #65

merged 1 commit into from
Jul 13, 2011
+19 −22

Conversation

stefanbirkner
Copy link
Contributor

This is the second step mentioned in my last commit. From now on isStringBlank and isStringEmpty are the only methods using

"".equals(someString)

cbeust added a commit that referenced this pull request Jul 13, 2011
@cbeust
Merge pull request #65 from stefanbirkner/emptyblank2
897a865 
removed all occurences of "".equals(...) except in Utils
@cbeust cbeust merged commit 897a865 into testng-team:master Jul 13, 2011
@cbeust
Copy link
Collaborator

cbeust commented Jul 13, 2011

Thanks!

@stefanbirkner
Copy link
Contributor Author

I decided to be not consistent, because it makes no sense to me to use blank strings at the affected lines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@stefanbirkner @cbeust