Looks good.

=======================================
Eric Hyche (ehyche at real.com)
Principal Engineer
RealNetworks, Inc.

>-----Original Message-----
>From: datatype-dev-bounces at helixcommunity.org [mailto:datatype-dev-bounces at helixcommunity.org] On
>Behalf Of Varun Kathuria
>Sent: Thursday, December 11, 2008 5:06 AM
>To: datatype-dev at helixcommunity.org
>Subject: [datatype-dev] CR: Changes to support mp3 with encode type(.mp3) in mov container
>
>Synopsis:
>
>Changes to support mp3 in mov container
>
>
>
>Overview:
>
>Added encode type for mp3 i.e (.mp3) in QTKnownAtomType.
>
>
>
>Files Added:
>None
>
>
>
>Files Removed
>
>None
>
>
>
>Files Modified:
>
>/cvsroot/datatype/mp4/fileformat/qtatmmgs.cpp
>
>/cvsroot/datatype/mp4/fileformat/pub/qtatoms.h
>
>
>
>Image Size and Heap Use impact (Client -Only):
>None.
>
>
>
>Platforms and Profiles Affected:
>
>None
>
>
>
>Distribution Libraries Affected:
>None
>
>
>
>Distribution library impact and planned action:
>None
>
>
>
>Platforms and Profiles Build Verified:
>
>BIF branch   -> hxclient_3_1_0_atlas_restricted
>
>Target(s)      -> splay
>
>Profile          -> helix-client-all-defines
>
>SYSTEM_ID -> win32-i386-vc7
>
>
>
>Branch:
>HEAD, ATLAS310
>
>
>
>Files Attached:
>
>diff.txt
>
>
>
>Thanks & Regards
>Varun Kathuria