vcl/unx/gtk3_kde5/gtk3_kde5_filepicker_ipc.cxx | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
New commits: commit 5ad88c0d8a713da33b1b43a01b08cabf858b1303 Author: Michael Weghorn <m.wegh...@posteo.de> AuthorDate: Tue Apr 16 16:48:58 2019 +0200 Commit: Thorsten Behrens <thorsten.behr...@cib.de> CommitDate: Wed Apr 17 13:03:20 2019 +0200 gtk3_kde5: No longer kill fpicker process on timeout In some observed cases, 100 ms were not enough for the separate 'lo_kde5filepicker' process to terminate properly, in particular on slow or busy hardware. Drop the timeout here and just wait for the process to finish to avoid killing it unnecessarily, which could e.g. lead to file dialog settings/state not properly being saved by the native KDE Plasma QFileDialog (plasma-integration, e.g. in 'KDEPlatformFileDialogHelper::saveSize'), so that the last used mode or size would not be remembered. The assumption is that the separate fpicker process should now properly finish in all cases, in particular since commit 93815c2b04f1905e43c695caf5cc2c594bb897ce. Should that assumption not hold, let's rather fix the underlying issues than increasing the timeout here. Change-Id: I0a58c1425940fbfaecb5012a24fd9a0222cbaccb Reviewed-on: https://gerrit.libreoffice.org/70834 Tested-by: Jenkins Reviewed-by: Michael Weghorn <m.wegh...@posteo.de> (cherry picked from commit 4640575bd33c2d1010f3acd4fdf3cce510091716) Reviewed-on: https://gerrit.libreoffice.org/70866 Reviewed-by: Thorsten Behrens <thorsten.behr...@cib.de> Tested-by: Thorsten Behrens <thorsten.behr...@cib.de> diff --git a/vcl/unx/gtk3_kde5/gtk3_kde5_filepicker_ipc.cxx b/vcl/unx/gtk3_kde5/gtk3_kde5_filepicker_ipc.cxx index 36ddcaf40f6e..d873789be25f 100644 --- a/vcl/unx/gtk3_kde5/gtk3_kde5_filepicker_ipc.cxx +++ b/vcl/unx/gtk3_kde5/gtk3_kde5_filepicker_ipc.cxx @@ -139,9 +139,7 @@ Gtk3KDE5FilePickerIpc::~Gtk3KDE5FilePickerIpc() return; sendCommand(Commands::Quit); - TimeValue timeValue(std::chrono::milliseconds(100)); - if (osl_joinProcessWithTimeout(m_process, &timeValue) != osl_Process_E_None) - osl_terminateProcess(m_process); + osl_joinProcess(m_process); if (m_inputWrite) osl_closeFile(m_inputWrite); _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits