search

OSVR / OSVR-Unity-Rendering

Rendering plugin for Unity
Apache License 2.0
34 stars 13 forks source link

Cleanup and fixes #19

Closed rpavlik closed 8 years ago

rpavlik commented 8 years ago

I didn't clean up everything, but Palace kept working when I dropped in new builds on machines it previously worked on. There are todos marking things that I wouldn't expect to have worked at all, or questions I have, etc. but this should make it easier to maintain and also easier to port to non-windows platforms (lot of things weren't conditionally-compiling)

Also adds two optional, manual defines at the top of the main source file - turn those on, and it will dump out its log messages (and the stdout and stderr of render manager) to a log file.

DuFF14 commented 8 years ago

Looks OK to me, but I'll let @JeroMiya comment on the p/invoke convention. Tested that it works.

DuFF14 commented 8 years ago

Going to merge as this clears a path for some scene-switching fixes, and make a note to review p/invoke convention.