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

Avoid compiler complaining #3741

Closed
wants to merge 1 commit into from
Closed

Avoid compiler complaining #3741

wants to merge 1 commit into from

Conversation

InfoHunter
Copy link
Member

initialize some local variables

@InfoHunter
Avoid compiler complaining
d4235fc 
initialize some local variables
@InfoHunter InfoHunter mentioned this pull request Jun 21, 2017
Closed
richsalz
richsalz approved these changes Jun 21, 2017
View reviewed changes
Copy link
Contributor

@richsalz richsalz left a comment

Choose a reason for hiding this comment

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

ping @mattcaswell

@bernd-edlinger
Copy link
Member

gcc-4.8.4 and gcc-trunk does not warn here.
which compiler version does emit this wrong warning?

@mattcaswell mattcaswell added the approval: done This pull request has the required number of approvals label Jun 21, 2017
@richsalz
Copy link
Contributor

merging now.

@richsalz
Copy link
Contributor

clang complains.

@richsalz
Copy link
Contributor

fixed, thank you!

levitte pushed a commit that referenced this pull request Jun 21, 2017
@InfoHunter
Avoid compiler complaining
15b1688 
initialize some local variables

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #3741)
@levitte
Copy link
Member

levitte commented Jun 21, 2017

Rebase?

@richsalz richsalz closed this Jun 22, 2017
@InfoHunter InfoHunter deleted the fix-build branch June 22, 2017 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richsalz richsalz richsalz approved these changes

@mattcaswell mattcaswell mattcaswell approved these changes

Assignees
No one assigned
Labels
approval: done This pull request has the required number of approvals
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@InfoHunter @bernd-edlinger @richsalz @levitte @mattcaswell