Summary:ASTERISK-09568: replacing of ast_exists_extension for ast_goto_if_exists
Reporter:snuffy (snuffy)Labels:
Date Opened:2007-06-02 05:34:10Date Closed:2011-06-07 14:08:00
Versions:Frequency of
Environment:Attachments:( 0) ast_goto_if.patch
Description:Was looking at the janitor projects page.
I only replaced the most obvious ones, any that stated NULL ast_channels or null callerid I left alone

Attached is a patch against SVN
Comments:By: Tilghman Lesher (tilghman) 2007-06-03 00:25:09

Could you be more explicit as to why you think this change is necessary or even desireable?

By: snuffy (snuffy) 2007-06-03 02:40:02

Other than it was listed on the janitor list on the asterisk.org page.
I have no reason. sorry.

By: Tilghman Lesher (tilghman) 2007-06-03 02:59:06

That janitor is:
"Find all places that use ast_exists_extension() and direct setting of chan->priority and change them to ast_goto_if_exists()."

None of the places in this patch are places that actually set chan->priority, as far as I can see, and so they do not fall under that janitor project.

By: snuffy (snuffy) 2007-06-03 08:40:19

Sorry I guess i seen 'or' instead of and.
my bad