rust-av / avp

Another Video Player
MIT License
49 stars 9 forks source link

Add vorbis support #5

Closed est31 closed 6 years ago

est31 commented 6 years ago

I've verified it with a webm file that contained vp9 video and vorbis audio. I did have issues with the frequency but maybe the cause for this is that avp doesn't perform any resampling yet.

est31 commented 6 years ago

I think the issues I had with the sample rate are a more general bug of avp. I'm seeing the same behaviour with the sample webm files from the matroska crate.

lu-zero commented 6 years ago

Yes it is related to https://github.com/rust-av/avp/issues/2 . Probably the first incantation of av-resample will use @tmatth speexdsp resampler.

lu-zero commented 6 years ago

Manually merged w/out the Cargo.lock, thank you :)