Commit 0b8e7ab0 authored by Diego Biurrun's avatar Diego Biurrun

some more linebreak and brace placement cosmetics

Originally committed as revision 20071 to svn://svn.ffmpeg.org/ffmpeg/trunk
parent 5e56b30e
...@@ -39,7 +39,8 @@ unsigned int ff_vorbis_nth_root(unsigned int x, unsigned int n) ...@@ -39,7 +39,8 @@ unsigned int ff_vorbis_nth_root(unsigned int x, unsigned int n)
do { do {
++ret; ++ret;
for(i=0,j=ret;i<n-1;i++) j*=ret; for(i=0,j=ret;i<n-1;i++)
j*=ret;
} while (j<=x); } while (j<=x);
return ret - 1; return ret - 1;
...@@ -62,14 +63,16 @@ int ff_vorbis_len2vlc(uint8_t *bits, uint32_t *codes, uint_fast32_t num) ...@@ -62,14 +63,16 @@ int ff_vorbis_len2vlc(uint8_t *bits, uint32_t *codes, uint_fast32_t num)
GetBitContext gb; GetBitContext gb;
#endif #endif
for(p=0;(bits[p]==0) && (p<num);++p); for(p=0;(bits[p]==0) && (p<num);++p)
;
if (p==num) { if (p==num) {
// av_log(vc->avccontext, AV_LOG_INFO, "An empty codebook. Heh?! \n"); // av_log(vc->avccontext, AV_LOG_INFO, "An empty codebook. Heh?! \n");
return 0; return 0;
} }
codes[p]=0; codes[p]=0;
if (bits[p] > 32) return 1; if (bits[p] > 32)
return 1;
for(i=0;i<bits[p];++i) for(i=0;i<bits[p];++i)
exit_at_level[i+1]=1<<i; exit_at_level[i+1]=1<<i;
...@@ -84,12 +87,16 @@ int ff_vorbis_len2vlc(uint8_t *bits, uint32_t *codes, uint_fast32_t num) ...@@ -84,12 +87,16 @@ int ff_vorbis_len2vlc(uint8_t *bits, uint32_t *codes, uint_fast32_t num)
++p; ++p;
for(;p<num;++p) { for(;p<num;++p) {
if (bits[p] > 32) return 1; if (bits[p] > 32)
if (bits[p]==0) continue; return 1;
if (bits[p]==0)
continue;
// find corresponding exit(node which the tree can grow further from) // find corresponding exit(node which the tree can grow further from)
for(i=bits[p];i>0;--i) for(i=bits[p];i>0;--i)
if (exit_at_level[i]) break; if (exit_at_level[i])
if (!i) return 1; // overspecified tree break;
if (!i) // overspecified tree
return 1;
code=exit_at_level[i]; code=exit_at_level[i];
exit_at_level[i]=0; exit_at_level[i]=0;
// construct code (append 0s to end) and introduce new exits // construct code (append 0s to end) and introduce new exits
...@@ -109,7 +116,8 @@ int ff_vorbis_len2vlc(uint8_t *bits, uint32_t *codes, uint_fast32_t num) ...@@ -109,7 +116,8 @@ int ff_vorbis_len2vlc(uint8_t *bits, uint32_t *codes, uint_fast32_t num)
//no exits should be left (underspecified tree - ie. unused valid vlcs - not allowed by SPEC) //no exits should be left (underspecified tree - ie. unused valid vlcs - not allowed by SPEC)
for (p=1; p<33; p++) for (p=1; p<33; p++)
if (exit_at_level[p]) return 1; if (exit_at_level[p])
return 1;
return 0; return 0;
} }
...@@ -127,9 +135,11 @@ void ff_vorbis_ready_floor1_list(vorbis_floor1_entry * list, int values) ...@@ -127,9 +135,11 @@ void ff_vorbis_ready_floor1_list(vorbis_floor1_entry * list, int values)
for (j = 2; j < i; j++) { for (j = 2; j < i; j++) {
int tmp = list[j].x; int tmp = list[j].x;
if (tmp < list[i].x) { if (tmp < list[i].x) {
if (tmp > list[list[i].low].x) list[i].low = j; if (tmp > list[list[i].low].x)
list[i].low = j;
} else { } else {
if (tmp < list[list[i].high].x) list[i].high = j; if (tmp < list[list[i].high].x)
list[i].high = j;
} }
} }
} }
...@@ -212,7 +222,9 @@ void ff_vorbis_floor1_render_list(vorbis_floor1_entry * list, int values, ...@@ -212,7 +222,9 @@ void ff_vorbis_floor1_render_list(vorbis_floor1_entry * list, int values,
lx = x1; lx = x1;
ly = y1; ly = y1;
} }
if (lx >= samples) break; if (lx >= samples)
break;
} }
if (lx < samples) render_line(lx, ly, samples, ly, out); if (lx < samples)
render_line(lx, ly, samples, ly, out);
} }
...@@ -747,9 +747,8 @@ static int vorbis_parse_setup_hdr_mappings(vorbis_context *vc) ...@@ -747,9 +747,8 @@ static int vorbis_parse_setup_hdr_mappings(vorbis_context *vc)
if (mapping_setup->submaps>1) { if (mapping_setup->submaps>1) {
mapping_setup->mux=av_mallocz(vc->audio_channels * sizeof(uint_fast8_t)); mapping_setup->mux=av_mallocz(vc->audio_channels * sizeof(uint_fast8_t));
for(j=0;j<vc->audio_channels;++j) { for(j=0;j<vc->audio_channels;++j)
mapping_setup->mux[j]=get_bits(gb, 4); mapping_setup->mux[j]=get_bits(gb, 4);
}
} }
for(j=0;j<mapping_setup->submaps;++j) { for(j=0;j<mapping_setup->submaps;++j) {
...@@ -1162,7 +1161,8 @@ static uint_fast8_t vorbis_floor1_decode(vorbis_context *vc, ...@@ -1162,7 +1161,8 @@ static uint_fast8_t vorbis_floor1_decode(vorbis_context *vc,
int_fast16_t dy, err; int_fast16_t dy, err;
if (!get_bits1(gb)) return 1; // silence if (!get_bits1(gb)) // silence
return 1;
// Read values (or differences) for the floor's points // Read values (or differences) for the floor's points
......
...@@ -894,7 +894,8 @@ static int apply_window_and_mdct(vorbis_enc_context * venc, ...@@ -894,7 +894,8 @@ static int apply_window_and_mdct(vorbis_enc_context * venc,
float n = (float)(1 << venc->log2_blocksize[0]) / 4.; float n = (float)(1 << venc->log2_blocksize[0]) / 4.;
// FIXME use dsp // FIXME use dsp
if (!venc->have_saved && !samples) return 0; if (!venc->have_saved && !samples)
return 0;
if (venc->have_saved) if (venc->have_saved)
for (channel = 0; channel < venc->channels; channel++) for (channel = 0; channel < venc->channels; channel++)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment