<table><tr><td style="">kossebau 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/D14484">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/D14484#inline-76415">View Inline</a><span style="color: #4b4d51; font-weight: bold;">arrowd</span> wrote in <span style="color: #4b4d51; font-weight: bold;">test_gdb.cpp:982</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Was just following what <tt style="background: #ebebeb; font-size: 13px;">SKIP_IF_ATTACH_FORBIDDEN();</tt> does.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">... which though inside <tt style="background: #ebebeb; font-size: 13px;">KDevMI::isAttachForbidden(...)</tt> calls <tt style="background: #ebebeb; font-size: 13px;">QTest::qSkip(...)</tt> :)<br />
Was also initially confused by the name of <tt style="background: #ebebeb; font-size: 13px;">KDevMI::isAttachForbidden(...)</tt> but it is actually from the testhelper part. Someone(tm) should improve that for some explicit namespace or method name...</p>

<p style="padding: 0; margin: 8px;">For the text, i would propose to be more explicit "GDB on FreeBSD produces an unexpected error message, on which KDevelop chokes") and perhaps leave the link to the bug report only as normal comment in the code, not sure how nice multi-line skip messages are in the output?</p>

<p style="padding: 0; margin: 8px;">Please note the actual error directly in the sources, so one does not have to hunt for the commit message or open a browser to read the bug report.</p>

<p style="padding: 0; margin: 8px;">Long nitpicking, let me give a short example how I personally would prefer it: :)</p>

<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);">#ifdef Q_OS_FREEBSD
    // Despite successful attach GDB MI spits out a error message "Can't allocate registers". This gets caught by KDevMI layer and gets interpreted as error.
    // PR: https://sourceware.org/bugzilla/show_bug.cgi?id=23464
    QSKIP("GDB on FreeBSD produces an unexpected error message, on which KDevelop chokes");
#endif</pre></div>

<p style="padding: 0; margin: 8px;">That should serve the purpose of keeping the test log short. And allow any reader/editor of the code to directly see what the issue is. What do you think?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14484">https://phabricator.kde.org/D14484</a></div></div><br /><div><strong>To: </strong>arrowd, KDevelop<br /><strong>Cc: </strong>kossebau, kdevelop-devel, antismap, iodelay, vbspam, njensen, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>