[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: Mon, 2 May 2016 12:47:39 +0200

Hi,

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

Am 01/05/2016 um 11:42 schrieb Dario Casalinuovo:

 * 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.


It wasn't about the implementation, it was about the need for it.

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?


Who would be interested in that? And why? Do you have any particular use
case for this? I cannot think of anything useful.


I might be in favor of putting it in private API, but how do you suggest to
solve then the shutdown_media_server notifications problem without
something like SyncToServer?




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.


Anybody could restart the server before, too, what's launch_daemon
specific about this? And what's the deal, anyway? Where is the difference
when you call launch_media_server() manually?


Is this really a question? I'm surprised how you express things to be
obvious but don't understand the meaning of that? launch_daemon is
obviously the major and most probable of messing with the restart service,
but not the only. Since the introduction of the launch_daemon I'm not sure
we need a launch_media_server anymore.

Other related posts: