Sent from my iPhone
On 24/08/2012, at 19:49, Sébastien Michel <sebastien.mic...@atos.net> wrote: > 2012/8/24 Greg Banks <g...@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. >> > Indeed. > > 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. > Cool. > 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. Greg.