<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/3193/">https://reviewboard.asterisk.org/r/3193/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On February 6th, 2014, 1:50 p.m. CST, <b>Joshua Colp</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I like the idea of being able to mark stuff as deprecated and get a warning, but I still dislike the idea of hidden fields. ^_^</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Two things here:
(1) There is already support for deprecated fields and information hiding in config_options. As sorcery is, for the most part, built on that framework, I would have expected to see those APIs being used, and not a re-creation of them. I suspect that there is a large amount of the implementation in here that could be simplified as a result.
(2) For the problem being solved here, this feels like overkill. If a consumer wants to hide information from the user, then the consumer should choose to hide that information. A framework should only hide that information when it is absolutely sure that no consumer, ever, in any module ever created (or that ever will be created), should have access to that information. I don't think that is the case here with 'disallow'.</pre>
<br />
<p>- Matt</p>
<br />
<p>On February 6th, 2014, 12:50 p.m. CST, Scott Griepentrog wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Scott Griepentrog.</div>
<p style="color: grey;"><i>Updated Feb. 6, 2014, 12:50 p.m.</i></p>
<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/jira/browse/ASTERISK-23092">ASTERISK-23092</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<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;">WAS:
Insert a ! prefix in the display of endpoint disallow value. Result is:
disallow : !(ulaw|alaw)
NOW:
Remove the disallow option from generated lists, while still accepting it from a configuration file.
This is OPTION 2 - option 1 is https://reviewboard.asterisk.org/r/3136/</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Ran command and checked output.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/branches/12/res/res_pjsip_outbound_registration.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/res/res_pjsip_endpoint_identifier_ip.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/res/res_pjsip_acl.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/res/res_pjsip/pjsip_options.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/res/res_pjsip/pjsip_configuration.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/res/res_pjsip/location.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/res/res_pjsip/config_transport.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/res/res_pjsip/config_system.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/res/res_pjsip/config_global.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/res/res_pjsip/config_domain_aliases.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/res/res_pjsip/config_auth.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/main/sorcery.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/main/bucket.c <span style="color: grey">(407566)</span></li>
<li>/branches/12/include/asterisk/sorcery.h <span style="color: grey">(407566)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3193/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>