<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/1116/">https://reviewboard.asterisk.org/r/1116/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers, Russell Bryant, dimas, Leif Madsen, and dafe_von_cetin.</div>
<div>By irroot.</div>
<p style="color: grey;"><i>Updated 2011-02-27 11:26:58.050713</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">been busy over the weekend ...
1)chan_sip now has a T38 Switchover option as a partner to fax detect to trigger T38 via res_fax
2)asterisk internals will see this switchover and try negotiate on peer when answered using res_fax
There is now no need for a "gateway" app as per mantis patch / callweaver all is done via 3 exports from res_fax
1)A procedure that handles all T38 parameter negotiations and stores them in the fax data store and passes them to the channel this is the hook for all T38 across all channels no data is manipulated here only handled and stored this is important to maintain transparency
2)a switchover routine that will allow any channel regardless of T38 support to enable the gateway on fax detection as can be seen in the chan_Sip patch [included] this is straight forward
3)before a channel is bridged if it has been switched over above or if the endpoint has switched over [there will be a T38 parameter on the channel in the faxstore] a request for negotiation with the appropriate settings will be sent to the peer.
Regards Greg</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi there the patch that was going around circa 2008 to implement this in 1.4/1.6 app_fax has been moved to trunk [1.10]
ive made some cleanups and moved it into res_fax res_fax_spandsp this is the framework and not production code
unfortunately i have no means of testing it at the moment and require help.
i have cleaned the code up substantially it is related to R459
hope this is found useful and aids in the goal to get it in 1.10.
Adds application FaxGateway / FaxDetect
Adds alternate bridge to Dial with new option.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/view.php?id=13405">13405</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/channels/chan_sip.c <span style="color: grey">(308944)</span></li>
<li>/trunk/channels/sip/include/sip.h <span style="color: grey">(308944)</span></li>
<li>/trunk/include/asterisk/channel.h <span style="color: grey">(308944)</span></li>
<li>/trunk/include/asterisk/res_fax.h <span style="color: grey">(308944)</span></li>
<li>/trunk/main/channel.c <span style="color: grey">(308944)</span></li>
<li>/trunk/res/res_fax.c <span style="color: grey">(308944)</span></li>
<li>/trunk/res/res_fax_spandsp.c <span style="color: grey">(308944)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1116/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>