From c4f39f4a3193982f78cff0d0f201283d3f2f042e Mon Sep 17 00:00:00 2001 From: James Zern Date: Wed, 13 Nov 2013 21:08:07 -0800 Subject: [PATCH] demux: cosmetics: remove a useless break + reword a comment Change-Id: I52b64abcde2c8322b59c594f2f75b509ed367bff --- src/demux/demux.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/demux/demux.c b/src/demux/demux.c index a3affff4..52d79fce 100644 --- a/src/demux/demux.c +++ b/src/demux/demux.c @@ -360,8 +360,8 @@ static ParseStatus ParseFragment(WebPDemuxer* const dmux, frame->x_offset_ = 2 * ReadLE24s(mem); frame->y_offset_ = 2 * ReadLE24s(mem); - // Store a fragment only if the fragments flag is set there is some data for - // this fragment is available. + // Store a fragment only if the 'fragments' flag is set and there is some + // data available. status = StoreFrame(frame_num, frgm_payload_size, mem, frame); if (status != PARSE_ERROR && has_fragments && frame->frame_num_ > 0) { added_fragment = AddFrame(dmux, frame); @@ -682,7 +682,6 @@ static int IsValidExtendedFormat(const WebPDemuxer* const dmux) { } if (!has_fragments && frame_count > 1) return 0; if (fragment_count > 0 && frame_count != fragment_count) return 0; - if (f == NULL) break; } return 1; }