-
Notifications
You must be signed in to change notification settings - Fork 261
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
hello_xr: Fix a warning #26
Merged
daveh-lunarg
merged 1 commit into
KhronosGroup:master
from
rpavlik:helloxr_pessimization
Apr 2, 2019
Merged
hello_xr: Fix a warning #26
daveh-lunarg
merged 1 commit into
KhronosGroup:master
from
rpavlik:helloxr_pessimization
Apr 2, 2019
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
daveh-lunarg
approved these changes
Apr 2, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure.
rpavlik
added a commit
that referenced
this pull request
May 9, 2019
No API changes, and only minimal consistency changes to the spec/registry. Mostly an update for tooling, layers, loader, and sample code. Header version has been bumped to 43, but no symbols that should have actually been in use have changed. ### GitHub Pull Requests These had been integrated into the public repo incrementally. - General, Build, Other - #8, #11, #12 - Improve BUILDING and README - #9 - Make Vulkan SDK dependency optional - #17 - Add install target to CMake files - #17 - API dump layer, build: timespec extension fixes - #19 - build: fix CMAKE_PRESENTATION_BACKEND default on linux - #34 - list: Fix list test output - validation layer - #18, #22, #23 - Fix build and execution - #24 - Fix crash and refactor - hello_xr - #13 - Do not query GL context API version before creating context - #26 - Fix a warning - Loader - #3 - Don't cross 32/64 registry silos - #14 - Initialize XrExtensionProperties array parameter for rt_xrEnumerateInstanceExtensionProperties - #20 - Fix Linux manifest file search - #30 - Add default implementations of API functions to dispatch chains - #32 - Avoid crash when evaluating layer disable environment vars - #35 - Add 'unknown' strings to loader's *ToString fallback functions - #36 - Allow null instance in xrGetInstanceProcAddr() for certain entry points - #39 - Default to static loader only on Windows ### Internal Issues - General, Build, Other - Unify (for the most part) the OpenXR and Vulkan generator scripts. (internal MR 1166) - List instance extensions in the "list" test. (internal MR 1169) - Avoid dllexport for all apps compiled with `openxr_platform_defines.h` (internal MR 1187) - Don't offer `BUILD_SPECIFICATION` unless the spec makefile is there. (internal MR 1179) - Add simple input example to hello_xr. (internal MR 1178) - Add a clang-format script for ease of development. - API Registry and Headers - Remove impossible and undocumented error codes. (internal MR 1185 and 1189) - Mark layers in `XrFrameEndInfo` as optional. (internal MR 1151, internal issue 899) - Remove unused windows types from `openxr_platform.h` (internal MR 1197) - Make `openxr_platform.h` include `openxr.h` on which it depends. (internal MR 1140, internal issue 918) - Remove unused, undocumented defines. (internal MR 1238, internal issue 1012) - Loader - Fix loader regkey search logic so 64bit application loads 64bit regkey value. (internal MR 1180) - Modify loader to be friendly to UWP (Universal Windows Platform) build target. (internal MR 1198)
rpavlik
added a commit
that referenced
this pull request
Aug 3, 2019
Patch release for the 1.0 series. ### GitHub Pull Requests These had been integrated into the public repo incrementally. - General, Build, Other - #87 - Fix makefiles - #88 - Remove unneeded generation (corresponds to issue #74, internal issue 1139, internal MR 1491) - #101 - Fix install of header and loader. - Loader - #91 - Fix a loader bug which prevented Layers from not implementing all XR functions - #95 - Guard config includes/defines (relates to #81, #92) - #97 - Remove a constant static std::vector, use a std::array instead. - Layers - #84 - Fix Linux warning for apidump - From OpenXR-Docs: - #26 - Proposal for unbounded space and spatial anchor extensions (vendor extensions) ### Internal issues - General, Build, Other - Makefile cleanups (internal MR 1469, 1489) - Add release scripts (internal MR 1496) - Registry - Reserve Oculus extension numbers (internal MR 1493) - Add Monado headless (vendor extension) (internal MR 1482) - Loader - Remove unnecessary `#ifdef _WIN32` in loader. (internal MR 1487) ### New extensions - `XR_MND_headless` - `XR_MSFT_spatial_anchor` - `XR_MSFT_unbounded_reference_space`
rpavlik
added a commit
that referenced
this pull request
Aug 27, 2019
Patch release for the 1.0 series. Note that the loader on Windows has a **security fix**: All developers incorporating the OpenXR loader should update immediately. ### GitHub Pull Requests These had been integrated into the public repo incrementally. - General, Build, Other - #112 - Update active runtime search documentation - #106 - List app changes - #114 - Support for building WindowsStore loader and layers, and simplified filename - #96 - Misc cleanup: build simplification, install hello_xr, allow building as subproject, fix null deref in validation layer. - Loader - #102 - Default to catching exceptions, since not being able to catch (and having a non-throwing standard library) is less common - #109 - Factor out some debug-utils related code from the loader, and migrate validation layer to that shared code. - #108 - Update json_stream initialization to improve compatibility - #118 - Fix logic error in Linux active runtime search - #115, #117 - Simplification and refactoring. - Layers - #111 - Some fixes to Validation Layer (as found applying to the UE4 OpenXR plugin) - #110 - Fix cleaning up session labels in validation layer - From OpenXR-Docs: - #26 - Proposal for unbounded space and spatial anchor extensions (vendor extensions) ### Internal issues - General, Build, Other - Allow project to be included in a parent project. (Internal MR 1512) - hello_xr - Fix OpenGL version number to be XrVersion. (Internal MR 1515) - Make D3D11 debug device handling more friendly. (Internal MR 1504) - Registry - Fix error in extension-added function. (Internal MR 1510) - Add Oculus Android extension. (Internal MR 1518) - Reserve additional extension number for Oculus. (Internal MR 1517) - Loader - **Security fix**: Do not use HKEY_CURRENT_USER or environment variables when running as a medium-integrity process or higher on Windows. (Internal issue 1205, internal MR 1511) - Small updates to the loader documentation. ### New extension - `XR_OCULUS_android_session_state_enable`
rhabacker
pushed a commit
to rhabacker/OpenXR-SDK-Source
that referenced
this pull request
Nov 16, 2022
Patch release for the 1.0 series. ### GitHub Pull Requests These had been integrated into the public repo incrementally. - General, Build, Other - KhronosGroup#87 - Fix makefiles - KhronosGroup#88 - Remove unneeded generation (corresponds to issue KhronosGroup#74, internal issue 1139, internal MR 1491) - KhronosGroup#101 - Fix install of header and loader. - Loader - KhronosGroup#91 - Fix a loader bug which prevented Layers from not implementing all XR functions - KhronosGroup#95 - Guard config includes/defines (relates to KhronosGroup#81, KhronosGroup#92) - KhronosGroup#97 - Remove a constant static std::vector, use a std::array instead. - From OpenXR-Docs: - KhronosGroup#26 - Proposal for unbounded space and spatial anchor extensions (vendor extensions) ### Internal issues - General, Build, Other - Makefile cleanups (internal MR 1469, 1489) - Add release scripts (internal MR 1496) - Registry - Reserve Oculus extension numbers (internal MR 1493) - Add Monado headless (vendor extension) (internal MR 1482) - Loader - Remove unnecessary `#ifdef _WIN32` in loader. (internal MR 1487) ### New extensions - `XR_MND_headless` - `XR_MSFT_spatial_anchor` - `XR_MSFT_unbounded_reference_space`
rhabacker
pushed a commit
to rhabacker/OpenXR-SDK-Source
that referenced
this pull request
Nov 16, 2022
Patch release for the 1.0 series. Note that the loader on Windows has a **security fix**: All developers incorporating the OpenXR loader should update immediately. ### GitHub Pull Requests These had been integrated into the public repo incrementally. - General, Build, Other - KhronosGroup#112 - Update active runtime search documentation - KhronosGroup#106 - List app changes - KhronosGroup#114 - Support for building WindowsStore loader and layers, and simplified filename - KhronosGroup#96 - Misc cleanup: build simplification, install hello_xr, allow building as subproject, fix null deref in validation layer. - Loader - KhronosGroup#102 - Default to catching exceptions, since not being able to catch (and having a non-throwing standard library) is less common - KhronosGroup#109 - Factor out some debug-utils related code from the loader, and migrate validation layer to that shared code. - KhronosGroup#108 - Update json_stream initialization to improve compatibility - KhronosGroup#118 - Fix logic error in Linux active runtime search - KhronosGroup#115, KhronosGroup#117 - Simplification and refactoring. - Layers - KhronosGroup#111 - Some fixes to Validation Layer (as found applying to the UE4 OpenXR plugin) - KhronosGroup#110 - Fix cleaning up session labels in validation layer - From OpenXR-Docs: - KhronosGroup#26 - Proposal for unbounded space and spatial anchor extensions (vendor extensions) ### Internal issues - General, Build, Other - Allow project to be included in a parent project. (Internal MR 1512) - hello_xr - Fix OpenGL version number to be XrVersion. (Internal MR 1515) - Make D3D11 debug device handling more friendly. (Internal MR 1504) - Registry - Fix error in extension-added function. (Internal MR 1510) - Add Oculus Android extension. (Internal MR 1518) - Reserve additional extension number for Oculus. (Internal MR 1517) - Loader - **Security fix**: Do not use HKEY_CURRENT_USER or environment variables when running as a medium-integrity process or higher on Windows. (Internal issue 1205, internal MR 1511) - Small updates to the loader documentation. ### New extension - `XR_OCULUS_android_session_state_enable`
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
My version of GCC locally pointed out that moving a return value is actually a pessimization for some reason.