Skip to content
This repository has been archived by the owner on Dec 4, 2018. It is now read-only.

Fix retain cycle in -oct_parsedResults #143

Closed
wants to merge 1 commit into from

Conversation

jspahrsummers
Copy link
Collaborator

Because NSAssert is a huge trollface. Hopefully fixes #142.

@jacksonh Can you give this branch a try?

Because `NSAssert` is a huge trollface. Hopefully fixes #142.
@dannygreg
Copy link

lolobjc.

@dannygreg
Copy link

lol

@jacksonh
Copy link
Contributor

jacksonh commented Jan 9, 2014

Here is an updated trace: http://hsc.io/octophile02.trace.zip results look similar but I didn't dig in too much.

@jspahrsummers
Copy link
Collaborator Author

As far as I can tell, it seems to be saying that the leak is originating within this -map: block.

Can you try removing the assertion there, just out of curiosity? It might be an ARC bug or another weird thing with NSAssert.

@jacksonh
Copy link
Contributor

Sorry for the delay. Here is an updated trace with that assert removed.

http://hsc.io/octophile-03.trace.zip

@jspahrsummers
Copy link
Collaborator Author

I'm out of ideas here.

@octokit/octokit-objc Anyone else wanna take a look?

@joshaber
Copy link
Contributor

Looks like the original problem is with how CocoaPods configured OctoKit.

This shouldn't be necessary.

@joshaber joshaber closed this Jan 21, 2014
@joshaber joshaber deleted the enable-hairy-octo-tyrions branch January 21, 2014 21:34
@jacksonh
Copy link
Contributor

@joshaber can you expand on the config issue? I can try fixing the Podspec.

I'll also try building the app with the non-CocoaPods version and see if I can duplicate.

@joshaber
Copy link
Contributor

@jacksonh I wrote a bit more in #142.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ARC not enabled in CocoaPods Podspec
4 participants