Skip to content

Add property [IsDisposed] for CudaContext & CudaRuntimeCompiler#100

Open
dinggeonly wants to merge 1 commit into
kunzmi:masterfrom
dinggeonly:master
Open

Add property [IsDisposed] for CudaContext & CudaRuntimeCompiler#100
dinggeonly wants to merge 1 commit into
kunzmi:masterfrom
dinggeonly:master

Conversation

@dinggeonly

Copy link
Copy Markdown

Add property [IsDisposed] for CudaContext & CudaRuntimeCompiler,
to make it is possible to know whether the object is disposed.

@kunzmi

kunzmi commented Jun 5, 2021

Copy link
Copy Markdown
Owner

Hi, sorry for the late answer. Do you know if this property is common for the IDisposible interface? Should we add it to all IDisposible classes?

Cheers,
Michael

@dinggeonly

Copy link
Copy Markdown
Author

@kunzmi
Q:Should we add it to all IDisposible classes?
A:In my opinion,it's unnecessary to change other calsses,unless we need to know the status of it.

@bjorg

bjorg commented Jun 11, 2025

Copy link
Copy Markdown

I've never seen or used IsDisposed. Isn't the expectation for Dispose() to be idempotent?

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