Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Fix -Wlogical-not-parentheses warning#79

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
facchinm merged 3 commits intoarduino-libraries:masterfromvaipatel:master
Feb 26, 2021

Conversation

vaipatel
Copy link

@vaipatelvaipatel commentedNov 30, 2019•
edited
Loading

Hello, the above warning is issued forlines 87 and97. I believe the warning occurs becauseoperator!()has higher precedence thanoperator>().

Alternatively, we can use:

iClient->connect(iServerAddress, iServerPort) <= 0

facchinm reacted with thumbs up emoji
The warning occurs because `operator!()` has higher precedence than `operator>()`.Alternatively, we can use:```iClient->connect(iServerAddress, iServerPort) <= 0```
Copy link

@semcneilsemcneil left a comment

Choose a reason for hiding this comment

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

This fix worked for me as well. Can it be merged?

@FedeBev
Copy link

FedeBev commentedJun 5, 2020•
edited
Loading

Hi there,

what about this?

Copy link

@xxthunderxxthunder left a comment

Choose a reason for hiding this comment

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

Should be merged.

@facchinmfacchinm merged commit9a5afdf intoarduino-libraries:masterFeb 26, 2021
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@xxthunderxxthunderxxthunder approved these changes

@semcneilsemcneilsemcneil approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

5 participants
@vaipatel@FedeBev@xxthunder@semcneil@facchinm
[8]ページ先頭
©2009-2025 Movatter.jp