asterisk/funcs
Steve Murphy 4c937339e5 As per discussion on bug 7862, the problem wasn't the fact that the documentation differed from behavior, but rather that users are used to REGEX having that space after the double quote in 1.2.x. So, in keeping with history, I investigated a little deeper, and discovered that the change in behavior was due to the modification of the function to use the AST_DECLARE_APP_ARGS and AST_NONSTANDARD_APP_ARGS() to parse the args. The code to skip the blank was left out. So, what I did was add code to throw out the first blank (space or tab) after the double quote, IF IT IS THERE. If not, nothing is done.Verbage is added to the function description saying that the space is optional, and skipped if it is there. If a space is desired, then the documentation advises putting two spaces there. This should make it compatible for 1.2 users, and not mess up new users who are used to using it with no space. It WILL mess up new users who WANT a space. Hopefully, they will double check the doc strings for this func and add the extra space. Hopefully, this class of new user is very small.
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@42423 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2006-09-08 16:44:38 +00:00
..
Makefile GNU make already knows how to quietly ignore non-existent files in 'include' directives 2006-08-26 23:58:47 +00:00
func_base64.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_callerid.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_cdr.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_channel.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_curl.c Some changes/fixes for func_curl. curl_global_init is only supposed to be called once, and if it returns non-zero, we need to give up on further executions with that instance. Additionally, let's set absolute timeout values for the CURL connections to try and prevent possible Zap (and possibly other channel tech) channel lockouts. 2006-09-03 19:07:58 +00:00
func_cut.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_db.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_enum.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_env.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_global.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_groupcount.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_language.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_logic.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_math.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_md5.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_moh.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_odbc.c Missing field terminator 2006-09-05 17:23:50 +00:00
func_rand.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_realtime.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_sha1.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_strings.c As per discussion on bug 7862, the problem wasn't the fact that the documentation differed from behavior, but rather that users are used to REGEX having that space after the double quote in 1.2.x. So, in keeping with history, I investigated a little deeper, and discovered that the change in behavior was due to the modification of the function to use the AST_DECLARE_APP_ARGS and AST_NONSTANDARD_APP_ARGS() to parse the args. The code to skip the blank was left out. So, what I did was add code to throw out the first blank (space or tab) after the double quote, IF IT IS THERE. If not, nothing is done.Verbage is added to the function description saying that the space is optional, and skipped if it is there. If a space is desired, then the documentation advises putting two spaces there. This should make it compatible for 1.2 users, and not mess up new users who are used to using it with no space. It WILL mess up new users who WANT a space. Hopefully, they will double check the doc strings for this func and add the extra space. Hopefully, this class of new user is very small. 2006-09-08 16:44:38 +00:00
func_timeout.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00
func_uri.c merge new_loader_completion branch, including (at least): 2006-08-21 02:11:39 +00:00