Skip to content

initialize local variables before use#693

Open
kraj wants to merge 2 commits into
eclipse-upm:masterfrom
kraj:master
Open

initialize local variables before use#693
kraj wants to merge 2 commits into
eclipse-upm:masterfrom
kraj:master

Conversation

@kraj

@kraj kraj commented Dec 24, 2019

Copy link
Copy Markdown
Contributor

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>
Comment thread CMakeLists.txt Outdated
clang also introduced this warning now

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

Propanu commented Jan 10, 2020

Copy link
Copy Markdown
Contributor

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

kraj commented Jan 10, 2020

Copy link
Copy Markdown
Contributor Author

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

Propanu commented Jan 10, 2020

Copy link
Copy Markdown
Contributor

@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.

3 participants