pass of cosmetics

Change-Id: Id031221b7499e8cfc460c873d0f8124c9bef3aa3
This commit is contained in:
Pascal Massimino 2012-01-17 08:49:56 +00:00 committed by James Zern
parent 241ddd38e2
commit f0d2c7a76f

View File

@ -26,7 +26,7 @@ extern "C" {
// 4...7 size of image data (including metadata) starting at offset 8 // 4...7 size of image data (including metadata) starting at offset 8
// 8...11 "WEBP" our form-type signature // 8...11 "WEBP" our form-type signature
// The RIFF container (12 bytes) is followed by appropriate chunks: // The RIFF container (12 bytes) is followed by appropriate chunks:
// 12..15 "VP8 ": 4-bytes tags, describing the raw video format used // 12..15 "VP8 ": 4-bytes tags, signaling the use of VP8 video format
// 16..19 size of the raw VP8 image data, starting at offset 20 // 16..19 size of the raw VP8 image data, starting at offset 20
// 20.... the VP8 bytes // 20.... the VP8 bytes
// Or, // Or,
@ -50,8 +50,7 @@ VP8StatusCode WebPParseRIFF(const uint8_t** data, uint32_t* data_size,
assert(data_size); assert(data_size);
assert(riff_size); assert(riff_size);
if (*data_size >= RIFF_HEADER_SIZE && if (*data_size >= RIFF_HEADER_SIZE && !memcmp(*data, "RIFF", TAG_SIZE)) {
!memcmp(*data, "RIFF", TAG_SIZE)) {
if (memcmp(*data + 8, "WEBP", TAG_SIZE)) { if (memcmp(*data + 8, "WEBP", TAG_SIZE)) {
return VP8_STATUS_BITSTREAM_ERROR; // Wrong image file signature. return VP8_STATUS_BITSTREAM_ERROR; // Wrong image file signature.
} else { } else {
@ -483,17 +482,15 @@ static VP8StatusCode GetFeatures(const uint8_t* data, uint32_t data_size,
uint32_t flags = 0; uint32_t flags = 0;
uint32_t vp8x_skip_size = 0; uint32_t vp8x_skip_size = 0;
uint32_t vp8_skip_size = 0; uint32_t vp8_skip_size = 0;
int* const width = &features->width;
int* const height = &features->height;
VP8StatusCode status; VP8StatusCode status;
if (features == NULL) { if (features == NULL || data == NULL) {
return VP8_STATUS_INVALID_PARAM; return VP8_STATUS_INVALID_PARAM;
} }
DefaultFeatures(features); DefaultFeatures(features);
if (data == NULL) {
return VP8_STATUS_INVALID_PARAM;
}
// Skip over RIFF header. // Skip over RIFF header.
status = WebPParseRIFF(&data, &data_size, &riff_size); status = WebPParseRIFF(&data, &data_size, &riff_size);
if (status != VP8_STATUS_OK) { if (status != VP8_STATUS_OK) {
@ -501,8 +498,8 @@ static VP8StatusCode GetFeatures(const uint8_t* data, uint32_t data_size,
} }
// Skip over VP8X. // Skip over VP8X.
status = WebPParseVP8X(&data, &data_size, &vp8x_skip_size, &features->width, status = WebPParseVP8X(&data, &data_size, &vp8x_skip_size,
&features->height, &flags); width, height, &flags);
if (status != VP8_STATUS_OK) { if (status != VP8_STATUS_OK) {
return status; // Wrong VP8X / insufficient data. return status; // Wrong VP8X / insufficient data.
} }
@ -522,8 +519,7 @@ static VP8StatusCode GetFeatures(const uint8_t* data, uint32_t data_size,
} }
// Validates raw VP8 data. // Validates raw VP8 data.
if (!VP8GetInfo(data, data_size, vp8_chunk_size, if (!VP8GetInfo(data, data_size, vp8_chunk_size, width, height)) {
&features->width, &features->height)) {
return VP8_STATUS_BITSTREAM_ERROR; return VP8_STATUS_BITSTREAM_ERROR;
} }