[prev in list] [next in list] [prev in thread] [next in thread]
List: helix-common-dev
Subject: RE: [Common-dev] CR: symbian asf file recognition
From: <Ashish.As.Gupta () nokia ! com>
Date: 2007-07-27 19:23:17
Message-ID: 60A427E0B02CC34B98D22BDCAA2536120480BA59 () daebe101 ! NOE ! Nokia ! com
[Download RAW message or body]
Thanks Greg. This change has been checked in to Head & 210CayS.
Ashish
-----Original Message-----
From: ext Greg Wright [mailto:gwright@real.com]
Sent: Thursday, July 26, 2007 7:07 PM
To: Gupta Ashish.As (Nokia-TP-MSW/Dallas)
Cc: common-dev@helixcommunity.org
Subject: Re: [Common-dev] CR: symbian asf file recognition
Looks good.
--greg.
Ashish.As.Gupta@nokia.com wrote:
> ASF file can't be scanned into music library
>
> Nokia submits this code under the terms of a commercial contribution
> agreement with RealNetworks, and I a m authorized to contribute this
> code under said agreement."
>
> Modified by: ashish.as.gupta@nokia.com
>
> Reviewed by:
>
> Date: 25-July-2007
>
> Project: SymbianMmf_Rel
>
> ErrorId: EJZO-73JEWY
>
> Synopsis: ASF file can't be scanned into music library
>
> Files Modified:
>
> Problem is that the system ASF recognizer is not working correctly.
> This CR adds the logic to recognize ASF files based on unique GUID.
> This will be used by metadata engine to enable the recognition of ASF
> files. Further, asfff.dll will be used to parse the file to
> distinguish between audio and video.
>
> Please note that this only partial fix. It also depends on the
> corresponding changes in the recognizer to use the metadata utility to
> recognize the ASF files.
>
> common/system/platform/symbian/symbianrecognizer.cpp
>
> Files Added: None
>
> Image Size and Heap Use impact: no major impact
>
> Module Release Testing (STIF) : MRT test cases run and passed.
>
> Root Cause: new feature/implementation
>
> Test case(s) Added : None. File recognition is not done through MMF
> APIs.
>
> Memory leak check performed : Yes. No new leaks introduced.
>
> Platforms and Profiles Build Verified: helix-client-s60-32-mmf-mdf-arm
>
> Platforms and Profiles Functionality verified: armv5, winscw
>
> Branch: Head & 210CayS
>
> ? common_system_platform_symbian_diff.txt
> Index: symbianrecognizer.cpp
> ===================================================================
> RCS file:
> /cvsroot/common/system/platform/symbian/symbianrecognizer.cpp,v
> retrieving revision 1.2.156.2
> diff -u -w -r1.2.156.2 symbianrecognizer.cpp
> --- symbianrecognizer.cpp 19 Sep 2006 21:04:35 -0000
> 1.2.156.2
> +++ symbianrecognizer.cpp 26 Jul 2007 22:56:42 -0000
> @@ -119,6 +119,39 @@
> return bRet;
> }
>
> +static HXBOOL IsASFHeader(const TDesC& fileName,
> + const TDesC8& fileBuf,
> + TDataType &type) {
> + // known GUID header for ASF file.
> +
> + _LIT8(KASF_HEADER_GUID,
> +
> "\x30\x26\xB2\x75\x8E\x66\xCF\x11\xA6\xD9\x00\xAA\x00\x62\xCE\x6C");
> +
> + // mime type to be returned.
> + _LIT8(KASFMimeType , "application/vnd.ms-asf" );
> +
> + if (fileBuf.Length() < 16)
> + {
> + return FALSE;
> + }
> +
> +
> + HXBOOL bRet = TRUE;
> +
> + TPtrC8 buffer = fileBuf.Mid(0, 16);
> + if (buffer.Compare(KASF_HEADER_GUID) == 0)
> + {
> + type = TDataType(KASFMimeType);
> + }
> + else
> + {
> + bRet = FALSE;
> + }
> + return bRet;
> +}
> +
> +
> static TInt DoGetMimeType(const TDesC& fileName,
> const TDesC8& fileBuf,
> TDataType &type, @@ -134,6 +167,10 @@
> {
> return ret;
> }
> + else if (IsASFHeader(fileName, fileBuf, type))
> + {
> + return ret;
> + }
> else
> {
> // check if system recognizer has been disabled.
>
>
>
>
>
> ----------------------------------------------------------------------
> --
>
> _______________________________________________
> Common-dev mailing list
> Common-dev@helixcommunity.org
> http://lists.helixcommunity.org/mailman/listinfo/common-dev
_______________________________________________
Common-dev mailing list
Common-dev@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/common-dev
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic