<p>Patch set 4:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/10151">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/#/c/10151/4/configure.ac">File configure.ac:</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/10151/4/configure.ac@691">Patch Set #4, Line 691:</a> <code style="font-family:monospace,monospace"> JANSSON_DEFINE_JSON_INT()</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This should probably pass $JANSSON_INCLUDE to deal with any packaged version where the header is not in /usr/include/jansson.h.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/10151/4/configure.ac@787">Patch Set #4, Line 787:</a> <code style="font-family:monospace,monospace">AC_CHECK_FUNCS([asprintf atexit closefrom dup2 eaccess endpwent euidaccess ffsll ftruncate getcwd gethostbyname gethostname getloadavg gettimeofday glob ioperm inet_ntoa isascii memchr memmove memset mkdir mkdtemp munmap newlocale ppoll putenv re_comp regcomp select setenv socket strcasecmp strcasestr strchr strcspn strdup strerror strlcat strlcpy strncasecmp strndup strnlen strrchr strsep strspn strstr strtod strtol strtoll strtold strtoq unsetenv uselocale utime vasprintf getpeereid sysctl swapctl])</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Not a big deal but this change is no longer needed.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/#/c/10151/4/third-party/jansson/configure.m4">File third-party/jansson/configure.m4:</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/10151/4/third-party/jansson/configure.m4@98">Patch Set #4, Line 98:</a> <code style="font-family:monospace,monospace">                  [exit(!JSON_INTEGER_IS_LONG_LONG || sizeof(json_int_t) != sizeof(long long));])],</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think we should do the sizeof comparison.  We could do:<br>#if !JSON_INTEGER_IS_LONG_LONG<br>#error "not long long"<br>#endif</p><p style="white-space: pre-wrap; word-wrap: break-word;">This would work with AC_COMPILE_IFELSE and which is cross-compile compatible.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10151">change 10151</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/10151"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I2af275286ee5e795b79f0c3d450d9e4b28e958b0 </div>
<div style="display:none"> Gerrit-Change-Number: 10151 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 21 Sep 2018 17:52:42 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>