<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17502">View Change</a></p><p>27 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File CREDITS:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/2848be7d_e0f28555">Patch Set #1, Line 202:</a> <code style="font-family:monospace,monospace">          mailbox in voicemail.  Also an issue with voicemail </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">i presume the pink here is trailing whitespace. I try not to do more than what it says on the tin, but if there's a policy about whitespace...</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File Makefile:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/434e2348_b7ea9f3c">Patch Set #1, Line 761:</a> <code style="font-family:monospace,monospace">install: install</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I meant to drop this</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File apps/app_minivm.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/882a78d1_324e8b7f">Patch Set #1, Line 2957:</a> <code style="font-family:monospace,monospace">    if ((chanvar = ast_variable_retrieve(cfg, "general", "emailaddress")))</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Oops, this shouldn't be here (the next if covers it)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File channels/chan_iax2.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/363bb10f_b339d39c">Patch Set #1, Line 5852:</a> <code style="font-family:monospace,monospace"> const struct ast_channel *requestor, unsigned int cacheable)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">If the `cacheable` changes are annoying, I can drop them</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File channels/chan_skinny.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/297411ec_351f5f35">Patch Set #1, Line 716:</a> <code style="font-family:monospace,monospace">#define SOFTKEY_TRANSFER 0x04</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Calling this (and its sibling) out in case they're interesting</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File channels/chan_unistim.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/69c17c31_e27f0fd7">Patch Set #1, Line 3159:</a> <code style="font-family:monospace,monospace">               send_text_status(s, ustmtext("TransferCancel", s));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">the corresponding localization is changed below</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File codecs/lpc10/README:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/b46ab987_fafbe13a">Patch Set #1, Line 71:</a> <code style="font-family:monospace,monospace">The f2c output originally had a file called control_com.c, that defined</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I should drop these changes</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File codecs/lpc10/analys.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/4196a735_1d9d78b8">Patch Set #1, Line 37:</a> <code style="font-family:monospace,monospace">/* comlen control_ 12 */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Happy to drop if this is annoying</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/659791a8_ffb11c51">Patch Set #1, Line 134:</a> <code style="font-family:monospace,monospace">/* that was defined in the COMMON block CONTRL in control.fh. */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I can't find `contrl.fh` ... This change is probably wrong, but??</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File codecs/lpc10/dyptrk.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/6059caf9_dc1d949c">Patch Set #1, Line 318:</a> <code style="font-family:monospace,monospace">/* SEESAW: Construct a pitch pointer array and intermediate winner function</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This isn't a `control` :-)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File codecs/lpc10/voicin.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/a0a84574_b32d4a9d">Patch Set #1, Line 545:</a> <code style="font-family:monospace,monospace">/*  The VOICE array contains the result of the linear discriminant function</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">not `control`</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File configs/samples/res_config_sqlite3.conf.sample:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/f9a324dd_dc0b3298">Patch Set #1, Line 32:</a> <code style="font-family:monospace,monospace">; busy timeout - timeout in milliseconds to stop waiting for locked database.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This might be an upstream file (I have sent changes to sqlite, they weren't integrated into trunk)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File contrib/scripts/install_prereq:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/b0a7bc7a_c876e371">Patch Set #1, Line 7:</a> <code style="font-family:monospace,monospace"># prerequisites</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">notable word change</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File contrib/scripts/managerproxy.pl:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/651212a6_8bfc8a55">Patch Set #1, Line 212:</a> <code style="font-family:monospace,monospace">    my $command = shift;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File funcs/func_odbc.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/2597b8a5_b06548a8">Patch Set #1, Line 64:</a> <code style="font-family:monospace,monospace">                                  <value name="SUCCESS"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">notable</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File include/asterisk/bridge.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/7c4bce45_608948e1">Patch Set #1, Line 1044:</a> <code style="font-family:monospace,monospace">void ast_bridge_set_remb_behavior(struct ast_bridge *bridge, enum ast_bridge_video_sfu_remb_behavior behavior);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">api change</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File include/asterisk/causes.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/68950b7e_7e4c402e">Patch Set #1, Line 100:</a> <code style="font-family:monospace,monospace">#define AST_CAUSE_MISDIALED_TRUNK_PREFIX         5</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">api?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File main/channel.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/58f263e0_7af1d03b">Patch Set #1, Line 139:</a> <code style="font-family:monospace,monospace">   { AST_CAUSE_PREEMPTED, "PREEMPTED", "Preempted" },</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">hope this is ok, easily dropped if not</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File main/pbx.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/f6159a3d_df9f9aeb">Patch Set #1, Line 3947:</a> <code style="font-family:monospace,monospace"> /* Cleanup the NOTIFYs if hint is removed */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This is mostly to make my tool happy, certainly droppable.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Can't tell if `NOTIFY` is a technical thing, or if this could be written as `notifications`</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File missing:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/b36a9ee8_946acac8">Patch Set #1, Line 191:</a> <code style="font-family:monospace,monospace">         it often tells you about the needed prerequisites for installing</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Notable change</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_corosync.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/552333e7_7d808ec7">Patch Set #1, Line 753:</a> <code style="font-family:monospace,monospace">                        ast_log(LOG_WARNING, "Failed to extract Corosync node ID for this node. Not informing cluster of existence.\n");</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">not `cacheable` (and following)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_pjsip_publish_asterisk.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/945c2ca0_7e5ee0d3">Patch Set #1, Line 497:</a> <code style="font-family:monospace,monospace">       int cacheable = ast_json_integer_get(ast_json_object_get(json, "cacheable"));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">this would be some sort of api change, and so either needs a compat thing or to just be dropped.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/a45ef05c_d3005eb9">Patch Set #1, Line 518:</a> <code style="font-family:monospace,monospace">              cacheable == AST_DEVSTATE_CACHABLE ? AST_DEVSTATE_CACHABLE : AST_DEVSTATE_NOT_CACHABLE,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I can't remember why I didn't change the uppercase flavor.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_xmpp.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/85331f35_fbcc41dc">Patch Set #1, Line 1663:</a> <code style="font-family:monospace,monospace"> * \brief Dial plan function to retrieve the status of a buddy.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">not `cacheable` and following</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File rest-api/api-docs/endpoints.json:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/dba12bc6_0f0b098e">Patch Set #1, Line 58:</a> <code style="font-family:monospace,monospace">                                                     "description": "The \"variables\" key in the body object holds technology specific key/value pairs to append to the message. These can be interpreted and used by the various resource types; for example, pjsip and sip resource types will add the key/value pairs as SIP headers,",</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File static-http/ajamdemo.html:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/8ab4c904_04748b63">Patch Set #1, Line 30:</a> <code style="font-family:monospace,monospace">   var transferredchan = "";</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File tests/test_utils.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17502/comment/607e0600_5a751cbf">Patch Set #1, Line 51:</a> <code style="font-family:monospace,monospace">   const char *in = "abcdefghijklmnopqrstuvwxyz ABCDEFGHIJKLMNOPQRSTUVWXYZ 1234567890 ~`!@#$%^&*()_-+={[}]|\\:;\"'<,>.?/";</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">notable</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17502">change 17502</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/17502"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ic6b76bfe3fd3f92983c5af911e91081a3a3cfa3d </div>
<div style="display:none"> Gerrit-Change-Number: 17502 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Josh Soref <jsoref@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 17 Nov 2021 05:41:20 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>