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

initialize local variables before use #693

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

kraj
Copy link
Contributor

@kraj kraj commented Dec 24, 2019

fixes
error: 'ctrl_reg2_data' may be used uninitialized in this function [-Werror=maybe-uninitialized]

Signed-off-by: Khem Raj raj.khem@gmail.com

fixes
error: 'ctrl_reg2_data' may be used uninitialized in this function [-Werror=maybe-uninitialized]

Signed-off-by: Khem Raj <raj.khem@gmail.com>
CMakeLists.txt Outdated Show resolved Hide resolved
clang also introduced this warning now

Signed-off-by: Khem Raj <raj.khem@gmail.com>
@Propanu
Copy link
Contributor

Propanu commented Jan 10, 2020

Hi @kraj, these changes are good but you would need to sign the ECA for us to be able to merge this. You can find the details on how to do so here: https://www.eclipse.org/legal/ECA.php
Thanks!

@kraj
Copy link
Contributor Author

kraj commented Jan 10, 2020

Hi @kraj, these changes are good but you would need to sign the ECA for us to be able to merge this. You can find the details on how to do so here: https://www.eclipse.org/legal/ECA.php
Thanks!

I have tried to install it, please recheck,

@Propanu
Copy link
Contributor

Propanu commented Jan 10, 2020

@kraj, you shouldn't have to install anything. Simply fill and submit the ECA Form on the eclipse.org website using the same e-mail from your sign-off. We can then refresh this PR and the eclipse ECA check will pass.

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.

None yet

3 participants