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

Memory leaks when converting data to/from C #1199

Closed
maelle opened this issue Feb 5, 2024 · 2 comments · Fixed by #1310
Closed

Memory leaks when converting data to/from C #1199

maelle opened this issue Feb 5, 2024 · 2 comments · Fixed by #1310
Assignees
@Antonov548
Labels
c code
Milestone
upgrade-2

Comments

@maelle
Copy link
Contributor

maelle commented Feb 5, 2024

#1196
#1173 (comment)

@szhorvat I'm opening an issue so we can close this once we're sure everything has been covered.
@maelle maelle added the c code label Feb 5, 2024
@szhorvat
Copy link
Member

szhorvat commented Feb 5, 2024

Some of this was done in #1196, but it needs to be verified that nothing is left

@szhorvat szhorvat changed the title memory leaks when converting data to C Memory leaks when converting data to/from C Feb 6, 2024
@krlmlr krlmlr added this to the upgrade-2 milestone Feb 20, 2024
@szhorvat
Copy link
Member

#1195 will help with testing that we covered everything already.

@Antonov548 Antonov548 mentioned this issue Mar 19, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Antonov548 Antonov548

Labels
c code
Projects
None yet
Milestone
upgrade-2
Development

Successfully merging a pull request may close this issue.

ci: run examples with comment \dontrun{} igraph/rigraph
4 participants
@szhorvat @krlmlr @maelle @Antonov548