Summary: | ASTERISK-09568: replacing of ast_exists_extension for ast_goto_if_exists | ||
Reporter: | snuffy (snuffy) | Labels: | |
Date Opened: | 2007-06-02 05:34:10 | Date Closed: | 2011-06-07 14:08:00 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
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. http://www.asterisk.org/developers/janitor 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 |