commit "mboxevent: Rewrite JSON formatting"
gnb at fastmail.fm
Fri Aug 24 16:23:41 EDT 2012
Sent from my iPhone
On 24/08/2012, at 19:49, Sébastien Michel <sebastien.michel at atos.net> wrote:
> 2012/8/24 Greg Banks <gnb at fastmail.fm>:
>>> If nobody else expresses opinion, whether to put xjson in libcyrus or libcyrus_imap, it is up to you. I just told you my opinion.
>> My 2c: JSON code is a utility and should go in libcyrus.
> As discussed on IRC, we decided to change the library to format to
> JSON, from libjson (that will support 64bit integer only in the next
> release) to jansson that is also a mature library, available on major
> Linux distro and already support 64bit.
> The last debate is on the bugzilla ticket #3605. It is about removing
> the internal xjson.[ch] json formatter and add a default option
> --disable-event-notification or --enable-event-notification and force
> requirement on jansson library if enabled.
> That involves decorating the code with a C macro in all source files
> that refer to mboxevent.h.
Personally I'd be happier with only one set of code to test.
More information about the Cyrus-devel