[svn-commits] murf: branch 1.6.0 r119931 - in /branches/1.6.0: ./ res/ael/pval.c

[SVN commits to the Digium repositories]

Author: murf
Date: Tue Jun  3 10:12:28 2008
New Revision: 119931

URL: http://svn.digium.com/view/asterisk?view=rev&rev=119931
Log:
Merged revisions 119930 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
r119930 | murf | 2008-06-03 09:07:20 -0600 (Tue, 03 Jun 2008) | 24 lines

Merged revisions 119929 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r119929 | murf | 2008-06-03 08:49:46 -0600 (Tue, 03 Jun 2008) | 16 lines

as per http://lists.digium.com/pipermail/asterisk-users/2008-June/212934.html,
which is a message from Philipp Kempgen, requesting that the WARNING
that an extension is empty be reduced to a NOTICE or less, as empty
extensions are syntactically possible, and no big deal.

With which I agree, and have removed that WARNING message entirely.
I think it is not necessary to see this message. It didn't 
state that a NoOp() was inserted automatically on your behalf,
and really, as users, who cares? Why freak out dialplan writers
with unnecessary warnings? The details of the machinations a compiler goes
thru to produce working assembly code is of little interest
to most programmers-- we will follow the unix principal of
doing our work silently.



........

................

Modified:
    branches/1.6.0/   (props changed)
    branches/1.6.0/res/ael/pval.c

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

Modified: branches/1.6.0/res/ael/pval.c
URL: http://svn.digium.com/view/asterisk/branches/1.6.0/res/ael/pval.c?view=diff&rev=119931&r1=119930&r2=119931
==============================================================================
--- branches/1.6.0/res/ael/pval.c (original)
+++ branches/1.6.0/res/ael/pval.c Tue Jun  3 10:12:28 2008
@@ -4108,12 +4108,6 @@
 						exten-> return_target = np2;
 					}
 					/* is the last priority in the extension a label? Then add a trailing no-op */
-					if( !exten->plist_last )
-					{
-						ast_log(LOG_WARNING, "Warning: file %s, line %d-%d: Empty Extension!\n",
-								p2->filename, p2->startline, p2->endline);
-					}
-					
 					if ( exten->plist_last && exten->plist_last->type == AEL_LABEL ) {
 						struct ael_priority *np2 = new_prio();
 						np2->type = AEL_APPCALL;