diff options
| author | Jordan Petridis <jordan@centricular.com> | 2020-08-04 15:17:35 +0300 |
|---|---|---|
| committer | Jordan Petridis <jordan@centricular.com> | 2020-08-04 15:19:08 +0300 |
| commit | 31d683d59e4b48401b915c90588afa0fed20f029 (patch) | |
| tree | 58ad54205f89ef7c63cc68cca9ff4ed1510bbb9b | |
| parent | 0d246fb8b0c5712294524e698007f29817f02572 (diff) | |
| download | gstreamer-plugins-base-31d683d59e4b48401b915c90588afa0fed20f029.tar.gz | |
alsamidisrc: fix compiler warning with clang 10
```
../subprojects/gst-plugins-base/ext/alsa/gstalsamidisrc.c:201:54: error: converting the result of '<<' to a boolean always evaluates to false [-Werror,-Wtautological-constant-compare]
snd_seq_ev_schedule_real (&ev, alsamidisrc->queue, SND_SEQ_TIME_MODE_ABS,
^
/usr/include/alsa/seq_event.h:215:34: note: expanded from macro 'SND_SEQ_TIME_MODE_ABS'
```
The ALSA API expects 0 or 1 here and will then add the flags accordingly,
and that's also what other code using this API does.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/-/merge_requests/777>
| -rw-r--r-- | ext/alsa/gstalsamidisrc.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/ext/alsa/gstalsamidisrc.c b/ext/alsa/gstalsamidisrc.c index 0e6b0767a..d7cc6197f 100644 --- a/ext/alsa/gstalsamidisrc.c +++ b/ext/alsa/gstalsamidisrc.c @@ -198,8 +198,7 @@ schedule_next_tick (GstAlsaMidiSrc * alsamidisrc) GST_TIME_TO_TIMESPEC (alsamidisrc->tick * MIDI_TICK_PERIOD_MS * GST_MSECOND, time); - snd_seq_ev_schedule_real (&ev, alsamidisrc->queue, SND_SEQ_TIME_MODE_ABS, - &time); + snd_seq_ev_schedule_real (&ev, alsamidisrc->queue, 0, &time); ret = snd_seq_event_output (alsamidisrc->seq, &ev); if (ret < 0) |