Message ID | 20240518135039.12362-1-marcus@marcusspencer.xyz |
---|---|
State | New |
Headers | show |
Series | [FFmpeg-devel] avformat/aacdec: strictly conform to K&R style | expand |
Context | Check | Description |
---|---|---|
yinshiyou/make_loongarch64 | success | Make finished |
yinshiyou/make_fate_loongarch64 | success | Make fate finished |
andriy/make_x86 | success | Make finished |
andriy/make_fate_x86 | success | Make fate finished |
On Sat, 18 May 2024, Marcus B Spencer wrote: > In the K&R style, a single-statement block does not have braces. > Edit the code to conform to this rule. > > It is FFmpeg's code formatting convention to use K&R style. > > Signed-off-by: Marcus B Spencer <marcus@marcusspencer.xyz> > --- > libavformat/aacdec.c | 15 +++++---------- > 1 file changed, 5 insertions(+), 10 deletions(-) > > diff --git a/libavformat/aacdec.c b/libavformat/aacdec.c > index e267886e1a..3901a1dba8 100644 > --- a/libavformat/aacdec.c > +++ b/libavformat/aacdec.c > @@ -49,13 +49,12 @@ static int adts_aac_probe(const AVProbeData *p) > for (frames = 0; buf2 < end; frames++) { > uint32_t header = AV_RB16(buf2); > if ((header & 0xFFF6) != 0xFFF0) { > - if (buf != buf0) { > + if (buf != buf0) > // Found something that isn't an ADTS header, starting > // from a position other than the start of the buffer. > // Discard the count we've accumulated so far since it > // probably was a false positive. > frames = 0; > - } This should keep the braces, because this is in fact multiline. The rest looks good. Regards, Marton > break; > } > fsize = (AV_RB32(buf2 + 3) >> 13) & 0x1FFF; > @@ -143,9 +142,8 @@ static int handle_id3(AVFormatContext *s, AVPacket *pkt) > int ret; > > ret = av_append_packet(s->pb, pkt, ff_id3v2_tag_len(pkt->data) - pkt->size); > - if (ret < 0) { > + if (ret < 0) > return ret; > - } > > ffio_init_read_context(&pb, pkt->data, pkt->size); > ff_id3v2_read_dict(&pb.pub, &metadata, ID3v2_DEFAULT_MAGIC, &id3v2_extra_meta); > @@ -175,9 +173,8 @@ retry: > if (ret < 0) > return ret; > > - if (ret < ADTS_HEADER_SIZE) { > + if (ret < ADTS_HEADER_SIZE) > return AVERROR(EIO); > - } > > if ((AV_RB16(pkt->data) >> 4) != 0xfff) { > // Parse all the ID3 headers between frames > @@ -185,9 +182,8 @@ retry: > > av_assert2(append > 0); > ret = av_append_packet(s->pb, pkt, append); > - if (ret != append) { > + if (ret != append) > return AVERROR(EIO); > - } > if (!ff_id3v2_match(pkt->data, ID3v2_DEFAULT_MAGIC)) { > av_packet_unref(pkt); > ret = adts_aac_resync(s); > @@ -200,9 +196,8 @@ retry: > } > > fsize = (AV_RB32(pkt->data + 3) >> 13) & 0x1FFF; > - if (fsize < ADTS_HEADER_SIZE) { > + if (fsize < ADTS_HEADER_SIZE) > return AVERROR_INVALIDDATA; > - } > > ret = av_append_packet(s->pb, pkt, fsize - pkt->size); > > -- > 2.39.2 > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". >
diff --git a/libavformat/aacdec.c b/libavformat/aacdec.c index e267886e1a..3901a1dba8 100644 --- a/libavformat/aacdec.c +++ b/libavformat/aacdec.c @@ -49,13 +49,12 @@ static int adts_aac_probe(const AVProbeData *p) for (frames = 0; buf2 < end; frames++) { uint32_t header = AV_RB16(buf2); if ((header & 0xFFF6) != 0xFFF0) { - if (buf != buf0) { + if (buf != buf0) // Found something that isn't an ADTS header, starting // from a position other than the start of the buffer. // Discard the count we've accumulated so far since it // probably was a false positive. frames = 0; - } break; } fsize = (AV_RB32(buf2 + 3) >> 13) & 0x1FFF; @@ -143,9 +142,8 @@ static int handle_id3(AVFormatContext *s, AVPacket *pkt) int ret; ret = av_append_packet(s->pb, pkt, ff_id3v2_tag_len(pkt->data) - pkt->size); - if (ret < 0) { + if (ret < 0) return ret; - } ffio_init_read_context(&pb, pkt->data, pkt->size); ff_id3v2_read_dict(&pb.pub, &metadata, ID3v2_DEFAULT_MAGIC, &id3v2_extra_meta); @@ -175,9 +173,8 @@ retry: if (ret < 0) return ret; - if (ret < ADTS_HEADER_SIZE) { + if (ret < ADTS_HEADER_SIZE) return AVERROR(EIO); - } if ((AV_RB16(pkt->data) >> 4) != 0xfff) { // Parse all the ID3 headers between frames @@ -185,9 +182,8 @@ retry: av_assert2(append > 0); ret = av_append_packet(s->pb, pkt, append); - if (ret != append) { + if (ret != append) return AVERROR(EIO); - } if (!ff_id3v2_match(pkt->data, ID3v2_DEFAULT_MAGIC)) { av_packet_unref(pkt); ret = adts_aac_resync(s); @@ -200,9 +196,8 @@ retry: } fsize = (AV_RB32(pkt->data + 3) >> 13) & 0x1FFF; - if (fsize < ADTS_HEADER_SIZE) { + if (fsize < ADTS_HEADER_SIZE) return AVERROR_INVALIDDATA; - } ret = av_append_packet(s->pb, pkt, fsize - pkt->size);
In the K&R style, a single-statement block does not have braces. Edit the code to conform to this rule. It is FFmpeg's code formatting convention to use K&R style. Signed-off-by: Marcus B Spencer <marcus@marcusspencer.xyz> --- libavformat/aacdec.c | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-)