ebkalderon / renderdoc-rs

RenderDoc application bindings for Rust
Apache License 2.0
45 stars 6 forks source link

Fix semantics of shutdown(), remove remove_hooks() for now #92

Closed ebkalderon closed 4 years ago

ebkalderon commented 4 years ago

Changed

Removed

The previous changes from #91, moving shutdown() to the specific RenderDoc<V100> impl block and the addition of remove_hooks() to RenderDoc<V141>, won't actually work as intended in practice.

This is because the existing Deref and DerefMut implementations are not sufficient for either method to work properly. This would require the introduction of a DerefMove trait in the standard library to work (see https://github.com/rust-lang/rfcs/issues/997 for details).