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

readertest: Suppress "dangling-else" warning on GCC 7 and later #669

Merged
merged 1 commit into from
Jun 28, 2016
Merged

readertest: Suppress "dangling-else" warning on GCC 7 and later #669

merged 1 commit into from
Jun 28, 2016

Conversation

pah
Copy link
Contributor

@pah pah commented Jun 27, 2016

GCC 6.x doesn't yet support this warning flag, as reported by @ragnar-ouchterlony in #666.

This has now been verified on Linux, x86_64, with GCC versions

g++-5 (Debian 5.4.0-4) 5.4.0 20160609
g++-6 (Debian 6.1.1-7) 6.1.1 20160620
g++-snapshot (Debian 20160612-1) 7.0.0 20160612 (experimental) [trunk revision 237336]

@pah
readertest: Suppress "dangling-else" warning on GCC 7 and later
ad32940 
GCC 6.x doesn't yet support this warning flag, as reported by
@ragnar-ouchterlony.
@pah pah mentioned this pull request Jun 27, 2016
Closed
@coveralls
Copy link

coveralls commented Jun 27, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling ad32940 on pah:fixes/666 into 6da9227 on miloyip:master.

@miloyip miloyip merged commit f51d7c9 into Tencent:master Jun 28, 2016
@pah pah deleted the fixes/666 branch June 29, 2016 06:02
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
3 participants
@pah @coveralls @miloyip