Commit Graph

79 Commits

Author SHA1 Message Date
Vitor Sessak
aed39f6c02 Remove RA288Context.output buffer. This buffer is just RA288Context.sb
backwards (output[i] == sb[N-i], where N is the buffer length).

This makes the code slower, this will be fixed in my next commit.

Originally committed as revision 14446 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-28 02:53:07 +00:00
Vitor Sessak
b26d320583 Add a few comments pointing to the G.728 specification
Originally committed as revision 14444 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-27 22:22:51 +00:00
Vitor Sessak
854ce441c3 Remove the history buffer from the context. It can easily be evaluated
from the lhist buffer.

Originally committed as revision 14443 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-27 22:21:41 +00:00
Vitor Sessak
0143ec2a7e Rename two context buffers: pr{1,2} -> {sp,gain}_lpc. Also add a few doxy
comments.

Originally committed as revision 14437 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-27 20:58:29 +00:00
Vitor Sessak
59da0390dc Remove st1 and st2 temporary buffers in backward_filter() and use instead
RA288Context.pr{1,2}. Note that the pr{1,2} buffers are one unity smaller
than the st{1,2} buffers. My guess is that the original coder decided to
add one to the array sizes "just to be sure".

Originally committed as revision 14435 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-27 20:55:27 +00:00
Vitor Sessak
3df136d438 Change slightly the meaning of RA288Context.phase. This
allows to remove two intermediary buffers and avoid a few
memcpy's.

Originally committed as revision 14432 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-27 20:35:32 +00:00
Vitor Sessak
f42eb5715d Remove st1 and st2 variables from context, since they could be just local vars
Originally committed as revision 14421 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-26 15:14:50 +00:00
Vitor Sessak
272d258ab9 Rename st{1,2}{a,b} context variables to more meaningful names: {sp,gain}_{rec,hist}
Originally committed as revision 14420 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-26 15:10:17 +00:00
Vitor Sessak
efa86ebe4c Cosmetics: remove useless parenthesis
Originally committed as revision 14418 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-26 14:38:29 +00:00
Vitor Sessak
e07c5ade5e Move colmult() function to the beginning of file to group DSP-related functions.
Originally committed as revision 14417 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-26 14:35:50 +00:00
Vitor Sessak
5af20c207e Remove an useless comment describing colmult()
Originally committed as revision 14416 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-26 14:33:53 +00:00
Vitor Sessak
d899014ee2 Remove an useless comment describing prodsum()
Originally committed as revision 14415 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-26 14:33:25 +00:00
Vitor Sessak
952a980d75 Give the context variables of ra288 a better name: s/glob/ractx/
Originally committed as revision 14414 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-26 14:31:47 +00:00
Vitor Sessak
029e1c01b5 Give the context structure of ra288 a better name: s/Real288_internal/RA288Context/
Originally committed as revision 14413 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-26 14:29:39 +00:00
Vitor Sessak
a53b545068 Cosmetics: grammar nits
Originally committed as revision 14368 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 11:55:01 +00:00
Vitor Sessak
2477d60944 Rename update() function to backward_filter() and add a doxy comment
Originally committed as revision 14364 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 04:29:23 +00:00
Vitor Sessak
a6755f8d53 Rename and comment bandwidth broadening tables
(s/table1a/syn_bw_tab/, s/table2a/gain_bw_tab/).

Originally committed as revision 14363 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 04:15:50 +00:00
Vitor Sessak
5e65f5df0e Follow FFmpeg convention of returning negative values on error in eval_lpc_coeffs()
Originally committed as revision 14362 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 04:07:50 +00:00
Vitor Sessak
a3896c6377 Rename pred() to eval_lpc_coeffs() and add a doxy comment
Originally committed as revision 14361 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 04:05:41 +00:00
Vitor Sessak
a0cce1587b Give windowing tables a more descriptive name
(s/table1/syn_window/, s/table2/gain_window/).

Originally committed as revision 14360 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 03:50:06 +00:00
Vitor Sessak
475d0e1464 Add comments to do_hybrid_window()
Originally committed as revision 14359 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 03:28:42 +00:00
Vitor Sessak
0f72a2b7d3 Give parameters of do_hybrid_window() more meaningful names
Originally committed as revision 14358 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 03:28:01 +00:00
Vitor Sessak
427981c7a6 Rename co() function to the much more descriptive name of
do_hybrid_window(). Also change a misleading and wrong comment.

