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

List:       helix-player-dev
Subject:    RE: FW: [Player-dev] [Helix-Player-dev] CR: Add get plugin list for
From:       "Zhao, Halley" <halley.zhao () intel ! com>
Date:       2007-10-30 0:50:38
Message-ID: 094BCE01AFBE9646AF220B0B3F367AAB01503631 () pdsmsx413 ! ccr ! corp ! intel ! com
[Download RAW message or body]

¡¡¡¡Hi Greg, 
¡¡¡¡This module doesn't use HXLOG; and we don't expect it terminate the process by \
assert(). It is a warning. ¡¡¡¡How about use g_warning() instead?
¡¡¡¡Thanks.
¡¡¡¡
¡¡¡¡
¡¡¡¡>-----Original Message-----
¡¡¡¡>From: Greg Wright [mailto:gwright@real.com]
¡¡¡¡>Sent: 2007Äê10Ô 30ÈÕ 2:25
¡¡¡¡>To: Zhao, Halley
¡¡¡¡>Cc: ehyche@real.com; player-dev@helixcommunity.org
¡¡¡¡>Subject: Re: FW: [Player-dev] [Helix-Player-dev] CR: Add get plugin list
¡¡¡¡>for Helixdbus-server
¡¡¡¡>
¡¡¡¡>+                        printf("    ----WARNING----, there are so many
¡¡¡¡>plugins that the buffer may overflow\n");
¡¡¡¡>+                        printf("    ----WARNING----, we just list part
¡¡¡¡>of these plugins\n");
¡¡¡¡>+                        break;
¡¡¡¡>
¡¡¡¡>I am not sure 'printf' is the best way to output a
¡¡¡¡>warning. Perhaps use an HX_ASSERT or assert(). If
¡¡¡¡>this module uses HXLOG then that would be good too.
¡¡¡¡>
¡¡¡¡>--greg.
¡¡¡¡>
¡¡¡¡>
¡¡¡¡>
¡¡¡¡>Zhao, Halley wrote:
¡¡¡¡>> ¡¡¡¡Hi Greg & Eric.
¡¡¡¡>> ¡¡¡¡After fix the issue you raise, Could you help me review the patch?
¡¡¡¡>> ¡¡¡¡Thanks.
¡¡¡¡>>
¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡-----Original Message-----
¡¡¡¡>> From: player-dev-bounces@helixcommunity.org
¡¡¡¡>[mailto:player-dev-bounces@helixcommunity.org] On Behalf Of Zhao, Halley
¡¡¡¡>> Sent: 2007Äê10Ô 26ÈÕ 14:29
¡¡¡¡>> To: Greg Wright
¡¡¡¡>> Cc: Shen, Cathy; helix-client-dev@helixcommunity.org;
¡¡¡¡>player-dev@helixcommunity.org
¡¡¡¡>> Subject: RE: [Player-dev] [Helix-Player-dev] CR: Add get plugin list for
¡¡¡¡>Helixdbus-server
¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡Thanks for Greg and Eric to point out some mistakes.
¡¡¡¡>> ¡¡¡¡¡¡¡¡Resend the patch after fix the issues.
¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡BR.
¡¡¡¡>>
¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>-----Original Message-----
¡¡¡¡>> ¡¡¡¡¡¡¡¡>From: Greg Wright [mailto:gwright@real.com]
¡¡¡¡>> ¡¡¡¡¡¡¡¡>Sent: 2007Äê10Ô 26ÈÕ 1:12
¡¡¡¡>> ¡¡¡¡¡¡¡¡>To: Zhao, Halley
¡¡¡¡>> ¡¡¡¡¡¡¡¡>Cc: player-dev@helixcommunity.org; Shen, Cathy;
¡¡¡¡>> ¡¡¡¡¡¡¡¡>helix-client-dev@helixcommunity.org
¡¡¡¡>> ¡¡¡¡¡¡¡¡>Subject: Re: [Player-dev] [Helix-Player-dev] CR: Add get plugin
¡¡¡¡>list for
¡¡¡¡>> ¡¡¡¡¡¡¡¡>Helix dbus-server
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>In addition to Eric's comments:
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>+    printf("***
¡¡¡¡>helix_dbus_server_handle_get_plugin_list\n");
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>we don't need that.
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>+    // helix_dbus_server_free(server);
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>You SURE you want that commented out? If it is freed somewhere
¡¡¡¡>> ¡¡¡¡¡¡¡¡>else then we don't need that line at all.
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>+
¡¡¡¡>>
¡¡¡¡>>retVal=pUnkReloadPlugins->QueryInterface(IID_IHXPluginReloader,(void**
¡¡¡¡>)
¡¡¡¡>> ¡¡¡¡¡¡¡¡>&pPluginReloader);
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>You do not call HX_RELEASE(pPluginReloader) anywhere, that is
¡¡¡¡>a leak.
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>+        g_return_val_if_fail(pPluginReloader&&
¡¡¡¡>> ¡¡¡¡¡¡¡¡>SUCCEEDED(retVal),NULL);
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>It is probably OK in this case, but you do not initialize
¡¡¡¡>pPluginReloader
¡¡¡¡>> ¡¡¡¡¡¡¡¡>to NULL and then use a non-NULL value as an success condition.
¡¡¡¡>We
¡¡¡¡>> ¡¡¡¡¡¡¡¡>generally like to NULL those variables out in that case.
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>+        retVal =
¡¡¡¡>>
¡¡¡¡>>pUnk->QueryInterface(IID_IHXPlugin2Handler,(void**)&pPlugin2Handler);
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>You do not call HX_RELEASE(pPlugin2Handler) anywhere, that is
¡¡¡¡>a memory
¡¡¡¡>> ¡¡¡¡¡¡¡¡>leak. You also do not initialize pPlugin2Handler to NULL, like
¡¡¡¡>above.
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>+            retVal =
¡¡¡¡>pPluginProps->GetFirstPropertyCString(szPropName,
¡¡¡¡>> ¡¡¡¡¡¡¡¡>pPropValue);
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>You do not call HX_RELEASE(pPropValue), that is a memory leak.
¡¡¡¡>You
¡¡¡¡>> ¡¡¡¡¡¡¡¡>need to call HX_RELEASE(pPropValue) after each call to
¡¡¡¡>> ¡¡¡¡¡¡¡¡>GetFirstPropertyCString when you are done using pPropValue and
¡¡¡¡>> ¡¡¡¡¡¡¡¡>before you call GetFirstPropertyCString again. You don't have
¡¡¡¡>> ¡¡¡¡¡¡¡¡>to worry about szPropName.
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>--greg.
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>Zhao, Halley wrote:
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> Synopsis:
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>     Add get plugin list for Helix dbus-server.
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> Overview:
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>     Media application needs to know the media engine's
¡¡¡¡>capability (which
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> datatypes are supported by the engine).
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>     Add this method for Helix dbus-server
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> Files Added:
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>        No file added
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> Files Modified:
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>     helix-dbus-server.cpp:
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> (player/dbus-server/src/helix-dbus-server.cpp)
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>         dbus message handles
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>     hxplayer.cpp:
¡¡¡¡>(player/kit/dbus-server/src/hxplayer.cpp)
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>         Add the method implementation
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>     hxplayer.h: (player/kit/dbus-server/pub/hxplayer.h)
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>         method declaration
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>     Umakefil: (player/kit/dbus-server/Umakefil)
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>         Add include folder for the build
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> 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:
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>     helix-dbus-server-GetPluginCaps.diff
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> ZHAO, Halley (Aihua)
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> Email: halley.zhao@intel.com <mailto:aihua.zhao@intel.com>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> Tel: +86(21)61166476
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> iNet: 8821-6476
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> SSG/OTC/UMD
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>>
¡¡¡¡>>----------------------------------------------------------------------
¡¡¡¡>-
¡¡¡¡>> ¡¡¡¡¡¡¡¡>-
¡¡¡¡>> ¡¡¡¡¡¡¡¡>>
¡¡¡¡>> ¡¡¡¡¡¡¡¡>> This body part will be downloaded on demand.

_______________________________________________
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]