D4929: DrKonqi : lldb support

René J.V. Bertin noreply at phabricator.kde.org
Fri Mar 10 17:49:26 UTC 2017

rjvbb added inline comments.


> kfunk wrote in backtracegenerator.h:87
> Looks pretty unclean to have a backend-specific variable around here.

I suppose I could replace m_lldbDetached with something like

  bool Debugger::isDetached(const QString &outputLine=QString())
      if (!outputLine.isEmpty() && codeName() == "lldb") {
          QString line = outputLine.simplified();
          if (/*line contains "Process detached"*/) {
              m_isDetached = true;
      return m_isDetached;

and call that function instead of reading/setting m_lldbDetached.

Whether that's really less cumbersome I don't know?


To: rjvbb, #plasma_workspaces, kfunk
Cc: kfunk, mart, broulik, kde-mac, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-mac/attachments/20170310/95f08893/attachment.html>

More information about the kde-mac mailing list