Zulu-Inuoe/jzon

[FR] Add to `quicklisp`

NightMachinery opened this issue · 18 comments

[FR] Add to `quicklisp`

Hi @NightMachinary . I appreciate the support, unfortunately I'd like to avoid putting this on Quicklisp until it's more complete. At a minimum I wish to have:

  1. A roundtripping reader/writer (output JSON exactly as it was read)
  2. Object coercion mechanism for reading (eg read JSON as a CL class/structure)

Until these are done, I don't want to pollute QL with yet another JSON library.

+1 for adding to Quicklisp.
IMHO, jzon is already the best JSON library for rendering JSON views in web app.
So, I'd like to switch from Jonathan in my libraries, but can't because jzon is not in Quicklisp.

@fukamachi Thanks for the support.
Unfortunately I haven't been doing much programming in my free time as of late.
But if you think the library is useful as-is, then it's fair to add it to quicklisp, and I can add my other desired features over time as I can.

I just don't want to continue to contribute to the 100 JSON library meme on Quicklisp unless I offer some substantial benefit

@Zulu-Inuoe I think that jzon is a new generation JSON parser taking into account advantages and disadvantages of previous generation of parsers. And it has full coverage of JSON standard verified by third-party test suite. This is already good reasons for releasing v1 on quicklisp.

Yes, can you please add this to quicklisp, the pretty print feature is exceptionally useful :)

Throwing my hat onto the pile. I'd like to start using this.

phoe commented

Pretty please?

Oh my gosh. Let me see what I can do this weekend and I'll add v1 on Monday no matter what

haha jk did I say that weekend? I meant this one.

In all seriousness I've been doing some work and thinking over the week and will definitely have something solid here soon.
Thanks all

phoe commented

fbn7krvnq8381

fbn7krvnq8381

Unfortunately I am a busy tortoise :(

phoe commented

Do you need any support with that - as in, what would happen to me if I did things the naïve way and made a quicklisp-projects issue pointing to the current master branch?

I honestly don't remember the status of what things are WIP. I believe there's still a couple of issues with stringify, including the floating point printer, the string replacer, and the auto-coercion stuff in general.
As before it's ultimately: I still don't know if it offers anything truly above and beyond the other json libs on QL already.

The other issue is that if we make available a beta version as-is, it'll be hard to fix things without breaking users - the auto-coercion stuff especially scares me.

Alright this is actually finally happening. All my remaining stuff for v1 are resolved and I've decided what to leave out for a v2.

Code-wise, the true final final stuff is whether or not I keep the alist/plist heuristics in write-value which I am awaiting review.
The other changes are things I can do without breaking backwards-compat, like updating docs and optimizing my string-on-string parser

Great, thank you!

phoe commented

Added and available; huge thanks! I think this issue can now be closed.

Thanks for the heads up, @phoe