From 0cb4dc5c1fded81b4f47945b51b8de0b0151bf67 Mon Sep 17 00:00:00 2001
From: Friedemann Kleint <Friedemann.Kleint@qt.io>
Date: Thu, 20 Oct 2016 09:18:19 +0200
Subject: Alsa: Use QVarLengthArray

Fix g++ / Clang warnings:
alsaaudioinput.cpp: In member function 'qint64 QAlsaAudioInput::read(char*, qint64)':
qalsaaudioinput.cpp:530:36: warning: variable length array 'buffer' is used [-Wvla]
qalsaaudiooutput.cpp: In member function 'qint64 QAlsaAudioOutput::write(const char*, qint64)':
qalsaaudiooutput.cpp:570:23: warning: variable length array 'out' is used [-Wvla]

(variable length arrays being C99 features).

Change-Id: Id5024af7a42dffb56fbbeda59e1fadb41c660e41
Reviewed-by: Christian Stromme <christian.stromme@qt.io>
Reviewed-by: Yoann Lopes <yoann.lopes@qt.io>
---
 src/plugins/alsa/qalsaaudiooutput.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'src/plugins/alsa/qalsaaudiooutput.cpp')

diff --git a/src/plugins/alsa/qalsaaudiooutput.cpp b/src/plugins/alsa/qalsaaudiooutput.cpp
index 5e444a0a9..cd97ae85b 100644
--- a/src/plugins/alsa/qalsaaudiooutput.cpp
+++ b/src/plugins/alsa/qalsaaudiooutput.cpp
@@ -49,6 +49,7 @@
 //
 
 #include <QtCore/qcoreapplication.h>
+#include <QtCore/qvarlengtharray.h>
 #include <QtMultimedia/private/qaudiohelpers_p.h>
 #include "qalsaaudiooutput.h"
 #include "qalsaaudiodeviceinfo.h"
@@ -567,9 +568,9 @@ qint64 QAlsaAudioOutput::write( const char *data, qint64 len )
     frames = snd_pcm_bytes_to_frames(handle, space);
 
     if (m_volume < 1.0f) {
-        char out[space];
-        QAudioHelperInternal::qMultiplySamples(m_volume, settings, data, out, space);
-        err = snd_pcm_writei(handle, out, frames);
+        QVarLengthArray<char, 4096> out(space);
+        QAudioHelperInternal::qMultiplySamples(m_volume, settings, data, out.data(), space);
+        err = snd_pcm_writei(handle, out.constData(), frames);
     } else {
         err = snd_pcm_writei(handle, data, frames);
     }
-- 
cgit v1.2.3