[asterisk-bugs] [Asterisk 0013943]: [patch] Multiple bugs in app_minivm

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13943 
====================================================================== 
Reported By:                Marquis
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13943
Category:                   Applications/app_minivm
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 158308 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-11-21 09:51 CST
Last Modified:              2008-11-24 13:11 CST
====================================================================== 
Summary:                    [patch] Multiple bugs in app_minivm
Description: 
There are several minor bugs in MiniVM, and this patch resolves most of
them.  See Additional Information for details of the rest.
====================================================================== 

---------------------------------------------------------------------- 
 (0095391) blitzrage (administrator) - 2008-11-24 13:11
 http://bugs.digium.com/view.php?id=13943#c95391 
---------------------------------------------------------------------- 
OK this is my suggestion as to how to resolve the variable naming thing.

Personally I think the variables should start with MINIVM_ because MVM_
isn't very descriptive.

However, we ideally want to maintain backwards compatibility I suppose.

How about accepting both MVM_ and MINIVM_ in 1.4, with a deprecation
warning in 1.4 stating MVM_ would be removed in future versions of
Asterisk.

Then in trunk change the prefixes to MINIVM_ straight up.

Thoughts? 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-11-24 13:11 blitzrage      Note Added: 0095391                          
======================================================================