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

  • From: Dario Casalinuovo <b.vitruvio@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Mon, 4 Apr 2016 15:39:39 +0200

Hi,

On Mon, Apr 4, 2016 at 12:04 PM, Jérôme Duval <jerome.duval@xxxxxxxxx>
wrote:

2016-04-04 1:29 GMT+02:00  <b.vitruvio@xxxxxxxxx>:
diff --git a/src/servers/media/Jamfile b/src/servers/media/Jamfile
index c5edb71..aa638ee 100644
--- a/src/servers/media/Jamfile
+++ b/src/servers/media/Jamfile
@@ -2,7 +2,7 @@ SubDir HAIKU_TOP src servers media ;

 SetSubDirSupportedPlatformsBeOSCompatible ;

-UsePrivateHeaders media shared storage ;
+UsePrivateHeaders media shared storage app ;

This should be ordered alphabetically.

diff --git a/src/servers/media/media_server.cpp
b/src/servers/media/media_server.cpp
index f4d7dd6..9ed51d5 100644
--- a/src/servers/media/media_server.cpp
+++ b/src/servers/media/media_server.cpp
@@ -37,7 +37,7 @@ char __dont_remove_copyright_from_binary[] =
"Copyright (c) 2002, 2003 "
 #include <string.h>

 #include <Alert.h>
-#include <Application.h>
+#include <Server.h>
 #include <Autolock.h>
 #include <Directory.h>
 #include <Roster.h>

This should be ordered alphabetically.


Do you believe that I fixed them already in my branch but forgot to update
the patch in the end? :-)




SyncToServices seems like something we want implemented as media
notifications [1]
Maybe you just overlooked the TODO right after where you added a
SyncToServices() call [2]


I've added it myself, I've been planning to add something like that since
then but the idea was to don't make it public.


And besides, in case of no audio output available, it looks like the
SyncToServices() call would lock forever, or I miss something.


The function allow to specify a timeout, it made sense to me to implement
it this way.

Other related posts:

  1. Home
  2. haiku-commits
  3. Archive
  4. 04-2016