jamulussoftware / jamulussoftware/jamulus

Rewrite cclientdlg closeEvent, onAboutToQuit, ...

Ouverte
#3,402 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
refactoring
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.