]> rtime.felk.cvut.cz Git - frescor/ffmpeg.git/blobdiff - libavcodec/vorbis_enc.c
WMA: extend exponent range to 95
[frescor/ffmpeg.git] / libavcodec / vorbis_enc.c
index 43350deaf61741a6435302fbff921c9785760c20..61f78b576f1141483784d014ab21d5e01ee7a71c 100644 (file)
@@ -19,7 +19,7 @@
  */
 
 /**
- * @file vorbis_enc.c
+ * @file libavcodec/vorbis_enc.c
  * Native Vorbis encoder.
  * @author Oded Shimon <ods15@ods15.dyndns.org>
  */
 #include "vorbis_enc_data.h"
 
 #define BITSTREAM_WRITER_LE
-#include "bitstream.h"
+#include "put_bits.h"
 
 #undef NDEBUG
 #include <assert.h>
 
 typedef struct {
     int nentries;
-    uint8_t * lens;
-    uint32_t * codewords;
+    uint8_t *lens;
+    uint32_t *codewords;
     int ndimentions;
     float min;
     float delta;
     int seq_p;
     int lookup;
-    int * quantlist;
-    float * dimentions;
-    float * pow2;
+    int *quantlist;
+    float *dimentions;
+    float *pow2;
 } vorbis_enc_codebook;
 
 typedef struct {
     int dim;
     int subclass;
     int masterbook;
-    int * books;
+    int *books;
 } vorbis_enc_floor_class;
 
 typedef struct {
     int partitions;
-    int * partition_to_class;
+    int *partition_to_class;
     int nclasses;
-    vorbis_enc_floor_class * classes;
+    vorbis_enc_floor_class *classes;
     int multiplier;
     int rangebits;
     int values;
-    vorbis_floor1_entry * list;
+    vorbis_floor1_entry *list;
 } vorbis_enc_floor;
 
 typedef struct {
@@ -81,12 +81,12 @@ typedef struct {
 
 typedef struct {
     int submaps;
-    int * mux;
-    int * floor;
-    int * residue;
+    int *mux;
+    int *floor;
+    int *residue;
     int coupling_steps;
-    int * magnitude;
-    int * angle;
+    int *magnitude;
+    int *angle;
 } vorbis_enc_mapping;
 
 typedef struct {
@@ -98,54 +98,60 @@ typedef struct {
     int channels;
     int sample_rate;
     int log2_blocksize[2];
-    MDCTContext mdct[2];
-    const float * win[2];
+    FFTContext mdct[2];
+    const float *win[2];
     int have_saved;
-    float * saved;
-    float * samples;
-    float * floor; // also used for tmp values for mdct
-    float * coeffs; // also used for residue after floor
+    float *saved;
+    float *samples;
+    float *floor;  // also used for tmp values for mdct
+    float *coeffs; // also used for residue after floor
     float quality;
 
     int ncodebooks;
-    vorbis_enc_codebook * codebooks;
+    vorbis_enc_codebook *codebooks;
 
     int nfloors;
-    vorbis_enc_floor * floors;
+    vorbis_enc_floor *floors;
 
     int nresidues;
-    vorbis_enc_residue * residues;
+    vorbis_enc_residue *residues;
 
     int nmappings;
-    vorbis_enc_mapping * mappings;
+    vorbis_enc_mapping *mappings;
 
     int nmodes;
-    vorbis_enc_mode * modes;
+    vorbis_enc_mode *modes;
 
     int64_t sample_count;
 } vorbis_enc_context;
 
-static inline void put_codeword(PutBitContext * pb, vorbis_enc_codebook * cb, int entry) {
+static inline void put_codeword(PutBitContext *pb, vorbis_enc_codebook *cb,
+                                int entry)
+{
     assert(entry >= 0);
     assert(entry < cb->nentries);
     assert(cb->lens[entry]);
     put_bits(pb, cb->lens[entry], cb->codewords[entry]);
 }
 
-static int cb_lookup_vals(int lookup, int dimentions, int entries) {
-    if      (lookup == 1) return ff_vorbis_nth_root(entries, dimentions);
-    else if (lookup == 2) return dimentions * entries;
+static int cb_lookup_vals(int lookup, int dimentions, int entries)
+{
+    if (lookup == 1)
+        return ff_vorbis_nth_root(entries, dimentions);
+    else if (lookup == 2)
+        return dimentions *entries;
     return 0;
 }
 
-static void ready_codebook(vorbis_enc_codebook * cb) {
+static void ready_codebook(vorbis_enc_codebook *cb)
+{
     int i;
 
     ff_vorbis_len2vlc(cb->lens, cb->codewords, cb->nentries);
 
-    if (!cb->lookup)
+    if (!cb->lookup) {
         cb->pow2 = cb->dimentions = NULL;
-    else {
+    else {
         int vals = cb_lookup_vals(cb->lookup, cb->ndimentions, cb->nentries);
         cb->dimentions = av_malloc(sizeof(float) * cb->nentries * cb->ndimentions);
         cb->pow2 = av_mallocz(sizeof(float) * cb->nentries);
@@ -163,7 +169,7 @@ static void ready_codebook(vorbis_enc_codebook * cb) {
                 cb->dimentions[i * cb->ndimentions + j] = last + cb->min + cb->quantlist[off] * cb->delta;
                 if (cb->seq_p)
                     last = cb->dimentions[i * cb->ndimentions + j];
-                cb->pow2[i] += cb->dimentions[i * cb->ndimentions + j]*cb->dimentions[i * cb->ndimentions + j];
+                cb->pow2[i] += cb->dimentions[i * cb->ndimentions + j] * cb->dimentions[i * cb->ndimentions + j];
                 div *= vals;
             }
             cb->pow2[i] /= 2.;
@@ -171,7 +177,8 @@ static void ready_codebook(vorbis_enc_codebook * cb) {
     }
 }
 
-static void ready_residue(vorbis_enc_residue * rc, vorbis_enc_context * venc) {
+static void ready_residue(vorbis_enc_residue *rc, vorbis_enc_context *venc)
+{
     int i;
     assert(rc->type == 2);
     rc->maxes = av_mallocz(sizeof(float[2]) * rc->classifications);
@@ -179,15 +186,18 @@ static void ready_residue(vorbis_enc_residue * rc, vorbis_enc_context * venc) {
         int j;
         vorbis_enc_codebook * cb;
         for (j = 0; j < 8; j++)
-            if (rc->books[i][j] != -1) break;
-        if (j == 8) continue; // zero
+            if (rc->books[i][j] != -1)
+                break;
+        if (j == 8) // zero
+            continue;
         cb = &venc->codebooks[rc->books[i][j]];
         assert(cb->ndimentions >= 2);
         assert(cb->lookup);
 
         for (j = 0; j < cb->nentries; j++) {
             float a;
-            if (!cb->lens[j]) continue;
+            if (!cb->lens[j])
+                continue;
             a = fabs(cb->dimentions[j * cb->ndimentions]);
             if (a > rc->maxes[i][0])
                 rc->maxes[i][0] = a;
@@ -203,33 +213,35 @@ static void ready_residue(vorbis_enc_residue * rc, vorbis_enc_context * venc) {
     }
 }
 
-static void create_vorbis_context(vorbis_enc_context * venc, AVCodecContext * avccontext) {
-    vorbis_enc_floor * fc;
-    vorbis_enc_residue * rc;
-    vorbis_enc_mapping * mc;
+static void create_vorbis_context(vorbis_enc_context *venc,
+                                  AVCodecContext *avccontext)
+{
+    vorbis_enc_floor   *fc;
+    vorbis_enc_residue *rc;
+    vorbis_enc_mapping *mc;
     int i, book;
 
-    venc->channels = avccontext->channels;
+    venc->channels    = avccontext->channels;
     venc->sample_rate = avccontext->sample_rate;
     venc->log2_blocksize[0] = venc->log2_blocksize[1] = 11;
 
     venc->ncodebooks = FF_ARRAY_ELEMS(cvectors);
-    venc->codebooks = av_malloc(sizeof(vorbis_enc_codebook) * venc->ncodebooks);
+    venc->codebooks  = av_malloc(sizeof(vorbis_enc_codebook) * venc->ncodebooks);
 
     // codebook 0..14 - floor1 book, values 0..255
     // codebook 15 residue masterbook
     // codebook 16..29 residue
     for (book = 0; book < venc->ncodebooks; book++) {
-        vorbis_enc_codebook * cb = &venc->codebooks[book];
+        vorbis_enc_codebook *cb = &venc->codebooks[book];
         int vals;
         cb->ndimentions = cvectors[book].dim;
-        cb->nentries = cvectors[book].real_len;
-        cb->min = cvectors[book].min;
-        cb->delta = cvectors[book].delta;
-        cb->lookup = cvectors[book].lookup;
-        cb->seq_p = 0;
+        cb->nentries    = cvectors[book].real_len;
+        cb->min         = cvectors[book].min;
+        cb->delta       = cvectors[book].delta;
+        cb->lookup      = cvectors[book].lookup;
+        cb->seq_p       = 0;
 
-        cb->lens = av_malloc(sizeof(uint8_t) * cb->nentries);
+        cb->lens      = av_malloc(sizeof(uint8_t)  * cb->nentries);
         cb->codewords = av_malloc(sizeof(uint32_t) * cb->nentries);
         memcpy(cb->lens, cvectors[book].clens, cvectors[book].len);
         memset(cb->lens + cvectors[book].len, 0, cb->nentries - cvectors[book].len);
@@ -246,15 +258,15 @@ static void create_vorbis_context(vorbis_enc_context * venc, AVCodecContext * av
     }
 
     venc->nfloors = 1;
-    venc->floors = av_malloc(sizeof(vorbis_enc_floor) * venc->nfloors);
+    venc->floors  = av_malloc(sizeof(vorbis_enc_floor) * venc->nfloors);
 
     // just 1 floor
     fc = &venc->floors[0];
-    fc->partitions = 8;
+    fc->partitions         = 8;
     fc->partition_to_class = av_malloc(sizeof(int) * fc->partitions);
-    fc->nclasses = 0;
+    fc->nclasses           = 0;
     for (i = 0; i < fc->partitions; i++) {
-        static const int a[] = {0,1,2,2,3,3,4,4};
+        static const int a[] = {0, 1, 2, 2, 3, 3, 4, 4};
         fc->partition_to_class[i] = a[i];
         fc->nclasses = FFMAX(fc->nclasses, fc->partition_to_class[i]);
     }
@@ -263,16 +275,16 @@ static void create_vorbis_context(vorbis_enc_context * venc, AVCodecContext * av
     for (i = 0; i < fc->nclasses; i++) {
         vorbis_enc_floor_class * c = &fc->classes[i];
         int j, books;
-        c->dim = floor_classes[i].dim;
-        c->subclass = floor_classes[i].subclass;
+        c->dim        = floor_classes[i].dim;
+        c->subclass   = floor_classes[i].subclass;
         c->masterbook = floor_classes[i].masterbook;
-        books = (1 << c->subclass);
-        c->books = av_malloc(sizeof(int) * books);
+        books         = (1 << c->subclass);
+        c->books      = av_malloc(sizeof(int) * books);
         for (j = 0; j < books; j++)
             c->books[j] = floor_classes[i].nbooks[j];
     }
     fc->multiplier = 2;
-    fc->rangebits = venc->log2_blocksize[0] - 1;
+    fc->rangebits  = venc->log2_blocksize[0] - 1;
 
     fc->values = 2;
     for (i = 0; i < fc->partitions; i++)
@@ -292,17 +304,17 @@ static void create_vorbis_context(vorbis_enc_context * venc, AVCodecContext * av
     ff_vorbis_ready_floor1_list(fc->list, fc->values);
 
     venc->nresidues = 1;
-    venc->residues = av_malloc(sizeof(vorbis_enc_residue) * venc->nresidues);
+    venc->residues  = av_malloc(sizeof(vorbis_enc_residue) * venc->nresidues);
 
     // single residue
     rc = &venc->residues[0];
-    rc->type = 2;
-    rc->begin = 0;
-    rc->end = 1600;
-    rc->partition_size = 32;
+    rc->type            = 2;
+    rc->begin           = 0;
+    rc->end             = 1600;
+    rc->partition_size  = 32;
     rc->classifications = 10;
-    rc->classbook = 15;
-    rc->books = av_malloc(sizeof(*rc->books) * rc->classifications);
+    rc->classbook       = 15;
+    rc->books           = av_malloc(sizeof(*rc->books) * rc->classifications);
     {
         static const int8_t a[10][8] = {
             { -1, -1, -1, -1, -1, -1, -1, -1, },
@@ -321,59 +333,64 @@ static void create_vorbis_context(vorbis_enc_context * venc, AVCodecContext * av
     ready_residue(rc, venc);
 
     venc->nmappings = 1;
-    venc->mappings = av_malloc(sizeof(vorbis_enc_mapping) * venc->nmappings);
+    venc->mappings  = av_malloc(sizeof(vorbis_enc_mapping) * venc->nmappings);
 
     // single mapping
     mc = &venc->mappings[0];
     mc->submaps = 1;
-    mc->mux = av_malloc(sizeof(int) * venc->channels);
+    mc->mux     = av_malloc(sizeof(int) * venc->channels);
     for (i = 0; i < venc->channels; i++)
         mc->mux[i] = 0;
-    mc->floor = av_malloc(sizeof(int) * mc->submaps);
+    mc->floor   = av_malloc(sizeof(int) * mc->submaps);
     mc->residue = av_malloc(sizeof(int) * mc->submaps);
     for (i = 0; i < mc->submaps; i++) {
-        mc->floor[i] = 0;
+        mc->floor[i]   = 0;
         mc->residue[i] = 0;
     }
     mc->coupling_steps = venc->channels == 2 ? 1 : 0;
-    mc->magnitude = av_malloc(sizeof(int) * mc->coupling_steps);
-    mc->angle = av_malloc(sizeof(int) * mc->coupling_steps);
+    mc->magnitude      = av_malloc(sizeof(int) * mc->coupling_steps);
+    mc->angle          = av_malloc(sizeof(int) * mc->coupling_steps);
     if (mc->coupling_steps) {
         mc->magnitude[0] = 0;
-        mc->angle[0] = 1;
+        mc->angle[0]     = 1;
     }
 
     venc->nmodes = 1;
-    venc->modes = av_malloc(sizeof(vorbis_enc_mode) * venc->nmodes);
+    venc->modes  = av_malloc(sizeof(vorbis_enc_mode) * venc->nmodes);
 
     // single mode
     venc->modes[0].blockflag = 0;
-    venc->modes[0].mapping = 0;
+    venc->modes[0].mapping   = 0;
 
     venc->have_saved = 0;
-    venc->saved = av_malloc(sizeof(float) * venc->channels * (1 << venc->log2_blocksize[1]) / 2);
-    venc->samples = av_malloc(sizeof(float) * venc->channels * (1 << venc->log2_blocksize[1]));
-    venc->floor = av_malloc(sizeof(float) * venc->channels * (1 << venc->log2_blocksize[1]) / 2);
-    venc->coeffs = av_malloc(sizeof(float) * venc->channels * (1 << venc->log2_blocksize[1]) / 2);
+    venc->saved      = av_malloc(sizeof(float) * venc->channels * (1 << venc->log2_blocksize[1]) / 2);
+    venc->samples    = av_malloc(sizeof(float) * venc->channels * (1 << venc->log2_blocksize[1]));
+    venc->floor      = av_malloc(sizeof(float) * venc->channels * (1 << venc->log2_blocksize[1]) / 2);
+    venc->coeffs     = av_malloc(sizeof(float) * venc->channels * (1 << venc->log2_blocksize[1]) / 2);
 
     venc->win[0] = ff_vorbis_vwin[venc->log2_blocksize[0] - 6];
     venc->win[1] = ff_vorbis_vwin[venc->log2_blocksize[1] - 6];
 
-    ff_mdct_init(&venc->mdct[0], venc->log2_blocksize[0], 0);
-    ff_mdct_init(&venc->mdct[1], venc->log2_blocksize[1], 0);
+    ff_mdct_init(&venc->mdct[0], venc->log2_blocksize[0], 0, 1.0);
+    ff_mdct_init(&venc->mdct[1], venc->log2_blocksize[1], 0, 1.0);
 }
 
-static void put_float(PutBitContext * pb, float f) {
+static void put_float(PutBitContext *pb, float f)
+{
     int exp, mant;
     uint32_t res = 0;
     mant = (int)ldexp(frexp(f, &exp), 20);
     exp += 788 - 20;
-    if (mant < 0) { res |= (1 << 31); mant = -mant; }
+    if (mant < 0) {
+        res |= (1 << 31);
+        mant = -mant;
+    }
     res |= mant | (exp << 21);
-    put_bits(pb, 32, res);
+    put_bits32(pb, res);
 }
 
-static void put_codebook_header(PutBitContext * pb, vorbis_enc_codebook * cb) {
+static void put_codebook_header(PutBitContext *pb, vorbis_enc_codebook *cb)
+{
     int i;
     int ordered = 0;
 
@@ -382,7 +399,8 @@ static void put_codebook_header(PutBitContext * pb, vorbis_enc_codebook * cb) {
     put_bits(pb, 24, cb->nentries);
 
     for (i = 1; i < cb->nentries; i++)
-        if (cb->lens[i] < cb->lens[i-1]) break;
+        if (cb->lens[i] < cb->lens[i-1])
+            break;
     if (i == cb->nentries)
         ordered = 1;
 
@@ -394,7 +412,8 @@ static void put_codebook_header(PutBitContext * pb, vorbis_enc_codebook * cb) {
         while (i < cb->nentries) {
             int j;
             for (j = 0; j+i < cb->nentries; j++)
-                if (cb->lens[j+i] != len) break;
+                if (cb->lens[j+i] != len)
+                    break;
             put_bits(pb, ilog(cb->nentries - i), j);
             i += j;
             len++;
@@ -402,20 +421,23 @@ static void put_codebook_header(PutBitContext * pb, vorbis_enc_codebook * cb) {
     } else {
         int sparse = 0;
         for (i = 0; i < cb->nentries; i++)
-            if (!cb->lens[i]) break;
+            if (!cb->lens[i])
+                break;
         if (i != cb->nentries)
             sparse = 1;
         put_bits(pb, 1, sparse);
 
         for (i = 0; i < cb->nentries; i++) {
-            if (sparse) put_bits(pb, 1, !!cb->lens[i]);
-            if (cb->lens[i]) put_bits(pb, 5, cb->lens[i] - 1);
+            if (sparse)
+                put_bits(pb, 1, !!cb->lens[i]);
+            if (cb->lens[i])
+                put_bits(pb, 5, cb->lens[i] - 1);
         }
     }
 
     put_bits(pb, 4, cb->lookup);
     if (cb->lookup) {
-        int tmp = cb_lookup_vals(cb->lookup, cb->ndimentions, cb->nentries);
+        int tmp  = cb_lookup_vals(cb->lookup, cb->ndimentions, cb->nentries);
         int bits = ilog(cb->quantlist[0]);
 
         for (i = 1; i < tmp; i++)
@@ -432,7 +454,8 @@ static void put_codebook_header(PutBitContext * pb, vorbis_enc_codebook * cb) {
     }
 }
 
-static void put_floor_header(PutBitContext * pb, vorbis_enc_floor * fc) {
+static void put_floor_header(PutBitContext *pb, vorbis_enc_floor *fc)
+{
     int i;
 
     put_bits(pb, 16, 1); // type, only floor1 is supported
@@ -464,7 +487,8 @@ static void put_floor_header(PutBitContext * pb, vorbis_enc_floor * fc) {
         put_bits(pb, fc->rangebits, fc->list[i].x);
 }
 
-static void put_residue_header(PutBitContext * pb, vorbis_enc_residue * rc) {
+static void put_residue_header(PutBitContext *pb, vorbis_enc_residue *rc)
+{
     int i;
 
     put_bits(pb, 16, rc->type);
@@ -495,52 +519,50 @@ static void put_residue_header(PutBitContext * pb, vorbis_enc_residue * rc) {
     }
 }
 
-static int put_main_header(vorbis_enc_context * venc, uint8_t ** out) {
+static int put_main_header(vorbis_enc_context *venc, uint8_t **out)
+{
     int i;
     PutBitContext pb;
-    uint8_t buffer[50000] = {0}, * p = buffer;
+    uint8_t buffer[50000] = {0}, *p = buffer;
     int buffer_len = sizeof buffer;
     int len, hlens[3];
 
     // identification header
     init_put_bits(&pb, p, buffer_len);
     put_bits(&pb, 8, 1); //magic
-    for (i = 0; "vorbis"[i]; i++)
-        put_bits(&pb, 8, "vorbis"[i]);
-    put_bits(&pb, 32, 0); // version
-    put_bits(&pb, 8, venc->channels);
-    put_bits(&pb, 32, venc->sample_rate);
-    put_bits(&pb, 32, 0); // bitrate
-    put_bits(&pb, 32, 0); // bitrate
-    put_bits(&pb, 32, 0); // bitrate
-    put_bits(&pb, 4, venc->log2_blocksize[0]);
-    put_bits(&pb, 4, venc->log2_blocksize[1]);
-    put_bits(&pb, 1, 1); // framing
+    ff_put_string(&pb, "vorbis", 0);
+    put_bits32(&pb, 0); // version
+    put_bits(&pb,  8, venc->channels);
+    put_bits32(&pb, venc->sample_rate);
+    put_bits32(&pb, 0); // bitrate
+    put_bits32(&pb, 0); // bitrate
+    put_bits32(&pb, 0); // bitrate
+    put_bits(&pb,  4, venc->log2_blocksize[0]);
+    put_bits(&pb,  4, venc->log2_blocksize[1]);
+    put_bits(&pb,  1, 1); // framing
 
     flush_put_bits(&pb);
-    hlens[0] = (put_bits_count(&pb) + 7) / 8;
+    hlens[0] = put_bits_count(&pb) >> 3;
     buffer_len -= hlens[0];
     p += hlens[0];
 
     // comment header
     init_put_bits(&pb, p, buffer_len);
     put_bits(&pb, 8, 3); //magic
-    for (i = 0; "vorbis"[i]; i++)
-        put_bits(&pb, 8, "vorbis"[i]);
-    put_bits(&pb, 32, 0); // vendor length TODO
-    put_bits(&pb, 32, 0); // amount of comments
-    put_bits(&pb, 1, 1); // framing
+    ff_put_string(&pb, "vorbis", 0);
+    put_bits32(&pb, 0); // vendor length TODO
+    put_bits32(&pb, 0); // amount of comments
+    put_bits(&pb,  1, 1); // framing
 
     flush_put_bits(&pb);
-    hlens[1] = (put_bits_count(&pb) + 7) / 8;
+    hlens[1] = put_bits_count(&pb) >> 3;
     buffer_len -= hlens[1];
     p += hlens[1];
 
     // setup header
     init_put_bits(&pb, p, buffer_len);
     put_bits(&pb, 8, 5); //magic
-    for (i = 0; "vorbis"[i]; i++)
-        put_bits(&pb, 8, "vorbis"[i]);
+    ff_put_string(&pb, "vorbis", 0);
 
     // codebooks
     put_bits(&pb, 8, venc->ncodebooks - 1);
@@ -548,7 +570,7 @@ static int put_main_header(vorbis_enc_context * venc, uint8_t ** out) {
         put_codebook_header(&pb, &venc->codebooks[i]);
 
     // time domain, reserved, zero
-    put_bits(&pb, 6, 0);
+    put_bits(&pb,  6, 0);
     put_bits(&pb, 16, 0);
 
     // floors
@@ -564,7 +586,7 @@ static int put_main_header(vorbis_enc_context * venc, uint8_t ** out) {
     // mappings
     put_bits(&pb, 6, venc->nmappings - 1);
     for (i = 0; i < venc->nmappings; i++) {
-        vorbis_enc_mapping * mc = &venc->mappings[i];
+        vorbis_enc_mapping *mc = &venc->mappings[i];
         int j;
         put_bits(&pb, 16, 0); // mapping type
 
@@ -606,7 +628,7 @@ static int put_main_header(vorbis_enc_context * venc, uint8_t ** out) {
     put_bits(&pb, 1, 1); // framing
 
     flush_put_bits(&pb);
-    hlens[2] = (put_bits_count(&pb) + 7) / 8;
+    hlens[2] = put_bits_count(&pb) >> 3;
 
     len = hlens[0] + hlens[1] + hlens[2];
     p = *out = av_mallocz(64 + len + len/255);
@@ -624,7 +646,8 @@ static int put_main_header(vorbis_enc_context * venc, uint8_t ** out) {
     return p - *out;
 }
 
-static float get_floor_average(vorbis_enc_floor * fc, float * coeffs, int i) {
+static float get_floor_average(vorbis_enc_floor * fc, float *coeffs, int i)
+{
     int begin = fc->list[fc->list[FFMAX(i-1, 0)].sort].x;
     int end   = fc->list[fc->list[FFMIN(i+1, fc->values - 1)].sort].x;
     int j;
@@ -635,12 +658,14 @@ static float get_floor_average(vorbis_enc_floor * fc, float * coeffs, int i) {
     return average / (end - begin);
 }
 
-static void floor_fit(vorbis_enc_context * venc, vorbis_enc_floor * fc, float * coeffs, uint_fast16_t * posts, int samples) {
+static void floor_fit(vorbis_enc_context *venc, vorbis_enc_floor *fc,
+                      float *coeffs, uint_fast16_t *posts, int samples)
+{
     int range = 255 / fc->multiplier + 1;
     int i;
     float tot_average = 0.;
     float averages[fc->values];
-    for (i = 0; i < fc->values; i++){
+    for (i = 0; i < fc->values; i++) {
         averages[i] = get_floor_average(fc, coeffs, i);
         tot_average += averages[i];
     }
@@ -648,22 +673,27 @@ static void floor_fit(vorbis_enc_context * venc, vorbis_enc_floor * fc, float *
     tot_average /= venc->quality;
 
     for (i = 0; i < fc->values; i++) {
-        int position = fc->list[fc->list[i].sort].x;
+        int position  = fc->list[fc->list[i].sort].x;
         float average = averages[i];
         int j;
 
         average *= pow(tot_average / average, 0.5) * pow(1.25, position/200.); // MAGIC!
         for (j = 0; j < range - 1; j++)
-            if (ff_vorbis_floor1_inverse_db_table[j * fc->multiplier] > average) break;
+            if (ff_vorbis_floor1_inverse_db_table[j * fc->multiplier] > average)
+                break;
         posts[fc->list[i].sort] = j;
     }
 }
 
-static int render_point(int x0, int y0, int x1, int y1, int x) {
+static int render_point(int x0, int y0, int x1, int y1, int x)
+{
     return y0 +  (x - x0) * (y1 - y0) / (x1 - x0);
 }
 
-static void floor_encode(vorbis_enc_context * venc, vorbis_enc_floor * fc, PutBitContext * pb, uint_fast16_t * posts, float * floor, int samples) {
+static void floor_encode(vorbis_enc_context *venc, vorbis_enc_floor *fc,
+                         PutBitContext *pb, uint_fast16_t *posts,
+                         float *floor, int samples)
+{
     int range = 255 / fc->multiplier + 1;
     int coded[fc->values]; // first 2 values are unused
     int i, counter;
@@ -686,8 +716,10 @@ static void floor_encode(vorbis_enc_context * venc, vorbis_enc_floor * fc, PutBi
             coded[i] = 0; // must be used later as flag!
             continue;
         } else {
-            if (!coded[fc->list[i].low ]) coded[fc->list[i].low ] = -1;
-            if (!coded[fc->list[i].high]) coded[fc->list[i].high] = -1;
+            if (!coded[fc->list[i].low ])
+                coded[fc->list[i].low ] = -1;
+            if (!coded[fc->list[i].high])
+                coded[fc->list[i].high] = -1;
         }
         if (posts[i] > predicted) {
             if (posts[i] - predicted > room)
@@ -716,39 +748,46 @@ static void floor_encode(vorbis_enc_context * venc, vorbis_enc_floor * fc, PutBi
                     if (c->books[l] != -1)
                         maxval = venc->codebooks[c->books[l]].nentries;
                     // coded could be -1, but this still works, cause that is 0
-                    if (coded[counter + k] < maxval) break;
+                    if (coded[counter + k] < maxval)
+                        break;
                 }
                 assert(l != csub);
-                cval |= l << cshift;
+                cval   |= l << cshift;
                 cshift += c->subclass;
             }
             put_codeword(pb, book, cval);
         }
         for (k = 0; k < c->dim; k++) {
-            int book = c->books[cval & (csub-1)];
+            int book  = c->books[cval & (csub-1)];
             int entry = coded[counter++];
             cval >>= c->subclass;
-            if (book == -1) continue;
-            if (entry == -1) entry = 0;
+            if (book == -1)
+                continue;
+            if (entry == -1)
+                entry = 0;
             put_codeword(pb, &venc->codebooks[book], entry);
         }
     }
 
-    ff_vorbis_floor1_render_list(fc->list, fc->values, posts, coded, fc->multiplier, floor, samples);
+    ff_vorbis_floor1_render_list(fc->list, fc->values, posts, coded,
+                                 fc->multiplier, floor, samples);
 }
 
-static float * put_vector(vorbis_enc_codebook * book, PutBitContext * pb, float * num) {
+static float *put_vector(vorbis_enc_codebook *book, PutBitContext *pb,
+                         float *num)
+{
     int i, entry = -1;
     float distance = FLT_MAX;
     assert(book->dimentions);
     for (i = 0; i < book->nentries; i++) {
         float * vec = book->dimentions + i * book->ndimentions, d = book->pow2[i];
         int j;
-        if (!book->lens[i]) continue;
+        if (!book->lens[i])
+            continue;
         for (j = 0; j < book->ndimentions; j++)
             d -= vec[j] * num[j];
         if (distance > d) {
-            entry = i;
+            entry    = i;
             distance = d;
         }
     }
@@ -756,11 +795,14 @@ static float * put_vector(vorbis_enc_codebook * book, PutBitContext * pb, float
     return &book->dimentions[entry * book->ndimentions];
 }
 
-static void residue_encode(vorbis_enc_context * venc, vorbis_enc_residue * rc, PutBitContext * pb, float * coeffs, int samples, int real_ch) {
+static void residue_encode(vorbis_enc_context *venc, vorbis_enc_residue *rc,
+                           PutBitContext *pb, float *coeffs, int samples,
+                           int real_ch)
+{
     int pass, i, j, p, k;
-    int psize = rc->partition_size;
+    int psize      = rc->partition_size;
     int partitions = (rc->end - rc->begin) / psize;
-    int channels = (rc->type == 2) ? 1 : real_ch;
+    int channels   = (rc->type == 2) ? 1 : real_ch;
     int classes[channels][partitions];
     int classwords = venc->codebooks[rc->classbook].ndimentions;
 
@@ -774,9 +816,9 @@ static void residue_encode(vorbis_enc_context * venc, vorbis_enc_residue * rc, P
             max2 = FFMAX(max2, fabs(coeffs[samples + k / real_ch]));
         }
 
-        for (i = 0; i < rc->classifications - 1; i++) {
-            if (max1 < rc->maxes[i][0] && max2 < rc->maxes[i][1]) break;
-        }
+        for (i = 0; i < rc->classifications - 1; i++)
+            if (max1 < rc->maxes[i][0] && max2 < rc->maxes[i][1])
+                break;
         classes[0][p] = i;
     }
 
@@ -797,15 +839,16 @@ static void residue_encode(vorbis_enc_context * venc, vorbis_enc_residue * rc, P
                 for (j = 0; j < channels; j++) {
                     int nbook = rc->books[classes[j][p]][pass];
                     vorbis_enc_codebook * book = &venc->codebooks[nbook];
-                    float * buf = coeffs + samples*j + rc->begin + p*psize;
-                    if (nbook == -1) continue;
+                    float *buf = coeffs + samples*j + rc->begin + p*psize;
+                    if (nbook == -1)
+                        continue;
 
                     assert(rc->type == 0 || rc->type == 2);
                     assert(!(psize % book->ndimentions));
 
                     if (rc->type == 0) {
                         for (k = 0; k < psize; k += book->ndimentions) {
-                            float * a = put_vector(book, pb, &buf[k]);
+                            float *a = put_vector(book, pb, &buf[k]);
                             int l;
                             for (l = 0; l < book->ndimentions; l++)
                                 buf[k + l] -= a[l];
@@ -814,14 +857,14 @@ static void residue_encode(vorbis_enc_context * venc, vorbis_enc_residue * rc, P
                         int s = rc->begin + p * psize, a1, b1;
                         a1 = (s % real_ch) * samples;
                         b1 =  s / real_ch;
-                        s = real_ch * samples;
+                        s  = real_ch * samples;
                         for (k = 0; k < psize; k += book->ndimentions) {
                             int dim, a2 = a1, b2 = b1;
-                            float vec[book->ndimentions], * pv = vec;
+                            float vec[book->ndimentions], *pv = vec;
                             for (dim = book->ndimentions; dim--; ) {
                                 *pv++ = coeffs[a2 + b2];
                                 if ((a2 += samples) == s) {
-                                    a2=0;
+                                    a2 = 0;
                                     b2++;
                                 }
                             }
@@ -829,7 +872,7 @@ static void residue_encode(vorbis_enc_context * venc, vorbis_enc_residue * rc, P
                             for (dim = book->ndimentions; dim--; ) {
                                 coeffs[a1 + b1] -= *pv++;
                                 if ((a1 += samples) == s) {
-                                    a1=0;
+                                    a1 = 0;
                                     b1++;
                                 }
                             }
@@ -841,23 +884,26 @@ static void residue_encode(vorbis_enc_context * venc, vorbis_enc_residue * rc, P
     }
 }
 
-static int apply_window_and_mdct(vorbis_enc_context * venc, signed short * audio, int samples) {
+static int apply_window_and_mdct(vorbis_enc_context *venc, signed short *audio,
+                                 int samples)
+{
     int i, j, channel;
     const float * win = venc->win[0];
     int window_len = 1 << (venc->log2_blocksize[0] - 1);
     float n = (float)(1 << venc->log2_blocksize[0]) / 4.;
     // FIXME use dsp
 
-    if (!venc->have_saved && !samples) return 0;
+    if (!venc->have_saved && !samples)
+        return 0;
 
     if (venc->have_saved) {
-        for (channel = 0; channel < venc->channels; channel++) {
-            memcpy(venc->samples + channel*window_len*2, venc->saved + channel*window_len, sizeof(float)*window_len);
-        }
+        for (channel = 0; channel < venc->channels; channel++)
+            memcpy(venc->samples + channel * window_len * 2,
+                   venc->saved + channel * window_len, sizeof(float) * window_len);
     } else {
-        for (channel = 0; channel < venc->channels; channel++) {
-            memset(venc->samples + channel*window_len*2, 0, sizeof(float)*window_len);
-        }
+        for (channel = 0; channel < venc->channels; channel++)
+            memset(venc->samples + channel * window_len * 2, 0,
+                   sizeof(float) * window_len);
     }
 
     if (samples) {
@@ -868,18 +914,18 @@ static int apply_window_and_mdct(vorbis_enc_context * venc, signed short * audio
                 offset[i] = -audio[j] / 32768. / n * win[window_len - i - 1]; //FIXME find out why the sign has to be fliped
         }
     } else {
-        for (channel = 0; channel < venc->channels; channel++) {
-            memset(venc->samples + channel*window_len*2 + window_len, 0, sizeof(float)*window_len);
-        }
+        for (channel = 0; channel < venc->channels; channel++)
+            memset(venc->samples + channel * window_len * 2 + window_len,
+                   0, sizeof(float) * window_len);
     }
 
-    for (channel = 0; channel < venc->channels; channel++) {
-        ff_mdct_calc(&venc->mdct[0], venc->coeffs + channel*window_len, venc->samples + channel*window_len*2);
-    }
+    for (channel = 0; channel < venc->channels; channel++)
+        ff_mdct_calc(&venc->mdct[0], venc->coeffs + channel * window_len,
+                     venc->samples + channel * window_len * 2);
 
     if (samples) {
         for (channel = 0; channel < venc->channels; channel++) {
-            float * offset = venc->saved + channel*window_len;
+            float *offset = venc->saved + channel * window_len;
             j = channel;
             for (i = 0; i < samples; i++, j += venc->channels)
                 offset[i] = -audio[j] / 32768. / n * win[i]; //FIXME find out why the sign has to be fliped
@@ -891,9 +937,9 @@ static int apply_window_and_mdct(vorbis_enc_context * venc, signed short * audio
     return 1;
 }
 
-static av_cold int vorbis_encode_init(AVCodecContext * avccontext)
+static av_cold int vorbis_encode_init(AVCodecContext *avccontext)
 {
-    vorbis_enc_context * venc = avccontext->priv_data;
+    vorbis_enc_context *venc = avccontext->priv_data;
 
     if (avccontext->channels != 2) {
         av_log(avccontext, AV_LOG_ERROR, "Current FFmpeg Vorbis encoder only supports 2 channels.\n");
@@ -910,25 +956,28 @@ static av_cold int vorbis_encode_init(AVCodecContext * avccontext)
 
     avccontext->extradata_size = put_main_header(venc, (uint8_t**)&avccontext->extradata);
 
-    avccontext->frame_size = 1 << (venc->log2_blocksize[0] - 1);
+    avccontext->frame_size     = 1 << (venc->log2_blocksize[0] - 1);
 
-    avccontext->coded_frame = avcodec_alloc_frame();
+    avccontext->coded_frame            = avcodec_alloc_frame();
     avccontext->coded_frame->key_frame = 1;
 
     return 0;
 }
 
-static int vorbis_encode_frame(AVCodecContext * avccontext, unsigned char * packets, int buf_size, void *data)
+static int vorbis_encode_frame(AVCodecContext *avccontext,
+                               unsigned char *packets,
+                               int buf_size, void *data)
 {
-    vorbis_enc_context * venc = avccontext->priv_data;
-    signed short * audio = data;
+    vorbis_enc_context *venc = avccontext->priv_data;
+    signed short *audio = data;
     int samples = data ? avccontext->frame_size : 0;
-    vorbis_enc_mode * mode;
-    vorbis_enc_mapping * mapping;
+    vorbis_enc_mode *mode;
+    vorbis_enc_mapping *mapping;
     PutBitContext pb;
     int i;
 
-    if (!apply_window_and_mdct(venc, audio, samples)) return 0;
+    if (!apply_window_and_mdct(venc, audio, samples))
+        return 0;
     samples = 1 << (venc->log2_blocksize[0] - 1);
 
     init_put_bits(&pb, packets, buf_size);
@@ -937,7 +986,7 @@ static int vorbis_encode_frame(AVCodecContext * avccontext, unsigned char * pack
 
     put_bits(&pb, ilog(venc->nmodes - 1), 0); // 0 bits, the mode
 
-    mode = &venc->modes[0];
+    mode    = &venc->modes[0];
     mapping = &venc->mappings[mode->mapping];
     if (mode->blockflag) {
         put_bits(&pb, 1, 0);
@@ -945,40 +994,42 @@ static int vorbis_encode_frame(AVCodecContext * avccontext, unsigned char * pack
     }
 
     for (i = 0; i < venc->channels; i++) {
-        vorbis_enc_floor * fc = &venc->floors[mapping->floor[mapping->mux[i]]];
+        vorbis_enc_floor *fc = &venc->floors[mapping->floor[mapping->mux[i]]];
         uint_fast16_t posts[fc->values];
         floor_fit(venc, fc, &venc->coeffs[i * samples], posts, samples);
         floor_encode(venc, fc, &pb, posts, &venc->floor[i * samples], samples);
     }
 
-    for (i = 0; i < venc->channels * samples; i++) {
+    for (i = 0; i < venc->channels * samples; i++)
         venc->coeffs[i] /= venc->floor[i];
-    }
 
     for (i = 0; i < mapping->coupling_steps; i++) {
-        float * mag = venc->coeffs + mapping->magnitude[i] * samples;
-        float * ang = venc->coeffs + mapping->angle[i] * samples;
+        float *mag = venc->coeffs + mapping->magnitude[i] * samples;
+        float *ang = venc->coeffs + mapping->angle[i]     * samples;
         int j;
         for (j = 0; j < samples; j++) {
             float a = ang[j];
             ang[j] -= mag[j];
-            if (mag[j] > 0) ang[j] = -ang[j];
-            if (ang[j] < 0) mag[j] = a;
+            if (mag[j] > 0)
+                ang[j] = -ang[j];
+            if (ang[j] < 0)
+                mag[j] = a;
         }
     }
 
-    residue_encode(venc, &venc->residues[mapping->residue[mapping->mux[0]]], &pb, venc->coeffs, samples, venc->channels);
+    residue_encode(venc, &venc->residues[mapping->residue[mapping->mux[0]]],
+                   &pb, venc->coeffs, samples, venc->channels);
 
     avccontext->coded_frame->pts = venc->sample_count;
     venc->sample_count += avccontext->frame_size;
     flush_put_bits(&pb);
-    return (put_bits_count(&pb) + 7) / 8;
+    return put_bits_count(&pb) >> 3;
 }
 
 
-static av_cold int vorbis_encode_close(AVCodecContext * avccontext)
+static av_cold int vorbis_encode_close(AVCodecContext *avccontext)
 {
-    vorbis_enc_context * venc = avccontext->priv_data;
+    vorbis_enc_context *venc = avccontext->priv_data;
     int i;
 
     if (venc->codebooks)
@@ -1045,6 +1096,6 @@ AVCodec vorbis_encoder = {
     vorbis_encode_frame,
     vorbis_encode_close,
     .capabilities= CODEC_CAP_DELAY,
-    .sample_fmts = (enum SampleFormat[]){SAMPLE_FMT_S16,SAMPLE_FMT_NONE},
+    .sample_fmts = (const enum SampleFormat[]){SAMPLE_FMT_S16,SAMPLE_FMT_NONE},
     .long_name = NULL_IF_CONFIG_SMALL("Vorbis"),
 };