<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">The jansson >= 2.11 requirement was documented in UPGRADE.txt for 16.0.0 and above.  The configure script will not allow use of older jansson versions in ASterisk 16+.  The patch is valid for 13 and 13.30.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Is there a reason we still wouldn't include this? This code was present in the previous 16 code. It only landed in 16.4.0</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Asterisk 16.0.0 blocks using versions of jansson which lack support for "s?" and the updated code is simpler.  The fact that "s?" wasn't added here until 16.4.0 doesn't matter, 16.0.0 requires the thread safety provided by jansson 2.11 so we can freely use any feature of 2.11.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/13474">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/13474">change 13474</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/+/13474"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: I6d596007ae85e8724d928865d99968f679be1142 </div>
<div style="display:none"> Gerrit-Change-Number: 13474 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Joshua C. Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 17 Dec 2019 13:57:58 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>