<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7852">View Change</a></p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">res_http_websocket.c:311:52: error: pointer targets in passing<br>argument 3 of ‘ast_careful_fwrite’ differ in signedness<br>[-Werror=pointer-sign]</p><p style="white-space: pre-wrap; word-wrap: break-word;">This is actually a bug in ast_careful_fwrite(.) because it forces a<br>char* although it requires just a void*. No idea how to tackle this<br>for Asterisk 13 without going for a type cast. Any ideas?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I have no problem with changing ast_careful_fwrite() to take a void type<br>instead of a char type.  This is not C++ where that type of change would<br>change the mangled function name and thus the ABI.</p><p style="white-space: pre-wrap; word-wrap: break-word;">My own opinion of the mixing signed/unsigned warnings is that they are more<br>trouble than they are worth.  Those warnings rarely find anything useful.<br>Even more so when dealing with chars because of the hoops you have to jump<br>through to get the stupid compiler to shut up.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7852">change 7852</a>. To unsubscribe, 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/7852"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I7cceaa88e147cbdf81a3a7beec5c1c20210fa41e </div>
<div style="display:none"> Gerrit-Change-Number: 7852 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 06 Jan 2018 16:33:44 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>