Skip to content

Conversation

ajsecord
Copy link
Member

Copy material-components/material-components-ios#2456 until the warnings can be rationalized.

@ajsecord ajsecord requested a review from jverkoey November 16, 2017 19:31
@jverkoey
Copy link
Contributor

Are we intentionally disabling all of the unused warnings?

# * Our basic warnings cover most things and make warnings into errors.
# * If a warning is (transitively) enabled by our basic warnings, do not include it separately.
# * For warnings not enabled by our basic warnings, add them to the extra warnings.
# * To change a warning-as-error into just a warning, add it to the ignorable warnings.
Copy link
Contributor

Choose a reason for hiding this comment

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

"...using the format -Wno-error=<warning>"

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

# * If a warning is (transitively) enabled by our basic warnings, do not include it separately.
# * For warnings not enabled by our basic warnings, add them to the extra warnings.
# * To change a warning-as-error into just a warning, add it to the ignorable warnings.
# * To disable a warning completely, add it to the disabled warnings.
Copy link
Contributor

Choose a reason for hiding this comment

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

"...using the format -Wno-<warning>"

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

@ajsecord
Copy link
Member Author

No. -Wall implies -Wunused, which includes all of them. When -Wextra is included, -Wunused also turns on -Wunused-parameters, which is why we turn it back off again.

@jverkoey
Copy link
Contributor

Got it!

@ajsecord
Copy link
Member Author

Done, PTAL.

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