[haiku-commits] Re: haiku: hrev50241 - src/kits/media headers/private/media headers/os/media

  • From: Dario Casalinuovo <b.vitruvio@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 1 May 2016 11:42:14 +0200

On Sun, May 1, 2016 at 10:22 AM, Axel Dörfler <axeld@xxxxxxxxxxxxxxxx>
wrote:

Am 21/04/2016 um 19:23 schrieb b.vitruvio@xxxxxxxxx:

  * Reintroduce BMediaRoster::SyncToServices, this time based on local
  message passing rather than a global semaphore.


Again, WHY? You didn't reply to my comments last time you added this, but
just reverted your changes. Now you add it again.


I don't recall everything of the discussion anyway the major problem was to
do it using a semaphore where this implementation is different.


Could you please enlighten me why exactly this should be necessary?


Do you expect anyone that want to know when the services are up, to create
a BLooper and start watching for this?



  * SyncToServices is used in launch_media_server to make the process
  more launch_daemon safe and faster in the average case.


What does "launch_daemon safe" mean?


Being aware that since it's introduction the services restart are handled
differently. A legacy app might call launch_media_server after
shutdown_media_server then this function allow to ask the BMediaRoster
about what's the situation.

Other related posts: