[Home]

Summary:ASTERISK-05450: jumping janitor project
Reporter:BJ Weschke (bweschke)Labels:
Date Opened:2005-11-04 09:09:49.000-0600Date Closed:2008-01-15 15:54:10.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_chanisavail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_chanisavail_jumping.patch
Description:n+101 janitor project patch - STATUS variable was already being set, so just added the 'j' option.
Comments:By: Jason Parker (jparker) 2005-11-04 14:57:41.000-0600

Did you perhaps mean to do

+ if (strchr(options, 'j'))
+ priority_jump = 1;

?
By: BJ Weschke (bweschke) 2005-11-04 15:09:55.000-0600

yup. I did. nice catch. thanks.
By: Mark Spencer (markster) 2005-11-06 00:18:27.000-0600

This one used strchr originally, but shall we go ahead and use parse_options / (and less importantly parse_args) while we're at it?
By: BJ Weschke (bweschke) 2005-11-06 10:11:59.000-0600

markster: Yes. Can and will plan to do that, but the intent was to get through all of the apps that had "non-1.2 jumping behavior" and fix them prior to rc1 dropping so, at the very least, that behavior was consistent across all the apps. There's no other reason than just my own time management that I didn't go back and use the API functions.
By: Russell Bryant (russell) 2005-11-06 13:21:09.000-0600

added to cvs head, thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:54:10.000-0600

Repository: asterisk
Revision: 6980

U   trunk/ChangeLog
U   trunk/apps/app_chanisavail.c

------------------------------------------------------------------------
r6980 | russell | 2008-01-15 15:54:10 -0600 (Tue, 15 Jan 2008) | 2 lines

issue ASTERISK-5450

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=6980