[LAD] [ANN] Radium V1.9.1

Erik de Castro Lopo mle+la at mega-nerd.com
Sun Nov 11 22:00:29 UTC 2012


Kjetil Matheussen wrote:

> Done:
> https://github.com/erikd/libsndfile/issues/25
> 
> I've linked to a reproducable test case there.

Please don't CC email on this mail list. I am subscribed.

Also, to post code in a github ticket you need to make sure whole
block is indented by 4 spaces or more.

My update from github:

> Sorry, this code:
>
>     while(frames_read < sf_info.frames){
>         float samples[1024*sf_info.channels];
> 
>         int read_now = sf_readf_float(sndfile, samples, 1024);
>         printf("read_now: %d. frames_read: %d. num_frames: %d.\n",read_now,
>                frames_read,(int)sf_info.frames);
> 
>         frames_read += read_now;
>     }
> 
> should be considered broken.
> 
> The value in sf_info.frames can be incorrect (some validation and
> correction is done, but it is not foolproof). The correct usage is
> something like:
> 
>     float samples[1024*sf_info.channels];
>     while((read_now = sf_readf_float(sndfile, samples, 1024)) > 0){
> 
>         printf("read_now: %d. frames_read: %d. num_frames: %d.\n",read_now,
>                frames_read,(int)sf_info.frames);
> 
>         frames_read += read_now;
>     }
> 
> PS : The only way to ensure that sf_info.frames is correct is to read
> in the whole file and that would be slow for anything other than the
> shortest of files.


Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/



More information about the Linux-audio-dev mailing list