<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 />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">ive been striping the bits apart and making them more friendly to * and res_fax
to allow the "Gateway" to manage the T38 parameter's within res_fax i propose a callback
to be placed in the bridge config.
a further callback needs to be added for the actual gateway when the channel states are correct ast_generic_bridge must end and the
gateway start.
"t38_gateway_handle_parameters" res_fax will register this and if FAXOPTS and friends enable T38 Gateway on the call via a option this call back will return a result indicating handled / not handled / T38 Gateway Start.
if it is not handled either due to non existence or if not handled status quo is maintained.
on T38 Gateway start the audio loop must end and the T38 gateway bridge callback must be called this is in res_fax and calls
technology IE res_fax_spandsp.
this callback will need to go into ast_generic_bridge and replace the current AST_CONTROL_T38_PARAMETERS case.
Please let me know if this method is in line with the API requirements.
</pre>
<br />
<p>- irroot</p>
<br />
<p>On February 23rd, 2011, 2:56 a.m., irroot wrote:</p>
<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-23 02:56:49</i></p>
<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> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/apps/app_dial.c <span style="color: grey">(308283)</span></li>
<li>/trunk/include/asterisk/features.h <span style="color: grey">(308283)</span></li>
<li>/trunk/include/asterisk/res_fax.h <span style="color: grey">(308283)</span></li>
<li>/trunk/main/features.c <span style="color: grey">(308283)</span></li>
<li>/trunk/res/res_fax.c <span style="color: grey">(308283)</span></li>
<li>/trunk/res/res_fax_spandsp.c <span style="color: grey">(308283)</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>