Merge pull request #10326 from jonathoncobb/jonathoncobb/logging-issues

David Kline · keveleigh · commit c19de9f759aa · 2021-11-17T17:37:12.000-08:00
Fixed log spam that is coming from HandConstraintPalmUp.cs
diff --git a/Assets/MRTK/SDK/Features/Utilities/Solvers/HandConstraintPalmUp.cs b/Assets/MRTK/SDK/Features/Utilities/Solvers/HandConstraintPalmUp.cs
@@ -197,10 +197,6 @@ protected override bool IsValidController(IMixedRealityController controller)
 
                     return palmFacingThresholdMet;
                 }
-                else
-                {
-                    Debug.LogWarning($"HandConstraintPalmUp requires a palm joint, but none was provided by {controller.InputSource.SourceName}.");
-                }
 
                 return palmFacingThresholdMet;
             }

Original file line number	Diff line number	Diff line change
`@@ -197,10 +197,6 @@ protected override bool IsValidController(IMixedRealityController controller)`
`197`	`197`
`198`	`198`	`return palmFacingThresholdMet;`
`199`	`199`	`}`
`200`		`- else`
`201`		`- {`
`202`		`- Debug.LogWarning($"HandConstraintPalmUp requires a palm joint, but none was provided by {controller.InputSource.SourceName}.");`
`203`		`- }`
`204`	`200`
`205`	`201`	`return palmFacingThresholdMet;`
`206`	`202`	`}`