D11236: [KCrash] Establish socket to allow change of ptrace scope

David Faure noreply at phabricator.kde.org
Sun Oct 7 17:34:49 BST 2018


dfaure added a comment.


  This looks like the kind of commit where a review by Oswald Buddenhagen <ossi at kde.org> would be useful.

INLINE COMMENTS

> kcrash.cpp:672
> +                // wait while DrKonqi is running and the socket connection exists
> +                while (kill(pid, 0) >= 0 && pollDrKonqiSocket(pid, sockfd) >= 0);
> +                closeDrKonqiSocket(socketpath, sockfd);

Use {} rather than ';' for the body of the while loop, otherwise many tools (and humans...) will think the ';' is an error.

REPOSITORY
  R285 KCrash

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

To: croick, #frameworks
Cc: dfaure, lepagevalleeemmanuel, kde-frameworks-devel, sitter, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181007/f6d79716/attachment.html>


More information about the Kde-frameworks-devel mailing list