search for: speex_get_num_fram

...general approach is correct, I'll change code to use > modes from modes.c. I haven't tested, but it looks sane. Just a few comments: - printf("\tERROR not enough bits left after wb\n"); this is actually now an error. It just means you got to the end of the frame - I suggest speex_get_num_frames() instead. That works for both narrowband and wideband. Number of samples could be misleading because of nb vs wb frame size. - You can change it to use the nb mode info in modes.c, but leave the wideband like it is now. Otherwise, it would cause the wideband stuff to be linked in all the time....

Hi, Ok, here is cleaned up and fixed version. * Function is named speex_get_num_frames() now and return number of frames, as you suggested. * WB layers sizes are taken from wb_skip_table[], while NB frame sizes are calculated with speex_mode_query(). I've tested it with testenc in NB, WB and UWB modes with VBR enabled to test in as many cases as possible. It seems to work fine...

Alexander Chemeris a ?crit : > Hi Jean-Marc, > Is your proposition that ever this code will be written in terms > of internal Speex bits it will be included into libspeex still in force? yes. I'd like to include that if someone writes it using SpeexBits. Cheers, Jean-Marc

> Ok, here is cleaned up and fixed version. > > * Function is named speex_get_num_frames() now and return > number of frames, as you suggested. > * WB layers sizes are taken from wb_skip_table[], while NB frame > sizes are calculated with speex_mode_query(). Looking better. Just make sure to remove the stuff that isn't C99-compatible (e.g. // comments). > I've t...