Re: [PATCH v3] json_writer: new routines to create data in JSON format

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 3/23/2018 1:18 PM, Jonathan Nieder wrote:
git@xxxxxxxxxxxxxxxxx wrote:

From: Jeff Hostetler <jeffhost@xxxxxxxxxxxxx>

Add basic routines to generate data in JSON format.

Signed-off-by: Jeff Hostetler <jeffhost@xxxxxxxxxxxxx>

If I understand the cover letter correctly, this is a JSON-like
format, not actual JSON.  Am I understanding correctly?

What are the requirements for consuming this output?  Will a typical
JSON library be able to handle it without trouble?  If not, is there
some subset of cases where a typical JSON library is able to handle it
without trouble?

I'll add text to the commit message and the .h file explaining
the Unicode limitation in the next reroll.


Can you say a little about the motivation here?  (I know you already
put some of that in the cover letter, but since that doesn't become
part of permanent history, it doesn't help the audience that matters.)

I'll add a note about that to the commit message.
Thanks.


This would also be easier to review if there were an application of it
in the same series.  It's fine to send an RFC like this without such
an application, but I think we should hold off on merging it until we
have one.  Having an application makes review much easier since we can
see how the API works in practice, how well the approach fits what
users need, etc.

That's fine.  I'm planning to push up another patch series next week
that builds upon this, so hopefully that will help.


Thanks and hope that helps,
Jonathan

Thanks,
Jeff




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux