Skip to content

ModelicaInternal.c: Silence Clang warning #4666

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

ModelicaInternal.c: Silence Clang warning #4666

wants to merge 1 commit into from

Conversation

beutlich
Copy link
Member 

ModelicaInternal.c:1309:15: note: initialize the variable 'result' to silence this warning

@beutlich
Silence Clang warning
1829042 
ModelicaInternal.c:1309:15: note: initialize the variable 'result' to silence this warning
@beutlich beutlich added this to the MSL4.2.0 milestone Jun 19, 2025
@beutlich beutlich requested review from sjoelund and HansOlsson June 19, 2025 18:48
@beutlich beutlich added the L: C-Sources Issue addresses Modelica/Resources/C-Sources label Jun 19, 2025
@beutlich beutlich changed the title Silence Clang warning ModelicaInternal.c: Silence Clang warning Jun 19, 2025
sjoelund
sjoelund approved these changes Jun 19, 2025
View reviewed changes
Copy link
Member

@sjoelund sjoelund left a comment

Choose a reason for hiding this comment

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

I suppose that's a good default in case ModelicaError actually returns.

@beutlich beutlich enabled auto-merge (squash) June 23, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sjoelund sjoelund sjoelund approved these changes

@HansOlsson HansOlsson Awaiting requested review from HansOlsson

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
L: C-Sources Issue addresses Modelica/Resources/C-Sources
Projects
None yet
Milestone
MSL4.2.0
Development

Successfully merging this pull request may close these issues.
2 participants
@beutlich @sjoelund