diff options
author | Gloria Wang <gwang@google.com> | 2011-07-15 11:53:17 -0700 |
---|---|---|
committer | Gloria Wang <gwang@google.com> | 2011-07-15 12:08:00 -0700 |
commit | 6fb40672ac0f0b2f479367fa0ba4e8944df06dad (patch) | |
tree | f1566031f2f4c2a9c6cccee5cc6bdb952decfbc9 /media/libstagefright/XINGSeeker.cpp | |
parent | 3c76eaebad1e420937088e52ec89efcb76bb875d (diff) | |
download | frameworks_av-6fb40672ac0f0b2f479367fa0ba4e8944df06dad.zip frameworks_av-6fb40672ac0f0b2f479367fa0ba4e8944df06dad.tar.gz frameworks_av-6fb40672ac0f0b2f479367fa0ba4e8944df06dad.tar.bz2 |
Use unsigned char to store table of contents retrieved from XING header.
Our compiler treats char default to unsigned (trygon-eng on master branch).
If I force "signed char" for the table of contents array, some MediaFramework
tests for VBR mp3 would fail.
Change-Id: I725e2d4dfe57ff36fdb1d65fe3184f7ab41ad695
Diffstat (limited to 'media/libstagefright/XINGSeeker.cpp')
-rw-r--r-- | media/libstagefright/XINGSeeker.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/media/libstagefright/XINGSeeker.cpp b/media/libstagefright/XINGSeeker.cpp index 0d0d6c2..2091381 100644 --- a/media/libstagefright/XINGSeeker.cpp +++ b/media/libstagefright/XINGSeeker.cpp @@ -24,8 +24,8 @@ namespace android { static bool parse_xing_header( const sp<DataSource> &source, off64_t first_frame_pos, int32_t *frame_number = NULL, int32_t *byte_number = NULL, - char *table_of_contents = NULL, int32_t *quality_indicator = NULL, - int64_t *duration = NULL); + unsigned char *table_of_contents = NULL, + int32_t *quality_indicator = NULL, int64_t *duration = NULL); // static sp<XINGSeeker> XINGSeeker::CreateFromSource( @@ -94,7 +94,7 @@ bool XINGSeeker::getOffsetForTime(int64_t *timeUs, off64_t *pos) { static bool parse_xing_header( const sp<DataSource> &source, off64_t first_frame_pos, int32_t *frame_number, int32_t *byte_number, - char *table_of_contents, int32_t *quality_indicator, + unsigned char *table_of_contents, int32_t *quality_indicator, int64_t *duration) { if (frame_number) { *frame_number = 0; |