<table><tr><td style="">brauch 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/D9035" rel="noreferrer">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/D9035#173019" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D9035#173019</a>, <a href="https://phabricator.kde.org/p/wcancino/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@wcancino</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Yes, anyway I use that name because is consistent with the XDebugJob class (which uses the same mehtod name).</p></div>
</blockquote>

<p>Yes, but the new method doesn't do the same thing as the one in the other class. Thus the consistency is confusing instead of helpful, because the same name suggests the same functionality (which it does not have). If I see a method named "processError" I assume it is connected to the "error" signal of a QProcess or similar, which is the case in XDebugJob but not here. The "if error == QProcess::FailedToStart" is not helpful either, because that suggests it could be different (which it can't be).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R68 KDevelop XDebug Support</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9035" rel="noreferrer">https://phabricator.kde.org/D9035</a></div></div><br /><div><strong>To: </strong>wcancino, kfunk, brauch<br /><strong>Cc: </strong>brauch, kdevelop-devel<br /></div>