diff options
author | Martin Storsjö <martin@martin.st> | 2012-06-17 19:07:27 +0400 |
---|---|---|
committer | Martin Storsjö <martin@martin.st> | 2012-06-17 20:04:46 +0400 |
commit | 5f26d4d44896542e729c7515c784d983e396106a (patch) | |
tree | ecd34bd40bc6ad95d0f3dbf001f67dafbe8e65f6 /libavformat/amr.c | |
parent | 44fdf37c94ed1f5ecc42ef129cbaebff92addd0e (diff) |
amr: Cosmetic cleanup
Add spaces around operators, fix brace placement and whitespace to
match K&R style, vertically align code, remove redundant != 0 and
convert x == 0 into !x, drop useless braces.
Signed-off-by: Martin Storsjö <martin@martin.st>
Diffstat (limited to 'libavformat/amr.c')
-rw-r--r-- | libavformat/amr.c | 97 |
1 files changed, 40 insertions, 57 deletions
diff --git a/libavformat/amr.c b/libavformat/amr.c index 96f559b227..12b2b7fe58 100644 --- a/libavformat/amr.c +++ b/libavformat/amr.c @@ -28,27 +28,22 @@ Only mono files are supported. #include "avformat.h" #include "internal.h" -static const char AMR_header [] = "#!AMR\n"; -static const char AMRWB_header [] = "#!AMR-WB\n"; +static const char AMR_header[] = "#!AMR\n"; +static const char AMRWB_header[] = "#!AMR-WB\n"; #if CONFIG_AMR_MUXER static int amr_write_header(AVFormatContext *s) { - AVIOContext *pb = s->pb; + AVIOContext *pb = s->pb; AVCodecContext *enc = s->streams[0]->codec; s->priv_data = NULL; - if (enc->codec_id == CODEC_ID_AMR_NB) - { + if (enc->codec_id == CODEC_ID_AMR_NB) { avio_write(pb, AMR_header, sizeof(AMR_header) - 1); /* magic number */ - } - else if(enc->codec_id == CODEC_ID_AMR_WB) - { + } else if (enc->codec_id == CODEC_ID_AMR_WB) { avio_write(pb, AMRWB_header, sizeof(AMRWB_header) - 1); /* magic number */ - } - else - { + } else { return -1; } avio_flush(pb); @@ -65,11 +60,11 @@ static int amr_write_packet(AVFormatContext *s, AVPacket *pkt) static int amr_probe(AVProbeData *p) { - //Only check for "#!AMR" which could be amr-wb, amr-nb. - //This will also trigger multichannel files: "#!AMR_MC1.0\n" and - //"#!AMR-WB_MC1.0\n" (not supported) + // Only check for "#!AMR" which could be amr-wb, amr-nb. + // This will also trigger multichannel files: "#!AMR_MC1.0\n" and + // "#!AMR-WB_MC1.0\n" (not supported) - if(memcmp(p->buf,AMR_header,5)==0) + if (!memcmp(p->buf, AMR_header, 5)) return AVPROBE_SCORE_MAX; else return 0; @@ -86,83 +81,71 @@ static int amr_read_header(AVFormatContext *s) st = avformat_new_stream(s, NULL); if (!st) - { return AVERROR(ENOMEM); - } - if(memcmp(header,AMR_header,6)!=0) - { - avio_read(pb, header+6, 3); - if(memcmp(header,AMRWB_header,9)!=0) - { + if (memcmp(header, AMR_header, 6)) { + avio_read(pb, header + 6, 3); + if (memcmp(header, AMRWB_header, 9)) { return -1; } - st->codec->codec_tag = MKTAG('s', 'a', 'w', 'b'); - st->codec->codec_id = CODEC_ID_AMR_WB; + st->codec->codec_tag = MKTAG('s', 'a', 'w', 'b'); + st->codec->codec_id = CODEC_ID_AMR_WB; st->codec->sample_rate = 16000; - } - else - { - st->codec->codec_tag = MKTAG('s', 'a', 'm', 'r'); - st->codec->codec_id = CODEC_ID_AMR_NB; + } else { + st->codec->codec_tag = MKTAG('s', 'a', 'm', 'r'); + st->codec->codec_id = CODEC_ID_AMR_NB; st->codec->sample_rate = 8000; } - st->codec->channels = 1; + st->codec->channels = 1; st->codec->codec_type = AVMEDIA_TYPE_AUDIO; avpriv_set_pts_info(st, 64, 1, st->codec->sample_rate); return 0; } -static int amr_read_packet(AVFormatContext *s, - AVPacket *pkt) +static int amr_read_packet(AVFormatContext *s, AVPacket *pkt) { AVCodecContext *enc = s->streams[0]->codec; int read, size = 0, toc, mode; int64_t pos = avio_tell(s->pb); - if (s->pb->eof_reached) - { + if (s->pb->eof_reached) { return AVERROR(EIO); } -//FIXME this is wrong, this should rather be in a AVParset - toc=avio_r8(s->pb); + // FIXME this is wrong, this should rather be in a AVParset + toc = avio_r8(s->pb); mode = (toc >> 3) & 0x0F; - if (enc->codec_id == CODEC_ID_AMR_NB) - { - static const uint8_t packed_size[16] = {12, 13, 15, 17, 19, 20, 26, 31, 5, 0, 0, 0, 0, 0, 0, 0}; + if (enc->codec_id == CODEC_ID_AMR_NB) { + static const uint8_t packed_size[16] = { + 12, 13, 15, 17, 19, 20, 26, 31, 5, 0, 0, 0, 0, 0, 0, 0 + }; - size=packed_size[mode]+1; - } - else if(enc->codec_id == CODEC_ID_AMR_WB) - { - static uint8_t packed_size[16] = {18, 24, 33, 37, 41, 47, 51, 59, 61, 6, 6, 0, 0, 0, 1, 1}; + size = packed_size[mode] + 1; + } else if(enc->codec_id == CODEC_ID_AMR_WB) { + static uint8_t packed_size[16] = { + 18, 24, 33, 37, 41, 47, 51, 59, 61, 6, 6, 0, 0, 0, 1, 1 + }; - size=packed_size[mode]; - } - else - { + size = packed_size[mode]; + } else { assert(0); } - if ( (size==0) || av_new_packet(pkt, size)) - { + if (!size || av_new_packet(pkt, size)) return AVERROR(EIO); - } /* Both AMR formats have 50 frames per second */ s->streams[0]->codec->bit_rate = size*8*50; pkt->stream_index = 0; - pkt->pos = pos; - pkt->data[0]=toc; - pkt->duration= enc->codec_id == CODEC_ID_AMR_NB ? 160 : 320; - read = avio_read(s->pb, pkt->data+1, size-1); + pkt->pos = pos; + pkt->data[0] = toc; + pkt->duration = enc->codec_id == CODEC_ID_AMR_NB ? 160 : 320; + read = avio_read(s->pb, pkt->data + 1, size - 1); - if (read != size-1) - { + if (read != size - 1) { av_free_packet(pkt); return AVERROR(EIO); } |