On 2/24/2015 2:01 PM, Diego Biurrun wrote:
> Some questions for Niels ...

I'm not Niels, but...

>> +typedef struct XllChSetSubHeader {
>> +    int fs_interpolate;         ///< sampling frequency interpolation 
>> multiplier
> 
> What is "fs" supposed to stand for?

Frequency Sampling.

>> +    int hier_chset;
> 
> hier?

Hierarchical Channel Set or Channel Set Hierarchy.

>> +    int xll_banddata_crcen;     ///< presence of CRC16 within each 
>> frequency band
> 
> crcen?

CRC is Encoded.

> The case labels should be enums or defines IMO.

You're the one who "reengineered" it. ;)

>> +    /* align to 8-bit and skip 16-bit CRC */
> 
> You mean "Align to 8 bits and skip 16-bit CRC."?

Semantics. They mean the same thing.

These were all pretty clear from the comments and code, and I know
jack about DTS or audio in general.

- Derek

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to