AbstractJobConfig - Add verbose message comment

Depends on Future feature: decline-cookies/anvil-unity-core#112
decline-cookies · Apr 13, 2023 · 1a2e3d5 · 1a2e3d5
1 parent 500181c
commit 1a2e3d5
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/Scripts/Runtime/Entities/TaskDriver/TaskSet/Job/JobConfig/AbstractJobConfig.cs b/Scripts/Runtime/Entities/TaskDriver/TaskSet/Job/JobConfig/AbstractJobConfig.cs
@@ -139,6 +139,10 @@ private void ResolveDuplicateAccessWrappers(AbstractAccessWrapper accessWrapper)
             {
                 throw new Exception($"There is no compatibility between ${nameof(AccessType)} requires on the same type. See previous message for details.");
             }
+
+            //TODO: #112(anvil-unity-core) - Emit as a verbose warning
+            // Logger.Warning($"Duplicate access requires resolved to {m_AccessWrappers[accessWrapper.ID].AccessType}. Existing:{existingAccessWrapper.AccessType}, Incoming:{accessWrapper.AccessType}"
+            //     + $"\nThis is not necessarily a problem but could degrade scheduling performance less restrictive access was expected.");
         }