Lion compatibility #40

Closed
wants to merge 11 commits into
from

Conversation

Projects
None yet
2 participants
@andypotion

Hey John,

Now that Lion is out, I'm submitting this merge request again. It's the one that fixes the NSNumber problem because Lion now encodes CFNumbers into pointers in 64-bit systems.

  • Andy

johnezang added a commit that referenced this pull request Sep 23, 2011

Fixes issues #9, #15, #40, aka "crashes on 64-bit Lion / 10.7 ABI".
This commit implements a work around for a bug in 10.7 that was caused by
a 10.7 64-bit ABI breaking change.

Technically, this is not a bug in JSONKit, but with Mac OS X.

When making changes to the ABI, it is (at least de facto) required to bump
the "major version" of a shared library so that code designed around and
built against the "guarantees" provided by previous versions ABI / API
are not violated.

Not only was this not done in 10.7, the ABI breaking change isn't even
officially documented (to the best of my knowledge).  It's certainly not
mentioned in the 10.7 release notes.
@johnezang

This comment has been minimized.

Show comment
Hide comment
@johnezang

johnezang Sep 23, 2011

Owner

Fixed in commit c2ef692.

(note: the committed fix does not use this pull request).

Owner

johnezang commented Sep 23, 2011

Fixed in commit c2ef692.

(note: the committed fix does not use this pull request).

@johnezang johnezang closed this Sep 23, 2011

@ghost ghost assigned johnezang Sep 23, 2011

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