[prev in list] [next in list] [prev in thread] [next in thread] 

List:       helix-player-dev
Subject:    [Player-dev] RE: [Helix-client-dev] CR: Shutdown Helix dbus-server
From:       "Eric Hyche" <ehyche () real ! com>
Date:       2007-12-17 13:47:28
Message-ID: 000201c840b3$5e34c760$db68a8c0 () EHYCHED620
[Download RAW message or body]


Looks good.

=============================================
Eric Hyche (ehyche@real.com)
Technical Lead
RealNetworks, Inc.  

> -----Original Message-----
> From: helix-client-dev-bounces@helixcommunity.org 
> [mailto:helix-client-dev-bounces@helixcommunity.org] On 
> Behalf Of Zhao, Halley
> Sent: Monday, December 17, 2007 8:05 AM
> To: helix-client-dev@helixcommunity.org
> Cc: Shen, Cathy; player-dev@helixcommunity.org
> Subject: [Helix-client-dev] CR: Shutdown Helix dbus-server 
> when Quit() called
> 
> Synopsis: 
> 
>     Since Helix D-Bus service could be auto launched when 
> media application try to connect it.
> 
>     when Quit() is called, destroy all the dbus service (not 
> just the player instance), 
> 
>     this can release more resource.
> 
>     Also, removed dbus_connection_close() because connections 
> created with dbus_connection_open()
> 
>      or dbus_bus_get() are shared. These connections are 
> owned by libdbus, and applications 
> 
>      should only unref them, never close them.
> 
>     
> 
> Overview: 
> 
>     Since Helix D-Bus service could be auto launched when 
> media application try to connect it.
> 
>     when Quit() is called, destroy all the dbus service (not 
> just the player instance), 
> 
>     this can release more resource.
> 
>     Also, removed dbus_connection_close() because connections 
> created with dbus_connection_open()
> 
>      or dbus_bus_get() are shared. These connections are 
> owned by libdbus, and applications 
> 
>      should only unref them, never close them.
> 
>         
> 
> Files Added:
> 
>        No file added
> 
> Files Modified:
> 
>     helix-dbus-server-main.cpp: 
> (player/kit/dbus-server/pub/helix-dbus-server-main.cpp)
> 
>         remove static property of function 
> on_helix_dbus_server_shutdown()
> 
>     helix-dbus-server.cpp: 
> (player/kit/dbus-server/src/helix-dbus-server.cpp)
> 
>         shutdown the D-Bus service when Quit() called.
> 
>         remove dbus_connection_close()
> 
>         
> 
> Image Size and Heap Use impact (Client -Only):
> 
>     little
> 
> Platforms and Profiles Affected:
> 
>     platform:   linux-2.2-libc6-gcc32-i586
> 
>     profile:    helix-client-all-defines
> 
>  
> 
> Distribution Libraries Affected:
> 
>     <helix-dbus-server.bin>
> 
> Distribution library impact and planned action:
> 
>     <None>
> 
> Platforms and Profiles Build Verified:
> 
>     Set BIF branch  -> hxdbus_3_1_0_atlas
> 
>     Set Target(s)   -> dbus_server_with_video
> 
>     Set Profile     -> helix-client-all-defines
> 
>     System ID       -> linux-2.2-libc6-gcc32-i586
> 
> Branch: 
> 
>     HEAD, hxclient_3_1_0_atlas
> 
> Copyright assignment: <MUST be one of the following statements >
> 
>    2.      Intel has signed and delivered a Joint Copyright Assignment
> 
>            to RealNetworks, and received acknowledgment that the 
> 
>            agreement was received. 
> 
> Files Attached:
> 
>     dbus-server-shutdown.diff
> 
>  
> 
> ZHAO, Halley (Aihua)
> 
> Email: halley.zhao@intel.com <mailto:aihua.zhao@intel.com> 
> 
> Tel: +86(21)61166476
> 
> iNet: 8821-6476
> 
> SSG/OTC/UMD
> 
>  
> 
> 


_______________________________________________
Player-dev mailing list
Player-dev@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/player-dev
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic