Refactoriser MusicRenderer pour un comportement stateful complet #57

eric · 2026-01-16T20:57:38+01:00

eric commented

2026-01-16 20:57:38 +01:00

Cette modification refactorise l'architecture pour que chaque MusicRenderer gère son propre thread de surveillance (watcher), au lieu de déléguer le polling au ControlPoint centralisé.

Ajout d'un module watcher.rs avec WatchStrategy, WatchedState et fonctions helper
Implémentation de start_watching() et stop_watching() dans MusicRenderer
Centralisation de la gestion du watcher dans le constructeur et les méthodes has_been_seen_now()/mark_as_offline()
Suppression du polling central (~140 lignes) dans control_point.rs
Simplification du registry.rs avec suppression des appels manuels start/stop_watching()
Correction du bug dans refresh_device_presence() pour le traitement offline→online
Préparation pour le support futur des notifications push (OpenHome, Chromecast)

L'architecture est maintenant plus robuste avec une meilleure encapsulation, cohérence des événements et une gestion automatique du watcher.

Cette modification refactorise l'architecture pour que chaque `MusicRenderer` gère son propre thread de surveillance (watcher), au lieu de déléguer le polling au `ControlPoint` centralisé. - Ajout d'un module `watcher.rs` avec `WatchStrategy`, `WatchedState` et fonctions helper - Implémentation de `start_watching()` et `stop_watching()` dans `MusicRenderer` - Centralisation de la gestion du watcher dans le constructeur et les méthodes `has_been_seen_now()`/`mark_as_offline()` - Suppression du polling central (~140 lignes) dans `control_point.rs` - Simplification du `registry.rs` avec suppression des appels manuels `start/stop_watching()` - Correction du bug dans `refresh_device_presence()` pour le traitement offline→online - Préparation pour le support futur des notifications push (OpenHome, Chromecast) L'architecture est maintenant plus robuste avec une meilleure encapsulation, cohérence des événements et une gestion automatique du watcher.

eric added 1 commit 2026-01-16 20:57:39 +01:00

Refactoriser MusicRenderer pour un comportement stateful complet 02cff8e913

Cette modification refactorise l'architecture pour que chaque `MusicRenderer` gère son propre thread de surveillance (watcher), au lieu de déléguer le polling au `ControlPoint` centralisé. 

- Ajout d'un module `watcher.rs` avec `WatchStrategy`, `WatchedState` et fonctions helper
- Implémentation de `start_watching()` et `stop_watching()` dans `MusicRenderer`
- Centralisation de la gestion du watcher dans le constructeur et les méthodes `has_been_seen_now()`/`mark_as_offline()`
- Suppression du polling central (~140 lignes) dans `control_point.rs`
- Simplification du `registry.rs` avec suppression des appels manuels `start/stop_watching()`
- Correction du bug dans `refresh_device_presence()` pour le traitement offline→online
- Préparation pour le support futur des notifications push (OpenHome, Chromecast)

L'architecture est maintenant plus robuste avec une meilleure encapsulation, cohérence des événements et une gestion automatique du watcher.

eric merged commit cdd0e79be8 into main

2026-01-16 20:57:54 +01:00

eric referenced this issue from a commit

2026-01-16 20:57:56 +01:00

Merge pull request 'Refactoriser MusicRenderer pour un comportement stateful complet' (#57) from push-rkkrpwtnxvpx into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: eric/pmomusic#57