[asterisk-dev] Re: [asterisk-commits] murf: branch 1.4 r59228 - /branches/1.4/pbx/pbx_ael.c

[Steve Murphy]

On Tue, 2007-03-27 at 10:51 -0500, Kevin P. Fleming wrote:
> asterisk-commits at lists.digium.com wrote:
> > Author: murf
> > Date: Mon Mar 26 16:41:32 2007
> > New Revision: 59228
> > 
> > URL: http://svn.digium.com/view/asterisk?view=rev&rev=59228
> > Log:
> > fix for 9373 (duplicate context in AEL crashes asterisk). I turned a duplicate context from a WARNING to an ERROR. Now you get a module load failure, and asterisk just exits. That's better than a crash, right\?
> 
> An error loading a module should not result in Asterisk exiting, it
> should result in the module declining to load itself (returning
> AST_MODULE_LOAD_DECLINE) so that the user can use the rest of Asterisk,
> then fix their extensions.ael and manually load pbx_ael.

Good Point! 

Done. See 59264 (trunk) and r. 59261 (1.4).

murf

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3239 bytes
Desc: not available
Url : http://lists.digium.com/pipermail/asterisk-dev/attachments/20070327/4c97499b/smime.bin