In D24150#536169, @dfaure wrote:

I (thought I) fixed this crash already in commit 512967f6f4e887d4a5a0 by removing the ::exit() call (which appears in the backtrace of that bug report).

This being said, I'm not objecting to this patch, but I am wondering if you can still experience this crash with a kio that includes my commit above (from Sep 6).

In D24150#536113, @aacid wrote:

why the void? do you get a warning otherwise?

No, but it is used in several places in the repository (and in Qt), so I suppose it might generate one depending on the compiler.

Also a comment in that line quoting the Qt docs "Note that you should not delete codecs yourself: once created they become Qt's responsibility." makes sense IMHO otherwise someone will move it back to the old code to not have a leak

OK, done (but the void should already show that not storing it in a variable, so not deleting it, is deliberate).

This change breaks the previous behavior when closing a tab : now focus goes to the next tab (or to the last tab when closing the last tab).
Previously, it was going to the previously focused one.
https://bugs.kde.org/show_bug.cgi?id=405213