[asterisk-dev] Re: branch 1.2 r12927 - /branches/1.2/apps/app_dial.c

[Tony Mountifield]

In article <4421033E.5060003 at alphaque.com>,
Dinesh Nair <dinesh at alphaque.com> wrote:
> 
> On 03/17/06 21:13 Tony Mountifield said the following:
> > See below...
> >>when using the G() option to Dial, fix sending the called channel to 1 priority
> >>beyond what was specified (issue #6523)
> 
> as the person responsible for submitting issue #6523, i did also submit two 
> possible "fixes" for the problem:
> 
> 1. one was a patch to app_dial.c which made it behave like the documentation...
> 
> and 2. was a patch which made the documentation say exactly how it behaved.

Yes, I saw that when I looked at the bug report. But it had been actioned and
closed by the time I got there, which is why I raised it in asterisk-dev, in
order to point out the advantages I saw in retaining and documenting the
original behaviour.

> after this thread i believe russel would be backing out the app_dial.c 
> patch and committing the patch for the documentation instead.

He did so a couple of days ago.

Cheers
Tony
-- 
Tony Mountifield
Work: tony at softins.co.uk - http://www.softins.co.uk
Play: tony at mountifield.org - http://tony.mountifield.org