[asterisk-dev] [Code Review] The 'e' special extension fails to trigger in at least two cases.

[rmudgett]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1196/
-----------------------------------------------------------

(Updated 2011-04-25 16:15:07.116908)


Review request for Asterisk Developers.


Changes
-------

Prioritized and optimized some hangup tests associated with the 'e' extension.


Summary
-------

The 'e' extension is a fall back for the 'i', 't', or 'T' extensions if any of them do not exist.

Many of the places the 'e' extension was supposed to be invoked fail because the priority was set wrong.  There were two places where the 'e' extension was not even checked for fall back.


This addresses bug 19136.
    https://issues.asterisk.org/view.php?id=19136


Diffs (updated)
-----

  /branches/1.8/main/pbx.c 315348 

Diff: https://reviewboard.asterisk.org/r/1196/diff


Testing
-------

Code inspection.  The 'e' extension is invoked similarly to the other exception extensions: 'i', 't', and 'T' since it is a fallback for the other extensions.


Thanks,

rmudgett

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110425/c32cfc8d/attachment.htm>