FIX: ISXB-687 Added guard clause to RemovePointerAtIndex in InputSy…

Packages/com.unity.inputsystem/CHANGELOG.md

@@ -11,6 +11,7 @@ however, it has to be formatted properly to pass verification tests.
 ## [Unreleased] - yyyy-mm-dd
 
 ### Fixed
+- Fixed touch pointers being released twice causing an index out of bounds error. [ISXB-687](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-687)
 - Fixed Multiple interactions could breaks on Composite Binding. [ISXB-619](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-619)
 - Fixed memory leak when the OnScreenStick component was destroyed [ISXB-1070](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-1070). Contribution by [LukeUnityDev](https://github.com/LukeUnityDev).
 - Fixed Action Maps contextual menu in Action Editor UI that occasionally displays unrelated items.
@@ -28,6 +29,9 @@ however, it has to be formatted properly to pass verification tests.
 ### Fixed
 - Fixed memory allocation on every frame when using UIDocument without EventSystem. [ISXB-953](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-953)
 - Fixed Action Maps name edition which could be inconsistent in Input Action Editor UI.
+
+### Added
+- Added Hinge Angle sensor support for foldable devices.
 - Fixed InputDeviceTester sample only visualizing a given touch contact once. [ISXB-1017](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-1017)
 - Fixed an update loop in the asset editor that occurs when selecting an Action Map that has no Actions.
 - Fixed Package compilation when Unity Analytics module is not enabled on 2022.3. [ISXB-996](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-996)

Packages/com.unity.inputsystem/InputSystem/Plugins/UI/InputSystemUIInputModule.cs

@@ -1947,6 +1947,13 @@ private void RemovePointerAtIndex(int index)
         {
             Debug.Assert(m_PointerStates[index].eventData.pointerEnter == null, "Pointer should have exited all objects before being removed");
 
+            // We don't want to release touch pointers on the same frame they are released (unpressed). They get cleaned up one frame later in Process()
+            ref var state = ref GetPointerStateForIndex(index);
+            if (state.pointerType == UIPointerType.Touch && (state.leftButton.isPressed || state.leftButton.wasReleasedThisFrame))
+            {
+                return;
+            }
+
             // Retain event data so that we can reuse the event the next time we allocate a PointerModel record.
             var eventData = m_PointerStates[index].eventData;
             Debug.Assert(eventData != null, "Pointer state should have an event instance!");
@@ -2209,7 +2216,13 @@ private void FilterPointerStatesByType()
                     // We have input on a mouse or pen. Kill all touch and tracked pointers we may have.
                     for (var i = 0; i < m_PointerStates.length; ++i)
                     {
-                        if (m_PointerStates[i].pointerType != UIPointerType.MouseOrPen)
+                        ref var state = ref GetPointerStateForIndex(i);
+                        // Touch pointers need to get forced to no longer be pressed otherwise they will not get released in subsequent frames.
+                        if (m_PointerStates[i].pointerType == UIPointerType.Touch)
+                        {
+                            state.leftButton.isPressed = false;
+                        }
+                        if (m_PointerStates[i].pointerType != UIPointerType.MouseOrPen && m_PointerStates[i].pointerType != UIPointerType.Touch || (m_PointerStates[i].pointerType == UIPointerType.Touch && !state.leftButton.isPressed && !state.leftButton.wasReleasedThisFrame))
                         {
                             SendPointerExitEventsAndRemovePointer(i);
                             --i;
@@ -2221,6 +2234,8 @@ private void FilterPointerStatesByType()
                     // We have touch or tracked input. Kill mouse/pen pointer, if we have it.
                     for (var i = 0; i < m_PointerStates.length; ++i)
                     {
+                        ref var state = ref GetPointerStateForIndex(i);
+                        // Touch pointers need to get forced to no longer be pressed otherwise they will not get released in subsequent frames.
                         if (m_PointerStates[i].pointerType == UIPointerType.MouseOrPen)
                         {
                             SendPointerExitEventsAndRemovePointer(i);