<p> Attention is currently required from: George Joseph. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17578">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</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/+/17578?tab=comments">Patch Set #3:</a> </p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">That's the situation today.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">No. The situation today is that any warnings are written *only* to the .log. They do not appear in the console at all. Furthermore, the situation today is that the Makefile unintentionally writes its own comments to the console. Finally, the situation today is that the ‘Doxyfile.in’ contains deprecated configurations. Only those warnings are printed to the console.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">The whole point of this review is to get someone to do something about the<br>warnings right? </p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">No. The warnings are all fixed by now, at least those 2369 which I were able to find. The point of this change is to fix the issues above, and as a side-effect move the warnings from the log to the console because that .log concept did not work. Furthermore, I re-enable some warnings because they were disabled nine years ago probably because of a wrong configuration file upload as part of another change. Finally, I enable warnings for internal documentation (developer mode) and enable warnings for all undocumented things (noisy). The benefit of that is, that any new Doxygen user does not have to wade through thousands of warnings but can concentrate on his newly added ones. So it is about future warnings.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">we could remind folks via […] asterisk-dev list that if they create or change<br>doxygen comments, they should run "make progdocs"</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">OK. Yes, that was my plan, after this is submitted.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">they're still going to get lost in the normal output</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">As of today, they are not there, just in the doxygen.log. And even that is incomplete because warnings got disabled or not parsed like internal docs.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Search what?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">‘warning:’ or better ‘: warning: ’</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">If you can find a way to reduce doxygen and dot's verbosity to just a handful<br>of progress messages, that would be better.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">That is a change on top of this change actually. A plus to this change here. As of today, those messages are printed. I just *added* the warnings.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Doxygen offers the possibility of QUIET=YES. That prints *just* warnings then, which is alternative D. On my computer 'make progdocs' needs 65 seconds on a second run, a ten year old computer but with an SSD.</p><p style="white-space: pre-wrap; word-wrap: break-word;">You do not like that either. Then, I re-create the doxygen.log, leave the console output as is, but add a note to the Makefile/console, after Doxygen was executed, that any warnings can be found in doxygen.log.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17578">change 17578</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/+/17578"/><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: I4129092a199d5e24c319a09cd088614b121015af </div>
<div style="display:none"> Gerrit-Change-Number: 17578 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Attention: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 07 Dec 2021 16:34:06 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Comment-In-Reply-To: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>