Hi List, I'm in the process of fixing/updating/propagating the debug system in openchange (libmapi and tools).
Most of our libmapi functions still use MAPI_RETVAL_IF macro which returns -1 while our specs define that functions are returning a MAPI error. This got fixed by Brad in a previous commit -> introduction of OPENCHANGE_RETVAL_IF. I'll now propagate this change to the whole openchange framework. This should make GetLastError() obsolete, but we'll keep it for backward compatibility reasons[*]. Furthermore, we have a segfault with latest samba4+openchange couple when we run tools with -d (--debuglevel). I have found a quick fix, but will instead update openchange logging system so we can either save output within a file, stdout or stderr. I'll also introduce some debug assessors in libmapi like the one I have been posting yesterday, so debug can be controlled through functions and bindings can set debug without having to know about global_mapi_ctx. [*] Finally, making debug opaque both for applications and bindings will be a great advantage for libmapi-0.9 task. We are at the moment setting the errno global variable which is obviously incompatible with a multi-threaded implementation (global_mapi_ctx is also not great for this purpose, but this can easily be worked around). We will instead store error value within the session context. This means GetLastError() will have to take a mapi_session parameter. Will post a reply when I've finished to push changes. Cheers, Julien. -- Julien Kerihuel [email protected] OpenChange Project Manager GPG Fingerprint: 0B55 783D A781 6329 108A B609 7EF6 FE11 A35F 1F79
signature.asc
Description: This is a digitally signed message part
_______________________________________________ devel mailing list [email protected] http://mailman.openchange.org/listinfo/devel
