[Differential] [Updated] D2293: Merge LLDB plugin into master

apol (Aleix Pol Gonzalez) noreply at phabricator.kde.org
Thu Jul 28 23:31:52 UTC 2016


apol added a comment.


  :)

INLINE COMMENTS

> micommand.cpp:286
>              break;
> -        case FileListExecSections:
> +        case FileListExecSections: // FIXME: non-exist command
>              command = "file-list-exec-sections";

What does that `non-exist command` mean?

It only exists in gdb? On neither?

> midebuggerplugin.h:144
>          work, but they don't need any toolbar.  So, suppress toolbar action.  */
> +    /*
>      QList<QAction*> toolBarActions(QWidget* viewWidget) const override

Please don't leave commented, boilerplate, code.

> TODO.txt:102
> +        + lldb-mi crashes when break on a point where multiple threads running. (Fixed in latest lldb trunk version)
> +
> +relavent lldb settings:

Mention //attach to process// debugging at least?

REPOSITORY
  rKDEVELOP KDevelop

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

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

To: qi437103, #kdevelop, apol
Cc: kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160728/8995d0a2/attachment.html>


More information about the KDevelop-devel mailing list