Incorrect conversion of Float to Int

[dh1tw]

Hi,

I tried to convert a Float32Buffer to an IntBuffer, resulting in silence. Problem is, that the current implementation has a flaw:

audio/float_buffer.go

Line 36 in 9867513

newB.Data[i] = int(buf.Data[i])

audio/float_buffer.go

Line 107 in 9867513

newB.Data[i] = int(buf.Data[i])

Float audio samples use the value range of -1....0....1 casting them just to int won't do the job. For a proper conversion, the bit depth of the integer (8/16/32/64) must be known. Here is a good stack overflow article how to do it right.

Problem is, that within a Float32Buffer, the bit depth is not available :-(

[mattetti]

@dh1tw there is a transform for that https://github.com/go-audio/transforms/blob/master/pcm_scale.go#L27 It does expect that you kept around the original bit depth. you can see that the PCMBuffer uses SourceBitDepth to track that. The Float32Buffer also does have that field: https://github.com/go-audio/audio/blob/master/float_buffer.go#L85

I still do need to go and clean up the other types but I'm thinking about entirely dropping Floats and only keep float32s.