Originally committed as revision 14357 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 03:23:51 +00:00
Vitor Sessak
5ab3d66fe5 Simplify co(): write constant in a more readable way
Originally committed as revision 14356 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 03:22:19 +00:00
Vitor Sessak
bf75ac57b8 Simplify co(), use memcpy/memmove and colmult() when useful.
Originally committed as revision 14355 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-24 03:19:52 +00:00
Vitor Sessak
9f59e608d2 Cosmetics: alignment
Originally committed as revision 14304 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-19 15:43:34 +00:00
Vitor Sessak
3c617380e7 Simplify co(): do not abuse pointer aritmetics
Originally committed as revision 14303 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-19 15:41:15 +00:00
Vitor Sessak
6bc5714be6 Simplify co(): remove variables that are only used once
Originally committed as revision 14301 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-19 15:32:29 +00:00
Vitor Sessak
96e8987e5b Simplify: do not overuse pointer aritmetic
Originally committed as revision 14271 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-17 23:48:53 +00:00
Vitor Sessak
c52d2da905 Declare parameters of pred() that could be const as such
Originally committed as revision 14270 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-17 23:02:52 +00:00
Vitor Sessak
efa85e5e45 Another simplification
Originally committed as revision 14269 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-17 23:01:31 +00:00
Vitor Sessak
4a51e06bfc Simplify
Originally committed as revision 14268 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-17 22:59:53 +00:00
Vitor Sessak
02a1eb9329 Simplify
Originally committed as revision 14267 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-17 22:42:21 +00:00
Vitor Sessak
e4265007a6 Declare function parameters as const
Originally committed as revision 14266 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-17 22:39:29 +00:00
Vitor Sessak
0e3510a31f Revert r14218. FFMAX is a macro, so
sum = FFMAX(1, scalar_product(...));
might calculate the scalar product twice, which is slower.

Originally committed as revision 14230 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-14 14:41:01 +00:00
Vitor Sessak
b903a9285e Simplify
Originally committed as revision 14218 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-13 20:41:09 +00:00
Vitor Sessak
3819081f75 Another av_clipf() usage
Originally committed as revision 14217 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-13 20:39:55 +00:00
Vitor Sessak
aa64ee30d1 Do not declare as double a var that only stores a float
Originally committed as revision 14216 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-13 20:38:42 +00:00
Vitor Sessak
1894db1487 Simplify: use FFMAX
Originally committed as revision 14215 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-13 20:36:32 +00:00
Vitor Sessak
4138ad961c Add av_clipf() function to common.h and use it in ra288.c
Originally committed as revision 14213 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-13 19:59:44 +00:00
Vitor Sessak
5be30c59da Simplify: glob->phase is always == 3
Originally committed as revision 14094 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-06 14:23:30 +00:00
Vitor Sessak
f803495415 Remove phasep context var, it is just phase*5
Originally committed as revision 14092 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-06 12:03:21 +00:00
Vitor Sessak
eb73742d6e Slighly faster operation
Originally committed as revision 14091 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-06 12:01:59 +00:00
Vitor Sessak
8edfd15175 Cosmetics: remove braces
Originally committed as revision 14089 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-06 10:35:01 +00:00
Vitor Sessak
a5f0f504df Another use of scalar_product_float()
Originally committed as revision 14088 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-06 10:32:29 +00:00
Vitor Sessak
a03a6db1af Do not reimplement memmove()
Originally committed as revision 14087 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-06 10:30:21 +00:00
Vitor Sessak
647b90ae0e Cosmetics: remove braces
Originally committed as revision 14080 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-05 18:43:24 +00:00
Vitor Sessak
7a64649d68 Create a function for float dot product instead of duplicating all over the file
Originally committed as revision 14079 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-05 18:42:08 +00:00
Vitor Sessak
6f1e7a9e09 Write for loops in a more standard way. In my opinion, they are much more readable now.
Originally committed as revision 14068 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-04 19:55:58 +00:00
Vitor Sessak
69c23e6f33 Move vars declaration to inner loop when possible
Originally committed as revision 14066 to svn://svn.ffmpeg.org/ffmpeg/trunk
2008-07-04 15:37:52 +00:00