OSVR / OSVR-Core

The core libraries, applications, and plugins of the OSVR software platform.
Apache License 2.0
329 stars 124 forks source link

Android: auto-start API crash when server already running #548

Open JeroMiya opened 7 years ago

JeroMiya commented 7 years ago

Steps to reproduce:

  1. Run osvr_server from adb or OSVRServerLauncher app.
  2. Run the OSVROpenGL sample app, which uses the auto-start API to start in-process server.
  3. Note the exception escapes the C function.

Expected behavior: C function should log the failure (already does, with the right message) but otherwise fail silently. ClientKit will then connect to the already running server. Additionally, the shutdown function should not crash either (not tested prior to filing this issue, but test before closing this issue).

We probably are just missing an exception handler in the C API function. Should be no need to log the exception since the underlying implementation already logs the failure more specifically.