Skip to content

[clang-tidy] Simplify boolean expressions#1301

Closed
neheb wants to merge 1 commit into
facebook:masterfrom
neheb:hj
Closed

[clang-tidy] Simplify boolean expressions#1301
neheb wants to merge 1 commit into
facebook:masterfrom
neheb:hj

Conversation

@neheb

@neheb neheb commented Jan 3, 2020

Copy link
Copy Markdown
Contributor

Found with readability-simplify-boolean-expr

Signed-off-by: Rosen Penev rosenp@gmail.com

Comment thread folly/io/async/AsyncSSLSocket.cpp
Comment thread folly/io/async/AsyncSSLSocket.cpp
Found with readability-simplify-boolean-expr

Signed-off-by: Rosen Penev <rosenp@gmail.com>

@facebook-github-bot facebook-github-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Orvid has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@Orvid merged this pull request in 3c6f1f7.

dotconnor pushed a commit to 5448C8/folly that referenced this pull request Mar 19, 2021
Summary:
Found with readability-simplify-boolean-expr

Signed-off-by: Rosen Penev <rosenp@gmail.com>
Pull Request resolved: facebook#1301

Reviewed By: yfeldblum

Differential Revision: D19302630

Pulled By: Orvid

fbshipit-source-id: 9833e4ab3fe93592fa8f868f6ed2d56a263949fe
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants