Skip to content

Conversation

@larrylegend
Copy link

Thanks for the excellent library! cc @AntonTheDev

@larrylegend
Copy link
Author

Haha...goofed up on the first commit, you can just take the second one :)

…an NSString containing the 6-character string "<null>".
@marcammann
Copy link
Owner

Hey Larry!

Sorry, I just saw this pull request. I commented on you 2nd commit to clear up the indents, but looks good otherwise.

I'm slightly concerned that switching from signed to unsigned might break something somewhere for someone, but it's a countMapper after all, which really shouldn't ever be negative.
Approved as well.

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.

2 participants