Thanks! Checked into atlas_361 and atlas_362
henry
________________________________
From: Qiang Luo
Sent: Friday, February 18, 2011 7:36 AM
To: Henry Xing
Cc: Sheldon Fu; datatype-dev at helixcommunity.org
Subject: Re: CR: helix can't play mp3 files which head larger than 1M

Looks OK.  Please define and use MAX_FILESYS_READSIZE.

Qiang

On 2/17/2011 3:25 AM, Henry Xing wrote:
Modified by: hxing at real.com<mailto:hxing at real.com>
Date: 02/17/2011
Project: RealPlayer for Android Smartphones

Synopsis: helix can't play mp3 file with about 7M picture data

Overview: Some mp3 files from lenovo have about 7M picture saved in ID3 tag, and we were requested to support it.  So adding the determination when reading file header, only read 1M data, others was ignored. Actually helix also ignore this head if we read this large head data.

Files Added: None

Files Modified:
datatype/mp3/fileformat/mp3ff.cpp

Image Size and Heap Use impact (Client -Only): None

Platforms and Profiles Affected:
Platform: android-2.2-arm-qsd_8250,
Profile: helix-client-android-full

Distribution Libraries Affected:None

Distribution library impact and planned action:None

Platforms and Profiles Build Verified:
Platform: android-2.2-arm-qsd_8250
Profile: helix-client-android-full

Platforms and Profiles Functionality verified:
Platform: android-2.2-arm-qsd_8250
Profile: helix-client-android-full

Branch: 310atlas, 361atlas,

Copyright assignment: I am a RealNetworks employee or contractor


-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.helixcommunity.org/pipermail/datatype-dev/attachments/20110217/4010b92e/attachment-0001.html