Summary: | ASTERISK-06653: [patch] minor change and code clean in app_stack.c | ||
Reporter: | Peng Yong (ppyy) | Labels: | |
Date Opened: | 2006-03-29 08:19:49.000-0600 | Date Closed: | 2006-03-29 13:34:37.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_stack |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) patch.app_stack ( 1) patch.app_stack.2 | |
Description: | 1. remove LOCAL_USER_ADD in gosub_exec, it will called in the gosub_exec which is called by gosubif_exec 2. code clean | ||
Comments: | By: Tilghman Lesher (tilghman) 2006-03-29 08:30:33.000-0600 You obviously did not test this patch, because it does not work. By: Peng Yong (ppyy) 2006-03-29 08:38:28.000-0600 sorry, upload again By: Tilghman Lesher (tilghman) 2006-03-29 13:34:11.000-0600 The point of LOCAL_USER_ADD is that we don't attempt to unload while executing. This is why calling it twice is not harmful, but leaving it wide open could be. I have to disagree with your patch for this reason. |