Futile icon indicating copy to clipboard operation
Futile copied to clipboard

Consider integrating TinyJSON instead of MiniJSON

Open MattRix opened this issue 11 years ago • 3 comments

MiniJSON has been working fine for Futile's json-reading purposes, but I've heard from multiple people about issues with it when writing json. It'd be nice if people could rely on the json lib that comes with Futile and not have to have two of them in the same project. @pbhogan's TinyJson seems like it could be a great replacement candidate.

MattRix avatar Jul 19 '13 03:07 MattRix

I'm biased, of course, but I do think TinyJSON's syntax would simplify things for you. And there'd be someone to send bugs to, since MiniJSON isn't exactly maintained. :)

That said, I'm looking into that encoding issue (I suspect it's related to generic interfaces and AOT compiling on iOS). With the reflection magic TinyJSON uses, it could be affected by it.

pbhogan avatar Jul 19 '13 13:07 pbhogan

I was just also pointed at this updated/fixed version of MiniJSON that I could update Futile with in the meantime: https://gist.github.com/nikibobi/5330464

MattRix avatar Jul 31 '13 17:07 MattRix

Nice. I'm going to check out the diffs and see if anything needs to be pulled into TinyJSON too. It's probably wise to stick with MiniJSON until TinyJSON is more proven. I've added unit tests and fixed a bunch of iOS AOT issues, but the encoding thing is apparently a Mono bug and I don't think it can be reliably reproduced and fixed. You probably saw @prime31 mention it on Twitter.

pbhogan avatar Jul 31 '13 17:07 pbhogan