<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/1843/">https://reviewboard.asterisk.org/r/1843/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On April 3rd, 2012, 4 p.m., <b>Mark Michelson</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 took a look at the review you linked to, and it appears the only thing the review in question changed was the behavior of the 'N' option. The 'K' and 'n' options were unchanged by that review. As a result, you are changing the behavior of the 'K' and 'n' options in this review request to something they've never done before. As much as it may make sense to operate this way, that's not how the options have ever behaved, so you need to revert those changes.</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;">Alright, done. Only change kept was the N option.
I'm not posting the new diff because it's just:
===================================================================
--- apps/app_meetme.c (revision 361037)
+++ apps/app_meetme.c (working copy)
@@ -4601,7 +4601,7 @@
user->adminflags |= ADMINFLAG_MUTED;
break;
case 78: /* N: Mute all (non-admin) users */
- ao2_callback(cnf->usercontainer, OBJ_NODATA, user_set_muted_cb, NULL);
+ ao2_callback(cnf->usercontainer, OBJ_NODATA, user_set_muted_cb, &cnf);
break;
case 109: /* m: Unmute */
user->adminflags &= ~(ADMINFLAG_MUTED | ADMINFLAG_SELFMUTED | ADMINFLAG_T_REQUEST);
</pre>
<br />
<p>- jrose</p>
<br />
<p>On April 3rd, 2012, 3:39 p.m., jrose wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/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, Mark Michelson and Matt Jordan.</div>
<div>By jrose.</div>
<p style="color: grey;"><i>Updated April 3, 2012, 3:39 p.m.</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;">This bug was introduced when jpeeler did a major overhaul in MeetMe to prevent deadlocks by using ao2_callbacks. He added a condition which checked for the presence of a pointer to the conference which wasn't included with all of the ao2_callback calls which used those function pointers. This patch looked over the other call to user_set_muted_cb which worked fine and applied the difference (inclusion of a pointer to the conference) elsewhere.
Noting that, I applied the same thing to a few other similar commands in the area.
https://reviewboard.asterisk.org/r/746/
This is a review for the patch which appears to have introduced this.</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;">So far I've mostly just tested with the dialplan shown in the issue. It appears to work properly now.</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/jira/browse/ASTERISK-19335">ASTERISK-19335</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>/branches/1.8/apps/app_meetme.c <span style="color: grey">(361037)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1843/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>