[Differential] [Updated] D4095: Refactor to embrace a cmake server import backend

[Aleix Pol Gonzalez]

apol marked 4 inline comments as done.
apol added inline comments.

INLINE COMMENTS

> mwolff wrote in cmakeserver.cpp:64
> what if the first connect fails - we simply get an error and disconnect and don't try thereafter, right? couldn't we retry for up to N times (say every 100ms for up to 1s?) to make this a bit less brittle?

FWIW, this is the code in QtCreator:

  m_connectionTimer.setInterval(100);
  connect(&m_connectionTimer, &QTimer::timeout, this, &ServerMode::connectToServer);

I'd prefer doing the same as them and discussing with Tobias how to improve it, as he did cmake-server implementation as well as qtcreator.

REPOSITORY
  R32 KDevelop

REVISION DETAIL
  https://phabricator.kde.org/D4095

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: apol, kfunk, mwolff, #kdevelop
Cc: mwolff, antonanikin, arrowdodger, kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170130/c6df1e0c/attachment.html>