<p>Patch set 1:<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/+/14414">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/+/14414/1/configs/samples/logger.conf.sample">File configs/samples/logger.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/+/14414/1/configs/samples/logger.conf.sample@150">Patch Set #1, Line 150:</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;">Debug<br>; messages are also very verbose and can and do fill up logfiles quickly;<br>; this is another reason not to have debug mode on a production system unless<br>; you are in the process of debugging a specific issue.<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'd say add a note here, or specify "Debug and trace...", about trace's potential effect on one's system load.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14414/1/main/cli.c">File main/cli.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/+/14414/1/main/cli.c@664">Patch Set #1, Line 664:</a> <code style="font-family:monospace,monospace">                   ml = find_module_level(mod, &debug_modules);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">What happens here if both debug and trace levels are specified for a given module name? For instance, it seems like it could remove the debug module for debugging, and then add the trace one. But then you'd lose the debug level? And vice versa in the similar debug level code?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/14414/1/main/cli.c@716">Patch Set #1, Line 716:</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;">         AST_RWLIST_WRLOCK(&debug_modules);<br>                while ((ml = AST_RWLIST_REMOVE_HEAD(&debug_modules, entry))) {<br>                    ast_free(ml);<br>         }<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This removes all debug modules as well thus wiping out any debug levels set for those modules.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14414">change 14414</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/+/14414"/><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: Ic5ebb859883f9c10a08c5630802de33500cad027 </div>
<div style="display:none"> Gerrit-Change-Number: 14414 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 18 May 2020 21:31:30 +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>