<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/2546/">https://reviewboard.asterisk.org/r/2546/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 16th, 2013, 2:39 p.m. UTC, <b>Paul Belanger</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://reviewboard.asterisk.org/r/2546/diff/1/?file=38112#file38112line678" style="color: black; font-weight: bold; text-decoration: underline;">/team/jrose/bridge_projects/res/res_parking.c</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">678</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="tb"> </span><span class="tb"> </span><span class="n">ast_string_field_set</span><span class="p">(</span><span class="n">lot_cfg</span><span class="p">,</span> <span class="n">parkext</span><span class="p">,</span> <span class="s">"700"</span><span class="p">);</span></pre></td>
</tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Any way we cannot do this? Always been a peeve of mine, that asterisk defaults to an extension number. Seems to be the only place in asterisk were we do this.
I'd rather see the user has to explicitly define the extension number vs asterisk just assigning something.</pre>
</blockquote>
<p>On May 16th, 2013, 2:53 p.m. UTC, <b>jrose</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;">Let's take the question to the mailing list. I don't know the answer for this and I was improvising for maximum backwards compatability.
I'll go ahead and say though... it's not like this is forced now. If you just provide a [default] category in res_confbridge.conf, then this default won't be applied and it will use the regular options. Then if you don't put parkext in [default], it won't even generate extensions at all and you can take complete control of the extensions related to parking (Except for park-dial extensions, those get generated no matter what... I think I might add an option to disable that).</pre>
</blockquote>
<p>On May 16th, 2013, 3:24 p.m. UTC, <b>jrose</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;">Actually I'm hearing from Matt that having the default use the old behavior when it isn't overridden is the way to go for now. If you want to discuss this further, feel free to get the conversation going.</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Extending on what Jonathan just said - if you create a [default] parking lot, you'll define the extensions you want for the on ramps to parking - at which point, you've changed the 'default' parking lot. There won't be a '700' at that point - it will still be the 'default' parking lot, but it will have whatever extensions you've defined.
So you can override it - and there really isn't any other way to make that any more configurable.
</pre>
<br />
<p>- Matt</p>
<br />
<p>On May 15th, 2013, 10:02 p.m. UTC, jrose 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.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers, Matt Jordan and rmudgett.</div>
<div>By jrose.</div>
<p style="color: grey;"><i>Updated May 15, 2013, 10:02 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-21643">ASTERISK-21643</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;">Default Parking Lot. Pretty self explanatory. It's a parking lot. It's named default. It uses default options.
Actually, there is one difference between this parking lot and what you would get if you just stuck
[default]
in the config file, and that's that if the default parking lot isn't specified in this way, it will set parkext to 700 so that dialplan generation
will be used and an actual usable parking lot similar to the ones that used to be generated by features.conf will be set up.</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;">Tested with and without the presence of [default] in res_parking.conf
Tested reloads removing and adding [default]
Checked memory consumption before and after reloads for both adding and removing default.
Checked options in the default parking lot created without the presence of [default]
Checked options in the default parking lot created with the presence of [default]</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>/team/jrose/bridge_projects/res/res_parking.c <span style="color: grey">(388603)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2546/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>