<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/2320/">https://reviewboard.asterisk.org/r/2320/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<div>
<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/2320/diff/1/?file=33302#file33302line322" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/include/asterisk/sorcery.h</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">int ast_sorcery_apply_default(struct ast_sorcery *sorcery, const char *type, const char *name, const char *data);</pre></td>
</tr>
</tbody>
<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">322</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="kt">void</span> <span class="n">ast_sorcery_object_set_copy</span><span class="p">(</span><span class="k">struct</span> <span class="n">ast_sorcery</span> <span class="o">*</span><span class="n">sorcery</span><span class="p">,</span> <span class="k">const</span> <span class="kt">char</span> <span class="o">*</span><span class="n">type</span><span class="p">,</span> <span class="n">sorcery_copy_handler</span> <span class="n">copy</span><span class="p">);</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I initially misread this as ast_sorcery_objectset_copy(), as in copying a sorcery objectset as opposed to setting the copy handler for a sorcery object.
Maybe consider change the name to ast_sorcery_object_set_copy_handler()?
Same thing for ast_sorcery_object_set_diff().</pre>
</div>
<br />
<p>- Mark</p>
<br />
<p>On February 7th, 2013, 12:56 p.m., jcolp 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.</div>
<div>By jcolp.</div>
<p style="color: grey;"><i>Updated Feb. 7, 2013, 12:56 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;">Using sorcery in the new SIP infrastructure has uncovered a few areas where some additional sorcery features would be nice. I've implemented them, and then some!
First up is that there are two new API calls which allow a specific object type to be loaded or reloaded. This is useful for modules that extend an existing sorcery with their own object types and want to load only those upon being loaded themselves.
Second is two callbacks to allow native object copying and diffing. This is useful for performances reasons when you don't want to incur the translation from native type to key value pair. These callbacks are optional and if they aren't used the existing generic key value pair mechanism is used.
I've also added unit tests for the native copy and diff callbacks.</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 unit tests to confirm functionality.</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>/trunk/include/asterisk/sorcery.h <span style="color: grey">(381035)</span></li>
<li>/trunk/main/sorcery.c <span style="color: grey">(381035)</span></li>
<li>/trunk/tests/test_sorcery.c <span style="color: grey">(381035)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2320/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>