chris/Cog
2
1
Fork 0
Code Issues 65 Pull requests Projects Releases Packages Wiki Activity Actions

Bug Fix: Operation blocks should queue inputs

Browse source 
Operation blocks cannot expect their out of scope variables to be
present when the block executes, so design the block operation to pull
inputs from a queue array one at a time, like the rest of the blocks do.

Signed-off-by: Christopher Snowhill <kode54@gmail.com>
This commit is contained in:
Christopher Snowhill 2025-03-09 15:05:03 -07:00
parent 14a8a35dac
commit 3874d65ec2
1 changed files with 15 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
Playlist/PlaylistLoader.m
View file

@ -908,12 +908,19 @@ NSURL *_Nullable urlForPath(NSString *_Nullable path);
__block id<SentrySpan> mainTask = [SentrySDK startTransactionWithName:@"Loading tags" operation:@"Main tag operation"];
{
for(NSString *key in queueThisJob) {
__block NSLock *blockLock = [[NSLock alloc] init];
__block NSMutableArray *blockInputs = [queueThisJob mutableCopy];
for(size_t i = 0, j = [blockInputs count]; i < j; ++i) {
NSBlockOperation *op = [[NSBlockOperation alloc] init];
NSURL *url = urlForPath(key);
[op addExecutionBlock:^{
@autoreleasepool {
[blockLock lock];
NSString *key = [blockInputs objectAtIndex:0];
[blockInputs removeObjectAtIndex:0];
[blockLock unlock];
NSURL *url = urlForPath(key);
NSString *message = [NSString stringWithFormat:@"Loading metadata for %@", url];
DLog(@"%@", message);
id<SentrySpan> childTask = nil;
@ -947,7 +954,11 @@ NSURL *_Nullable urlForPath(NSString *_Nullable path);
}
@catch(NSException *e) {
DLog(@"Exception thrown while reading tags: %@", e);
if(e) {
[SentrySDK captureException:e];
} else {
[SentrySDK captureMessage:[NSString stringWithFormat:@"Null exception caught while reading tags for URL: %@", url]];
}
if(childTask) {
[childTask finishWithStatus:kSentrySpanStatusInternalError];
}