Fix error in SynchronizationStrategy.PlayerLoop

Assets/Unio/Internal/FileReadPromise.cs

@@ -107,11 +107,6 @@ public void WaitForComplete()
                 }
             }
 
-            if (!MoveNext())
-            {
-                throw new InvalidOperationException($"ReadHandle is completed but ReadHandle is not completed. {filePath} {status}");
-            }
-
             if (status != ReadStatus.Complete)
             {
                 throw new UnioIOException(filePath, status);
@@ -131,13 +126,13 @@ public bool MoveNext()
                         readHandle.Cancel();
                     }
                     completionSource?.TrySetCanceled();
+                    return false;
                 }
                 finally
                 {
                     ReleaseHandle();
                     ReleaseBuffer();
                 }
-                return false;
             }
 
             switch (readHandle.Status)
@@ -150,27 +145,28 @@ public bool MoveNext()
                         var result = GetResult();
                         completionSource.TrySetResult(result);
                     }
-                    break;
+                    return false;
                 }
                 case ReadStatus.Failed:
                 case ReadStatus.Truncated:
                 {
                     ReleaseHandle();
                     ReleaseBuffer();
                     completionSource?.TrySetException(new UnioIOException(filePath, readHandle.Status));
-                    break;
+                    return false;
                 }
                 case ReadStatus.Canceled:
                 {
                     ReleaseHandle();
                     ReleaseBuffer();
                     completionSource?.TrySetCanceled();
-                    break;
+                    return false;
                 }
+                case ReadStatus.InProgress:
+                    return true;
                 default:
                     throw new ArgumentOutOfRangeException();
             }
-            return true;
         }
 
         void ReleaseHandle()

Assets/Unio/Internal/PlayerLoopHelper.cs

@@ -95,6 +95,7 @@ static void InsertSubsystem(ref PlayerLoopSystem parentSystem, Type beforeType,
                 if (source[i].type == beforeType)
                 {
                     insertIndex = i;
+                    break;
                 }
             }
 
@@ -103,7 +104,7 @@ static void InsertSubsystem(ref PlayerLoopSystem parentSystem, Type beforeType,
                 throw new ArgumentException($"{beforeType.FullName} not in system {parentSystem} {parentSystem.type.FullName}");
             }
 
-            var dest = new PlayerLoopSystem[source.Length + 2];
+            var dest = new PlayerLoopSystem[source.Length + 1];
             for (var i = 0; i < dest.Length; i++)
             {
                 if (i == insertIndex)

Assets/Unio/NativeFile.cs

@@ -66,7 +66,6 @@ public static BytesTaskType ReadAllBytesAsync(string filePath, SynchronizationSt
                     break;
                 case SynchronizationStrategy.PlayerLoop:
                 {
-                    PlayerLoopHelper.EnsureInitialized();
                     PlayerLoopHelper.Dispatch(PlayerLoopTiming.Update, promise);
                     break;
                 }