On Fri, Oct 14, 2011 at 05:11:40AM -0600, John Brooks wrote: > On Thu, Oct 13, 2011 at 8:06 AM, Benjamin Larsson <[email protected]> > wrote: > > > > On 10/13/2011 11:34 AM, John Brooks wrote: > >> > >> The cbSize field should be included in all cases, even with PCM where > >> its value is ignored. > >> > >> Fixes encoding PCM audio in Matroska for some players which insist on > >> a full WAVEFORMATEX structure for A_MS/ACM audio. > > > > > > Looks ok. Is there regtests that tests both code paths? > > > > The attached patch should fix the regtests. To the best of my > knowledge, none of the affected codecs were broken..
looks good, thanks. Also ok-ed by Benjamin on irc. Squashed into the code change commit, will push in a moment. Janne _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
