<p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4; color: #000000;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14887">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/asterisk/+/14887/3/include/asterisk/logger_category.h">File include/asterisk/logger_category.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/+/14887/3/include/asterisk/logger_category.h@129">Patch Set #3, Line 129:</a> <code style="font-family:monospace,monospace">ast_debug_category_allow</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">ast_debug_is_category_allowed might be better.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14887/3/include/asterisk/rtp_engine.h">File include/asterisk/rtp_engine.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/+/14887/3/include/asterisk/rtp_engine.h@2838">Patch Set #3, Line 2838:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">#define AST_DEBUG_CATEGORY_RTP (1 << 0) /* RTP debug logging category id */<br>#define AST_DEBUG_CATEGORY_RTP_PACKET (1 << 1) /* RTP packet debug logging category id */<br>#define AST_DEBUG_CATEGORY_RTCP (1 << 2) /* RTCP debug logging category id */<br>#define AST_DEBUG_CATEGORY_RTCP_PACKET (1 << 3) /* RTCP packet debug logging category id */<br>#define AST_DEBUG_CATEGORY_DTLS (1 << 4) /* DTLS debug logging category id */<br>#define AST_DEBUG_CATEGORY_DTLS_PACKET (1 << 5) /* DTLS packet debug logging category id */<br>#define AST_DEBUG_CATEGORY_ICE (1 << 8) /* ICE debug logging category id */<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think the problem now is that the next person who comes along and wants to add category debugging will need to search through all the source files to see what ids are in use/available.  Do we really need the capability to test multiple categories for a debug message?  If so, then I'd have the register function return an id to the module rather than the module providing one.  Alternatively, you could take a list of names to check.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14887/3/include/asterisk/stun.h">File include/asterisk/stun.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/+/14887/3/include/asterisk/stun.h@41">Patch Set #3, Line 41:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">#define AST_DEBUG_CATEGORY_STUN (1 << 6) /* STUN debug logging category id */<br>#define AST_DEBUG_CATEGORY_STUN_PACKET (1 << 7) /* STUN packet debug logging category id */<br><br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This highlights my earlier comment about needing to know which ids are in use/available.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14887">change 14887</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/+/14887"/><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: I6e6cb247bb1f01dbf34750b2cd98e5b5b41a1849 </div>
<div style="display:none"> Gerrit-Change-Number: 14887 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 03 Sep 2020 12:31:54 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>