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

Changes to StreamingParser to work under latest llvm & ASLResponse NSEnumerator changes #15

Open
wants to merge 19 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@aufflick

aufflick commented Jul 31, 2012

Brought in Gzip header changes from ParserExample and reduced complier warnings & depreacated GC related errors. Still get odd warnings about the weak CFNetwork symbols not being declared - perhaps because they are #defines?

@aufflick

This comment has been minimized.

Show comment
Hide comment
@aufflick

aufflick Jul 31, 2012

Just discovered those odd warnings are fixed by pull request 11: #11

I have manually added that change to my fork, but haven't made another pull request since you can just pull request 11 (NB: haven't tested pull 11 under MacOS).

aufflick commented Jul 31, 2012

Just discovered those odd warnings are fixed by pull request 11: #11

I have manually added that change to my fork, but haven't made another pull request since you can just pull request 11 (NB: haven't tested pull 11 under MacOS).

@aufflick

This comment has been minimized.

Show comment
Hide comment
@aufflick

aufflick Aug 2, 2012

Hm, I misunderstood pull requests - seems to auto-add any commits I make. The prior two commits are for the OAuth support I wrote about here: http://mark.aufflick.com/blog/2012/08/02/aqxmlparser-oauth

aufflick commented Aug 2, 2012

Hm, I misunderstood pull requests - seems to auto-add any commits I make. The prior two commits are for the OAuth support I wrote about here: http://mark.aufflick.com/blog/2012/08/02/aqxmlparser-oauth

@aufflick aufflick changed the title from Changes to work under latest llvm to Changes to StreamingParser to work under latest llvm & ASLResponse NSEnumerator changes Oct 3, 2014

@aufflick

This comment has been minimized.

Show comment
Hide comment
@aufflick

aufflick Oct 3, 2014

Dang, I forgot this pull request was open, I should have made different branches for each of the changes…

aufflick commented Oct 3, 2014

Dang, I forgot this pull request was open, I should have made different branches for each of the changes…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment