tim-janik requested changes on this pull request.
> @@ -427,7 +427,8 @@ class AlsaPcmDriver : public PcmDriver {
if (write_handle_)
{
int n, buffer_length = n_periods_ * period_size_; // buffer size
chosen by ALSA based on latency request
- const float *zeros = bse_engine_const_zeros (buffer_length / 2); //
sizeof (int16) / sizeof (float)
+ const size_t frame_size = n_channels_ * sizeof (period_buffer_[0]);
+ const uint8 zeros[buffer_length * frame_size] = { 0, };
Clang doesn't support intiializing variable length arrays from non-const.
Generally, period_size_ should never be bigger than the engineblock size,
actually the two should be same. It's probably most simple to call
snd_pcm_writei for n_periods_ times.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/tim-janik/beast/pull/127#pullrequestreview-289763166_______________________________________________
beast mailing list
[email protected]
https://mail.gnome.org/mailman/listinfo/beast
