benvanik / vr.js

NPAPI plugin to expose fun VR devices to Javascript.
Apache License 2.0
498 stars 105 forks source link

Cleanup OVR shutdown. #2

Open benvanik opened 11 years ago

benvanik commented 11 years ago

The call to OVR::System::Destroy(); in ovr_manager is currently commented out as it causes a hang. The problem seems to be that the Thread::FinishAllThreads() call deadlocks waiting for threads that are never properly cleaned up by the SDK (or Firefox kills them before this code executes).

It'd be nice to clean this up so that shutdown was clean in case the SDK starts requiring it.