Merge pull request #1472 from finagolfin/droid

ahoppen · web-flow · commit 5a611fb95497 · 2024-06-11T00:34:14.000-07:00
Android: use the right setpriority() signature for Bionic
diff --git a/Sources/SKSupport/Process+Run.swift b/Sources/SKSupport/Process+Run.swift
@@ -127,7 +127,7 @@ private func setProcessPriority(pid: Process.ProcessID, newPriority: TaskPriorit
   if !SetPriorityClass(handle, UInt32(newPriority.windowsProcessPriority)) {
     logger.error("Failed to set process priority of \(pid) to \(newPriority.rawValue): \(GetLastError())")
   }
-  #elseif canImport(Darwin)
+  #elseif canImport(Darwin) || canImport(Android)
   // `setpriority` is only able to decrease a process's priority and cannot elevate it. Since Swift task’s priorities
   // can only be elevated, this means that we can effectively only change a process's priority once, when it is created.
   // All subsequent calls to `setpriority` will fail. Because of this, don't log an error.

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ private func setProcessPriority(pid: Process.ProcessID, newPriority: TaskPriorit`
`127`	`127`	`if !SetPriorityClass(handle, UInt32(newPriority.windowsProcessPriority)) {`
`128`	`128`	`logger.error("Failed to set process priority of \(pid) to \(newPriority.rawValue): \(GetLastError())")`
`129`	`129`	`}`
`130`		`- #elseif canImport(Darwin)`
	`130`	`+ #elseif canImport(Darwin) \|\| canImport(Android)`
`131`	`131`	// `setpriority` is only able to decrease a process's priority and cannot elevate it. Since Swift task’s priorities
`132`	`132`	`// can only be elevated, this means that we can effectively only change a process's priority once, when it is created.`
`133`	`133`	// All subsequent calls to `setpriority` will fail. Because of this, don't log an error.