<table><tr><td style="">vkrause 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/D24929">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/D24929#557692" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D24929#557692</a>, <a href="https://phabricator.kde.org/p/ahmadsamir/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ahmadsamir</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Yep, I agree, it is ugly; I spent some time trying to use several methods (ternaries and such) but came back empty handed; I'm genuinely interested in how this can be made smarter :)</p></div>
</blockquote>

<p>Maybe something like this:</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);">QNetworkProxy proxy;
proxy.setType(QNetworkProxy::NoProxy);
if (QSslSocket *sock = qobject_cast<...>(...)) {
    sock->setProxy(proxy);
} else if (KTCpSocket *sock = qobject_cast<...>(...)) {
    sock->setProxy(proxy);
} else {
   qWarning()
}</pre></div>

<p>Limits nesting depth to 1 on the ifs, avoids the nested boolean condition in the final if altogether, reduces scope of the variables and avoids the duplicated proxy code, and makes the eventual removal of the KTcpSocket branch fairly simple.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R44 KDE PIM Runtime</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24929">https://phabricator.kde.org/D24929</a></div></div><br /><div><strong>To: </strong>ahmadsamir, mlaurent<br /><strong>Cc: </strong>vkrause, kde-pim, fbampaloukas, dvasin, rodsevich, winterz, mlaurent, knauss, dvratil<br /></div>