jamulussoftware / jamulussoftware/jamulus

Rewrite cclientdlg closeEvent, onAboutToQuit, ...

Aperta
#3,402 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
refactoring
Lingua principale
C
Stelle
1.1k
Fork
248
Merge medio
2g 3h
PR unite (30g)
9

Descrizione

There was a lot of discussion on closing methods in cclientdlg in https://github.com/jamulussoftware/jamulus/pull/3372 This should probably be refactored.

> Maybe `closeEvent` should just call `QCoreApplication::instance()->exit();` and then do the work in `OnAboutToQuit` handlers? That should stop needing to call `XYZ.Stop()`.

> Of course, if that's what the "default implementation of this event handler routine" does, it would move all the code here into `ClientDlg.OnAboutToQuit` (plus handlers in each called class).

_Originally posted by @pljones in https://github.com/jamulussoftware/jamulus/pull/3372#discussion_r1763615127_, https://github.com/jamulussoftware/jamulus/pull/3372#discussion_r1763580248, https://github.com/jamulussoftware/jamulus/pull/3372#discussion_r1759707896

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia dagli handler closeEvent e OnAboutToQuit di cclientdlg, poi esamina la discussione sui metodi di chiusura nella pull request #3372. Traccia i percorsi di arresto e gli handler menzionati; il lavoro è completato quando il comportamento di chiusura è organizzato in modo coerente e le responsabilità di arresto esistenti rimangono corrette.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
c
Ambito
desktop
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
30/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.