Fixed random SEGV in Konqueror caused by klipper when the clipboard was being cleared. #178
Συγχωνευμένα
MicheleC
συγχώνευσε 1 υποβολές από issue/147 σε master 3 έτη πριν
Φόρτωση…
Αναφορά σε νέο ζήτημα
Δεν υπάρχει ακόμα περιεχόμενο.
Διαγραφή του Κλάδου 'issue/147'
Η διαγραφή του κλάδου είναι μόνιμη. ΔΕΝ ΜΠΟΡΕΙ να αναιρεθεί. Συνέχεια;
This resolves issue #147.
// Using a direct call may crash another application that was// changing the clipboard at the same time.bSavedSelectionMode = selectionMode;m_setClipboardTimer.start(0, TRUE);Could it be easier to use a singleShot timer call instead of creating a timer? Something like:
It could, I hadn't thought of that. But ultimately I think it would be less efficient. Single shot calls still create a timer internally and later destroy it once done. Since setting the clipboard could be a common operation in a user session, a permanent dedicated timer would have less overhead, being created only once.
Yeah, thank you, that makes sense.
e588c507dbστο014d4c77c84 έτη πρινWith the original code, there were still some crashes here and there. With the new commit, I have not been able to crash Konqueror any longer.
014d4c77c8στοb7e3eef5913 έτη πρινAdded check for empty clipboard after the short delay, as discussed with Slavek.
b7e3eef591στοe2158a0f0b3 έτη πρινe2158a0f0bσε master 3 έτη πρινe2158a0f0b.