Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix dispatch_get_current_queue crash (Issue #108) #121

Merged
merged 2 commits into from
Nov 1, 2013

Conversation

kolyuchiy
Copy link
Contributor

No description provided.

@kolyuchiy
Copy link
Contributor Author

Actually the next SDK might have a better solution but I'm not sure if it's OK to publish it now because of NDAs and stuff.

@shortstuffsushi
Copy link

It looks like this hasn't been resolved yet, so I'm going to ask if this could be the cause of the issue I'm seeing here.

My simple sample is in a brand new, single view project, in the ViewController class:

- (void)viewDidLoad {
    [super viewDidLoad];

    dispatch_queue_t newImages = dispatch_queue_create("FOO", NULL);
    dispatch_async(newImages, ^{
        DDLogError(@"This is a message");
    });
}

@shortstuffsushi
Copy link

Oh, it appears my issue was addressed by a related issue, but that also remains open. Is this issue a duplicate of that?

@rivera-ernesto
Copy link
Member

Would you mind updating this @kolyuchiy?

@bpoplauschi
Copy link
Member

From my POV, the pull request looks fine. I'm merging, since several people are waiting for this.

bpoplauschi added a commit that referenced this pull request Nov 1, 2013
Fix dispatch_get_current_queue crash (Issue #108)
@bpoplauschi bpoplauschi merged commit e997ea8 into CocoaLumberjack:master Nov 1, 2013
rivera-ernesto pushed a commit to rivera-ernesto/CocoaLumberjack that referenced this pull request Nov 8, 2013
No longer necessary after pull requests CocoaLumberjack#121 and CocoaLumberjack#156, and more importantly, that was ugly.
@rivera-ernesto rivera-ernesto mentioned this pull request Nov 8, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants