On Wed, Feb 29, 2012 at 01:44:23PM +0100, Diego Biurrun wrote: > Please get another mailer, this Yahoo shit is unbearable. > > On Tue, Feb 28, 2012 at 06:16:59PM -0800, Mashiat Sarker Shakkhar wrote: > > >> +static void decode_cdlms(WmallDecodeCtx *s) > > > > > > And what is cdlms anyway? You have so many Doxygen comments, but > > > this is not explained anywhere. > > > > CDLMS is like a regular LMS filter, but only operates on a single channel > > to reduce intra-channel redundancy. The exact acronym seems to be M$- > > specific; and it expands to "Cascaded Least Mean Squared" (found it in an > > M$ patent). > > > > No need to explain that in the code, anyone who knows audio codecs knows > > what an LMS filter is. (I don't know audio codecs, and even I know what > > an LMS filter is.) The multi-channel variant of the above is called MCLMS > > in this code and in the spec. > > Let me get this straight - you had to look this up in a patent yourself > but you feel there is no need to explain it to others?
>From my experience - I only wondered a bit what does "CD" mean but LMS part was clear to me (even if I don't know much about filtering or audio). So yes, there's no need to explain it. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
