<table><tr><td style="">kossebau added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22143">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D22143#502314" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D22143#502314</a>, <a href="https://phabricator.kde.org/p/meven/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@meven</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">Relying on undocumented names of generated sources files does not get my +1. That needs someone else to take responsibility :)</pre></div></blockquote>

<p>Well this kind of file generation is common and is indirectly documented through the ecm_qt_declare_logging_category macro.</p></div>
</blockquote>

<p>The current implementation of the macro is an internal detail. It is not part of the API contract.<br />
Binding one's code to current internal implementation has two disadvantages: it makes it harder for the macro developers to enhance the macro, because they would break your code, Or they do not know someone is relying on internal details, and your code breaks one day.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I don't think this is an important issue.</p>

<p>For what it is worth, the same technique is used in other repos :</p></blockquote>

<p>IMHO all cases which are bad code :)</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>This does not justify it but makes the points that other KDE devs did not see an issue with it.</p></blockquote>

<p>All us KDE devs have different backgrounds, so not surprising there are different opinions :)</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I would gladly use an alternative if presented with one.</p></blockquote>

<p>I just tried to give one in my previous comment: regenerating the respective logging source files in the autotest dir again, by calling the macro there once more.<br />
Or extend the documentation of the macro, to specify the cpp name. Then its part of the API contract officially :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R159 KActivities Statistics</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22143">https://phabricator.kde.org/D22143</a></div></div><br /><div><strong>To: </strong>meven, ivan, Frameworks, kossebau<br /><strong>Cc: </strong>kossebau, kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns<br /></div>