-
-
Notifications
You must be signed in to change notification settings - Fork 216
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
Add IDevice::close() #748
Comments
Hi, I am interested in solving this issue. |
Thanks, Leonardo! Be sure to pull fresh develop, it has some recent changes in IDevice. |
Hi @gavv , what status code should it return in case the device does not close successfully? Or it always return StatusOK? |
Same as from write - ErrFile or ErrDevice depending on device type. |
- Destructors should not panic if close was called, but failed. Device should be marked as closed in all cases. - Remove redundant log messages. - Formatting & methods order.
- Destructors should not panic if close was called, but failed. Device should be marked as closed in all cases. - Remove redundant log messages. - Formatting & methods order.
Implement explicit closing of devices & files.
sndio::IDevice is an interface implemented by audio devices and files and network senders and receivers. See doxygen. Currently devices perform closing in their destructor. We need to be able to do it explicitly before calling destructor. This is needed for #677.
Steps:
virtual status::StatusCode close() = 0
to sndio::IDeviceRelated: #703.
The text was updated successfully, but these errors were encountered: