Comments on: BSJSONAdditions 1.2

By: DavidPhillipOster

DavidPhillipOster — Wed, 23 Apr 2008 16:20:32 +0000

Found a bug in it though. This showed up in our unit tests, when passing the empty string:

Posting the update here:

In:

- (BOOL)scanJSONValue:(id *)value
{
BOOL result = NO;

[self scanJSONWhiteSpace];

add:
if ([self isAtEnd]) {
return result;
}

By: Matt Pat

Matt Pat — Mon, 08 Oct 2007 03:33:29 +0000

Hey Blake, great software! Found a bug in it though, which I emailed you about, but in case the email doesn’t get to you I thought I’d post it here so people can fix it themselves if they so wish.

On line 160 of NSScanner BSJSONAdditions.m, the hex character set only includes numbers and uppercase letters– if you’re having problems with Unicode characters not being properly recognized, just add the lowercase “abcdef” to the end of the character set. Problem solved.

Thanks again for some awesome software.

By: David

David — Sat, 28 Jul 2007 23:19:58 +0000

Great library Blake. I found this to be incredibly useful. Just wanted to drop by and say thanks.

By: blake

blake — Thu, 24 May 2007 16:22:19 +0000

Some of the older code isn’t tagged with a license, but consider it an MIT-style license.

By: Ian

Ian — Thu, 24 May 2007 02:53:53 +0000

Under what license are you releasing this? I see no mention in any of your files.

Thanks!