D11235: [DrKonqi] Request change of ptrace scope from KCrash
Oswald Buddenhagen
noreply at phabricator.kde.org
Wed Jan 16 15:52:04 GMT 2019
ossi requested changes to this revision.
ossi added inline comments.
This revision now requires changes to proceed.
INLINE COMMENTS
> ptracer.cpp:65
> + bytes += r;
> + else if (r == -1 && !(errno == EINTR))
> + break;
you really could just use != here ... ^^
> ptracer.cpp:78
> + bytes += r;
> + else if (r == -1 && !(errno == EINTR))
> + break;
ditto
> ptracer.cpp:82
> + if (bytes == msize && memcmp(msg, rmsg, msize) == 0)
> + qCInfo(DRKONQI_LOG) << "ptracer set to" << debuggerpid << "by debugged process";
> + }
a complementary warning if any of the steps goes wrong would seem appropriate.
> crashtest.cpp:139
>
> - //start drkonqi directly so that drkonqi's output goes to the console
> + //start drkonqi directly by default so that drkonqi's output goes to the console
> KCrash::CrashFlags flags = KCrash::AlwaysDirectly;
add comment: this can be disabled to be able to test kcrash's real default behavior.
> crashtest.cpp:143
> flags |= KCrash::AutoRestart;
> + if (parser.isSet(QStringLiteral("kdeinit")))
> + flags &= ~KCrash::AlwaysDirectly;
move that up, so the AlwaysDirectly stuff is in one block.
REPOSITORY
R871 DrKonqi
REVISION DETAIL
https://phabricator.kde.org/D11235
To: croick, #plasma_workspaces, #frameworks, ossi
Cc: ossi, lepagevalleeemmanuel, maximilianocuria, adridg, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190116/1d12e6ba/attachment.html>
More information about the Kde-frameworks-devel
mailing list