Replace 'int' with NSInteger #7

Merged
merged 1 commit into from May 9, 2012

Conversation

Projects
None yet
2 participants
Owner

pjrobertson commented May 9, 2012

These changes remove a deprecated method and use NSIntegers as opposed to int. Should allow for 64 bit compilation

Owner

pjrobertson commented May 9, 2012

Oh, and I've added a test to make sure that an object is not nil when adding it to the shelf. It's what was causing the problem at 792

Owner

skurfer commented May 9, 2012

Everything looks fine with this, so I’ll merge it before releasing a new plug-in.

But it still won’t build as 64-bit (unless /tmp/QS is 64-bit). :-(

@skurfer skurfer added a commit that referenced this pull request May 9, 2012

@skurfer skurfer Merge pull request #7 from pjrobertson/master
Replace 'int' with NSInteger
d700d31

@skurfer skurfer merged commit d700d31 into quicksilver:master May 9, 2012

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