Skip to content

Always initialize c_options=NULL#2751

Open
MichaelChirico wants to merge 1 commit into
igraph:mainfrom
MichaelChirico:patch-1
Open

Always initialize c_options=NULL#2751
MichaelChirico wants to merge 1 commit into
igraph:mainfrom
MichaelChirico:patch-1

Conversation

@MichaelChirico

Copy link
Copy Markdown
Contributor

This was flagged by clang's -Wsometimes-uninitialized.

@krlmlr

krlmlr commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Thanks for raising this. Could the problem be not the c_options but that Rz_SEXP_to_igraph_arpack_options() doesn't fully initialize the whole struct?

Do we need a specific version of clang to replicate this?

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.

2 participants