mirror of
https://github.com/samsonjs/SJSAssetExportSession.git
synced 2026-04-27 14:57:46 +00:00
Finish writing comment and add FIXME for macOS runtime error
This commit is contained in:
parent
79370a0e54
commit
66700260fb
1 changed files with 3 additions and 1 deletions
|
|
@ -21,7 +21,8 @@ actor SampleWriter {
|
||||||
)
|
)
|
||||||
|
|
||||||
// Execute this actor on the same queue we use to request media data so we can use
|
// Execute this actor on the same queue we use to request media data so we can use
|
||||||
// `assumeIsolated` to ensure that we
|
// `assumeIsolated` to ensure that we serialize access to our state without creating
|
||||||
|
// tasks and doing lots of needless context-switching.
|
||||||
public nonisolated var unownedExecutor: UnownedSerialExecutor {
|
public nonisolated var unownedExecutor: UnownedSerialExecutor {
|
||||||
queue.asUnownedSerialExecutor()
|
queue.asUnownedSerialExecutor()
|
||||||
}
|
}
|
||||||
|
|
@ -228,6 +229,7 @@ actor SampleWriter {
|
||||||
return await withTaskCancellationHandler {
|
return await withTaskCancellationHandler {
|
||||||
await withCheckedContinuation { continuation in
|
await withCheckedContinuation { continuation in
|
||||||
self.videoInput!.requestMediaDataWhenReady(on: queue) {
|
self.videoInput!.requestMediaDataWhenReady(on: queue) {
|
||||||
|
#warning("FIXME: why is this broken on macOS?!")
|
||||||
self.assumeIsolated { _self in
|
self.assumeIsolated { _self in
|
||||||
guard !_self.isCancelled else {
|
guard !_self.isCancelled else {
|
||||||
log.debug("Cancelled while encoding video")
|
log.debug("Cancelled while encoding video")
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue