[asterisk-commits] mmichelson: branch 1.6.0 r114578 - /branches/1.6.0/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Apr 22 20:00:55 CDT 2008

Author: mmichelson
Date: Tue Apr 22 20:00:54 2008
New Revision: 114578

URL: http://svn.digium.com/view/asterisk?view=rev&rev=114578
Blocked revisions 114577 via svnmerge

r114577 | mmichelson | 2008-04-22 19:58:49 -0500 (Tue, 22 Apr 2008) | 23 lines

Round 2 of IMAP_STORAGE app_voicemail.c fixes:

This fixes a bug that was thought to be fixed already.

app_voicemail, if using IMAP_STORAGE, has a problem because
the IMAP header files include syslog.h, which define LOG_WARNING
and LOG_DEBUG to be different than what Asterisk uses for those
same macros. This was "fixed" in the past by including all the 
IMAP header files prior to including asterisk.h. This fix worked...
unless you were to try to compile with MALLOC_DEBUG. MALLOC_DEBUG
prepends the inclusion of astmm.h to every file, which means that no
matter what order the includes are in in app_voicemail, the unexpected
values for LOG_WARNING and LOG_DEBUG will be in place.

The action taken for this fix was to define AST_LOG_* macros in addition
to the LOG_* macros already defined. These new macros are used in app_voicemail.c,
logger.h, and astobj.h right now, and their use will be encouraged in the future.

In consideration of those who have written third-party modules which use 
the LOG_* macros, these will NOT be removed from the source, however future use
of these macros is discouraged.


    branches/1.6.0/   (props changed)

Propchange: branches/1.6.0/
Binary property 'trunk-blocked' - no diff available.

More information about the asterisk-commits mailing list