Fixed random SEGV in Konqueror caused by klipper when the clipboard was being cleared. #178

Συγχωνευμένα
MicheleC συγχώνευσε 1 υποβολές από issue/147 σε master 3 έτη πριν
MicheleC σχολίασε 4 έτη πριν
Ιδιοκτήτης

This resolves issue #147.

This resolves issue #147.
MicheleC το πρόσθεσε στο R14.0.10 release ορόσημο 4 έτη πριν
SlavekB αξιολόγησε 4 έτη πριν
klipper/toplevel.cpp Παρωχημένο
// Using a direct call may crash another application that was
// changing the clipboard at the same time.
bSavedSelectionMode = selectionMode;
m_setClipboardTimer.start(0, TRUE);
SlavekB σχολίασε 4 έτη πριν
Ιδιοκτήτης

Could it be easier to use a singleShot timer call instead of creating a timer? Something like:

TQTimer::singleShot(0, this, TQT_SLOT(slotDelayedSetClipboard()));
Could it be easier to use a singleShot timer call instead of creating a timer? Something like: ``` TQTimer::singleShot(0, this, TQT_SLOT(slotDelayedSetClipboard())); ```
MicheleC σχολίασε 4 έτη πριν
Συντάκτης
Ιδιοκτήτης

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.

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.
SlavekB σχολίασε 4 έτη πριν
Ιδιοκτήτης

Yeah, thank you, that makes sense.

Yeah, thank you, that makes sense.
SlavekB σημείωση αυτή την συνομιλία ως επιλυμένη
MicheleC force-pushed issue/147 από το e588c507db στο 014d4c77c8 4 έτη πριν
MicheleC σχολίασε 4 έτη πριν
Συντάκτης
Ιδιοκτήτης

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.

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.
MicheleC force-pushed issue/147 από το 014d4c77c8 στο b7e3eef591 3 έτη πριν
MicheleC σχολίασε 3 έτη πριν
Συντάκτης
Ιδιοκτήτης

Added check for empty clipboard after the short delay, as discussed with Slavek.

Added check for empty clipboard after the short delay, as discussed with Slavek.
MicheleC force-pushed issue/147 από το b7e3eef591 στο e2158a0f0b 3 έτη πριν
MicheleC συγχώνευσε την υποβολή e2158a0f0b σε master 3 έτη πριν
MicheleC διέγραψε το κλάδο issue/147 3 έτη πριν
Το pull request έχει συγχωνευθεί ως e2158a0f0b.
Συνδεθείτε για να συμμετάσχετε σε αυτή τη συνομιλία.
Δεν υπάρχουν εξεταστές
Χωρίς Ορόσημο
Χωρίς Αποδέκτη
2 Συμμετέχοντες
Ειδοποιήσεις
Ημερομηνία Παράδοσης

Δεν ορίστηκε ημερομηνία παράδοσης.

Εξαρτήσεις

Δεν έχουν οριστεί εξαρτήσεις.

Αναφορά: TDE/tdebase#178
Φόρτωση…
Δεν υπάρχει ακόμα περιεχόμενο.