-
Notifications
You must be signed in to change notification settings - Fork 62
Add Vips::Image#close.
#340
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
Conversation
Hi. Just wondering. It is the thing that you've mentioned here? #67 (comment)
As I understand you said there it didn't help, only the GC.start did or something.
Wow that was a while ago.
Well, I have not run the benchmarks again. That being said, in theory a close method makes sense. In practice, more work might be needed to make it practical/useful.
Hi, good idea, I've been meaning to add this.
Could we call it invalidate? That's the name we've used in the other libvips bindings.
close is rather close to post_close, the name we use for cleanup-on-finalize handlers.
jdelStrother
commented
Jan 23, 2025
Doesn't this also need a Vips.attach_function :vips_image_invalidate_all, [:pointer], :void somewhere? Without it I get NoMethodError
I've been using this to help deal with memory usage issues.