https://bugs.kde.org/show_bug.cgi?id=459108
Lothar <loh....@googlemail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |loh....@googlemail.com
--- Comment #6 from Lothar <loh....@googlemail.com> ---
Hi, Eric pointed me to this report.
(In reply to Oded Arbel from comment #5)
> The problem appears to me to have been introduced in this commit:
> https://invent.kde.org/utilities/kate/-/commit/
> 1b93ede0e09fcc5ef069cf5c3be33e914c660162#1d89091b7718b0df550ed065e31436f1fab5
> 217c_1112_1123
> where the new code for KateMDI::Sidebar::saveSession assumes (without
> checking) that all items in the tab bar are already initialized tool views,
> but the problem is that the KateFileBrowser tool view - when it can't
> immediately identify the location from its configuration as a valid folder -
> starts a KIO stat job, which takes time to complete - and calls into the
> QEventLoop. The event loop may cause the SMC save_yourself API to be called
> to save the current session state - calling on KateMDI::Sidebar::saveSession
> to save the state of a partially initialized sidebar.
You seems to have located the issue very well, thank you. Will try to
reproduce the issue and fix it...
> I believe more error checking all around would be useful
...probably this way.
--
You are receiving this mail because:
You are watching all bug changes.
