[asterisk-bugs] [Asterisk 0017145]: AEL warning when using Return() application is misleading

[Asterisk Bug Tracker]

The following issue has been RESOLVED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17145 
====================================================================== 
Reported By:                okrief
Assigned To:                lmadsen
====================================================================== 
Project:                    Asterisk
Issue ID:                   17145
Category:                   Utilities/aelparse
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     resolved
Target Version:             1.6.2.9
Asterisk Version:           1.6.1.18 
JIRA:                       SWP-1274 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-04-07 01:00 CDT
Last Modified:              2010-05-12 14:31 CDT
====================================================================== 
Summary:                    AEL warning when using Return() application is
misleading
Description: 
When using Return() app in AEL script, CLI output includes:
[Apr  2 14:11:12] WARNING[2217]: ael/pval.c:2522 check_pval_item: Warning:
file /etc/asterisk/extensions.ael, line 660-660: application call to Return
affects flow of control, and needs to be re-written using AEL if, while,
goto, etc. keywords instead!
To my knowledge, using return keyword instead is not a desirable
replacement as you cannot add a value to it.

In UPGRADE-1.6.txt, you can read:
AEL:

* Macros are now implemented underneath with the Gosub() application.
  Heaven Help You if you wrote code depending on any aspect of this!
  Previous to 1.6, macros were implemented with the Macro() app, which
  provided a nice feature of auto-returning. The compiler will do its
  best to insert a Return() app call at the end of your macro if you did
  not include it, but really, you should make sure that all execution
  paths within your macros end in "return;".

My understanding of all this is that AEL warning is misleading as using
Return() is not deprecated. 
====================================================================== 

---------------------------------------------------------------------- 
 (0121831) svnbot (reporter) - 2010-05-12 14:31
 https://issues.asterisk.org/view.php?id=17145#c121831 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 262796

U   trunk/res/ael/pval.c

------------------------------------------------------------------------
r262796 | lmadsen | 2010-05-12 14:31:42 -0500 (Wed, 12 May 2010) | 4 lines

Remove unnecessary WARNING message in ael/pval.c

(closes issue https://issues.asterisk.org/view.php?id=17145)
Reported by: okrief
------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=262796 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-05-12 14:31 svnbot         Checkin                                      
2010-05-12 14:31 svnbot         Note Added: 0121831                          
2010-05-12 14:31 svnbot         Status                   acknowledged =>
assigned
2010-05-12 14:31 svnbot         Status                   assigned => resolved
2010-05-12 14:31 svnbot         Resolution               open => fixed       
======================================================================