-
-
Notifications
You must be signed in to change notification settings - Fork 34
Conversation
… profiles, that should be handled in the constructor of the respective data provider with a thrown exception if missing. - Moved the type null checks until after the platform check to better support false positive errors thrown when that type/assembly is probably missing because we're targeting a different platform. - updated typeRefPropertyDrawer to not throw an error if the type is missing because we're targeting a different platform.
@@ -330,11 +328,11 @@ private static Type[] FindTypesByName(string typeName, SystemTypeAttribute filte | |||
var types = new List<Type>(); | |||
var filteredTypes = GetFilteredTypes(filter); | |||
|
|||
for (var i = 0; i < filteredTypes.Count; i++) | |||
foreach (var type in filteredTypes) |
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.
Why change a performant FOR loop for a more expensive ForEach?
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.
I'm not worried about the for each loops in editor code.
{ | ||
string menuLabel = FormatGroupedTypeName(types[i], grouping); | ||
var menuLabel = FormatGroupedTypeName(type, grouping); |
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.
Why the sudden change of explicit types to using var, we already know the format and type for each of these.
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.
I think it's pretty obvious that it's a string from the context.
… profiles, that should be handled in the constructor of the respective data provider with a thrown exception if missing. (#120) - Moved the type null checks until after the platform check to better support false positive errors thrown when that type/assembly is probably missing because we're targeting a different platform. - updated typeRefPropertyDrawer to not throw an error if the type is missing because we're targeting a different platform.
Removed the profile check for the spatial awareness data provider's profiles, that should be handled in the constructor of the respective data provider with a thrown exception if missing.
Moved the type null checks until after the platform check to better support false positive errors thrown when that type/assembly is probably missing because we're targeting a different platform.
updated
TypeRefPropertyDrawer
to not throw an error if the type is missing because we're targeting a different platform.