search for: ogg_pcm_float32

...bit width is a mistake. I would suggest instead a single field that encodes all this information in a enumeration. Ie: OGG_PCM_U8 /* Unsigned 8 bit */ OGG_PCM_S8 /* Signed 8 bit. */ OGG_PCM_S16 OGG_PCM_S24 OGG_PCM_S32 OGG_PCM_FLOAT32 OGG_PCM_FLOAT64 and so on. This scheme makes it very difficult to get signed/unsigned and bitwith messed up. d) Don't bother implementing unsigned PCM for bit widths greater than 8 bits. No other common file format uses it and those unsigned formats are a pain...

...all this information > > in a enumeration. Ie: > > > > OGG_PCM_U8 /* Unsigned 8 bit */ > > OGG_PCM_S8 /* Signed 8 bit. */ > > OGG_PCM_S16 > > OGG_PCM_S24 > > OGG_PCM_S32 > > OGG_PCM_FLOAT32 > > OGG_PCM_FLOAT64 > > > > and so on. This scheme makes it very difficult to get > > signed/unsigned and bitwith messed up. You didn't address this issue. Do you think it is unimportant? > > d) Don't bother implementing unsigned PCM fo...

...instead a single field that encodes all this information > in a enumeration. Ie: > > OGG_PCM_U8 /* Unsigned 8 bit */ > OGG_PCM_S8 /* Signed 8 bit. */ > OGG_PCM_S16 > OGG_PCM_S24 > OGG_PCM_S32 > OGG_PCM_FLOAT32 > OGG_PCM_FLOAT64 > > and so on. This scheme makes it very difficult to get > signed/unsigned and bitwith messed up. > d) Don't bother implementing unsigned PCM for bit widths > greater than 8 bits. No other common file format uses > it and...

...t > instead a single field that encodes all this information > in a enumeration. Ie: > > OGG_PCM_U8 /* Unsigned 8 bit */ > OGG_PCM_S8 /* Signed 8 bit. */ > OGG_PCM_S16 > OGG_PCM_S24 > OGG_PCM_S32 > OGG_PCM_FLOAT32 > OGG_PCM_FLOAT64 > > and so on. This scheme makes it very difficult to get > signed/unsigned and bitwith messed up. With the exception of the top two being inverted, these tables look identical. You went on to suggest: > e) Consider whether the endianness sh...

Arc wrote: > Ok so we cap it to 64bit, since much more than that doesn't make sense (96bit > would be a "long double" C type) On x86 CPUs, "long double" is 80 bits. > I really don't like this idea, but I will entertain, formatting it as follows: > > ID Type Bits > 0 Int 8 > 1 uInt 8 > 2 Int 16 > 3 Int 24 > 4 Int