<table><tr><td style="">sredman marked an inline comment as done.<br />sredman added inline comments.
</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/D5941" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D5941#inline-24314" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">apol</span> wrote in <span style="color: #4b4d51; font-weight: bold;">CMakeLists.txt:16</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Ugh no, don't do that. Rename it if you want, but don't pass the name with a variable.<br />
Better be explicit.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Thank you for the feedback! I have never used CMake before, so I have not seen the failure case using a variable for the outputs would cause. Do you have a horror story (which you would like to share), or is this just general good practice?</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D5941#inline-24315" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">apol</span> wrote in <span style="color: #4b4d51; font-weight: bold;">connection.cpp:175</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">You added one comment in the cpp and the other on the header. Please do the same on both.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Do you mean I should have, for method-header comments, the same thing in both the .h and .cpp files? The reason I don't want to do this (at least not at present) is that everything is changing quite rapidly. It would be hard to keep the two sets of comments in sync (and they would surely get out of sync)</p>

<p style="padding: 0; margin: 8px;">To that end, I have been keeping all my header comments in the .cpp file. The comment in the .h file for the messageReceived(..) signal is only in the .h file because it doesn't appear in the .cpp</p>

<p style="padding: 0; margin: 8px;">In general, I like using something like doxygen to keep track of comments regardless. This solves the problem of wondering whether a header comment belongs in the source or header file, because it is easier and nicer-looking to refer to the doxygen output in any case. I have added a Doxyfile (and the related output to the .gitignore)</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R716 Telepathy KDE Connect Integration</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5941" rel="noreferrer">https://phabricator.kde.org/D5941</a></div></div><br /><div><strong>To: </strong>sredman, kdeconnect, davidedmundson<br /><strong>Cc: </strong>apol<br /></div>