Show valid subprocess CWD for %D tab title format
ClosedPublic

Authored by mglb on Sun, Dec 30, 3:08 AM.

Details

Summary

%D did work only with main process - for every subprocess it just
displayed "-". With the patch "-" is displayed only in processes for
which CWD can't be read (e.g. root processes).

Test Plan
  • Use %D in tab title format
  • Run vim
  • (optionally) run :cd /usr in vim

Actual result: "-" is shown in the title instead of %D
Expected result: valid CWD is shown in the title instead of %D

Diff Detail

Repository
R319 Konsole
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mglb requested review of this revision.Sun, Dec 30, 3:08 AM
mglb created this revision.
hindenburg accepted this revision.Sun, Dec 30, 10:58 PM
hindenburg added a subscriber: hindenburg.

Looks fine - thanks - I notice this seems to fix %u which hasn't worked for a while it seems

This revision is now accepted and ready to land.Sun, Dec 30, 10:58 PM
This revision was automatically updated to reflect the committed changes.