Review Request 128789: Fix checking of foreground process

Martin Tobias Holmedahl Sandsmark martin.sandsmark at kde.org
Sun Aug 28 17:18:21 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128789/
-----------------------------------------------------------

(Updated Aug. 28, 2016, 10:18 a.m.)


Status
------

This change has been marked as submitted.


Review request for Konsole and Kurt Hindenburg.


Changes
-------

Submitted with commit 79ca76a94696eda408927b122b0b531676d4ba93 by Martin T. H. Sandsmark to branch master.


Bugs: 367746
    http://bugs.kde.org/show_bug.cgi?id=367746


Repository: konsole


Description
-------

The old method of checking it has unnecessary overhead (doing a full process info update), and wasn't very reliable.

Instead just get the original shell PID from QProcess (via KProcess), and the foreground process from the PTY.


Diffs
-----

  src/MainWindow.cpp 6f56014 
  src/Session.cpp 14a0fbb 

Diff: https://git.reviewboard.kde.org/r/128789/diff/


Testing
-------

It fixes https://bugs.kde.org/show_bug.cgi?id=367746 for me, also added debug output to print the shell PID and the current process PID to verify that it acted correctly when doing stuff.


Thanks,

Martin Tobias Holmedahl Sandsmark

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20160828/e67e0ccc/attachment.html>


More information about the konsole-devel mailing list