Forum Discussion
mattnewport
10 years agoProtege
OculusRoomTiny (DX11) SDK 0.7 often crashes in Release
The Oculus demo code has been kind of ugly C++ for a while but it's got worse with SDK 0.7. There's some particularly nasty C-style use of goto's for error handling in the MainLoop() function which comes along with some buggy code.
This code at line 175 of main.cpp is disabling a very legitimate warning:
This frequently crashes in a Release build on my machine with an access violation resulting from an aligned SSE move instruction on memory that is not necessarily aligned to 16 bytes - exactly what the disabled warning is complaining about.
This is particularly unfortunate code since there is no good reason for the Camera to be dynamically allocated in the first place.
The demo code is pretty poor quality overall with numerous D3D resource leaks, and questionable C++ style. Are there plans to improve it before the 1.0 release?
This code at line 175 of main.cpp is disabling a very legitimate warning:
// Create camera
__pragma(warning(push))
__pragma(warning(disable:4316)) // Win32: object allocated on the heap may not be aligned 16
mainCam = new Camera(&XMVectorSet(0.0f, 1.6f, 5.0f, 0), &XMQuaternionIdentity());
__pragma(warning(pop))
This frequently crashes in a Release build on my machine with an access violation resulting from an aligned SSE move instruction on memory that is not necessarily aligned to 16 bytes - exactly what the disabled warning is complaining about.
This is particularly unfortunate code since there is no good reason for the Camera to be dynamically allocated in the first place.
The demo code is pretty poor quality overall with numerous D3D resource leaks, and questionable C++ style. Are there plans to improve it before the 1.0 release?
1 Reply
- cyberealityGrand ChampionLet me see if anyone here knows about this.
Quick Links
- Horizon Developer Support
- Quest User Forums
- Troubleshooting Forum for problems with a game or app
- Quest Support for problems with your device
Other Meta Support
Related Content
- 4 months ago
- 2 years ago
- 1 month ago