Review Request 128789: Fix checking of foreground process

Martin Tobias Holmedahl Sandsmark martin.sandsmark at kde.org
Sun Aug 28 13:29:55 UTC 2016


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

(Updated Aug. 28, 2016, 1:29 p.m.)


Review request for Konsole and Kurt Hindenburg.


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 (updated)
-----

  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/4f453bf7/attachment.html>


More information about the konsole-devel mailing list