replaces all SimpleJSONParser calls to ParseJSON

BrightcoveOS / Roku-Sample-App

Sample application for Roku using Brightcove VideoCloud

50 stars 24 forks source link

replaces all SimpleJSONParser calls to ParseJSON #13

Closed sudo97 closed 7 years ago

sudo97 commented 7 years ago

This is simple, but I saw a FIXME comment. Also left one new line in SimpleJSONParser() that just returns ParseJSON(), just in case.

sudo97 commented 7 years ago

I'm sorry, second commit was wrong

adiffmike commented 7 years ago

Just a heads up on this.

I don't have the bandwidth to test these commits - and haven't touched this in years - but if there's a native parser now for JSON thats an obvious enhancement and the code changes look solid so I'm going to merge this and cross my fingers a bit....

sudo97 commented 7 years ago

Is this project still being developed? Or is it freezed?

adiffmike commented 7 years ago

Its not being actively developed. I'll check in on it every so often if someone has a pull request etc. Its a bit long in the tooth at this point.