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.

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

Fix strict-warnings build #4395

Closed
wants to merge 1 commit into from
Closed

Fix strict-warnings build #4395

wants to merge 1 commit into from

Conversation

p-steuer
Copy link
Member

Compilation failed due to -Werror=misleading-identation.

Compilation failed due to -Werror=misleading-identation.

Signed-off-by: Patrick Steuer <patrick.steuer@de.ibm.com>
@richsalz richsalz added approval: done This pull request has the required number of approvals branch: master Merge to master branch labels Sep 20, 2017
@dot-asm
Copy link
Contributor

dot-asm commented Sep 21, 2017

There seem to be typo in commit message, it's indentation, not identation, right? This can be fixed by committer.

@levitte
Copy link
Member

levitte commented Sep 21, 2017

Correct, @dot-asm

Copy link
Contributor

@kaduk kaduk left a comment

Choose a reason for hiding this comment

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

Let's see if I can merge this one...

levitte pushed a commit that referenced this pull request Sep 21, 2017
Compilation failed due to -Werror=misleading-indentation.

Signed-off-by: Patrick Steuer <patrick.steuer@de.ibm.com>

Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Ben Kaduk <kaduk@mit.edu>
(Merged from #4395)
@kaduk
Copy link
Contributor

kaduk commented Sep 21, 2017

success! Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants