-
-
Notifications
You must be signed in to change notification settings - Fork 34
Conversation
…work with guid reference added guid references to all instanced classes
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.
Can we look to provide native GUID functions for areas where we already have GUID references and avoid needless string conversions.
Also I note you have removed some base constructor references (:Base()) and was curious why
XRTK-Core/Packages/com.xrtk.core/Editor/PropertyDrawers/PlatformEntryPropertyDrawer.cs
Outdated
Show resolved
Hide resolved
Mainly I'm trying to keep backwards compatibility with any references to |
They were redundant |
* fixed controller popout window and type reference property drawer to work with guid reference added guid references to all instanced classes * PR Review Changes * removed extra semicolon * removed unneeded local var * no need for ? * removed unused members * use const value * removed unused reference * fixed NRE when attempting to render mapping profile buttons
XRTK - Mixed Reality Toolkit Change Request
Overview
Previously whenever we refactored any class we were simply serializing the
AssemblyQualifiedName
as a string, which could lead to types being unable to be found after we either rename the class or change the namespace.This change is backwards compatible to what we had previously.
The two classes to review:
SystemType
TypeExtensions
Submodule Changes