jamulussoftware / jamulussoftware/jamulus
Rewrite cclientdlg closeEvent, onAboutToQuit, ...
- Langage dominant
- C
- Étoiles
- 1.1k
- Forks
- 248
- Merge moyen
- 2 j 3 h
- PR mergées (30 j)
- 9
Description
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
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par les gestionnaires closeEvent et OnAboutToQuit de cclientdlg, puis examinez la discussion sur les méthodes de fermeture dans la pull request #3372. Suivez les chemins d’arrêt et les gestionnaires qui y sont mentionnés ; le travail est terminé lorsque le comportement de fermeture est organisé de manière cohérente et que les responsabilités d’arrêt existantes restent correctes.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- c
- Domaine
- desktop
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 30/100