struct aout_sys_t
{
snd_pcm_t *pcm;
- void (*reorder) (void *, size_t, unsigned);
- float soft_gain;
+ unsigned rate; /**< Sample rate */
+ uint8_t chans_table[AOUT_CHAN_MAX]; /**< Channels order table */
+ uint8_t chans_to_reorder; /**< Number of channels to reorder */
+ uint8_t bits; /**< Bits per sample per channel */
+
bool soft_mute;
- audio_sample_format_t format;
+ float soft_gain;
+ char *device;
};
#include "volume.h"
static int Open (vlc_object_t *);
static void Close (vlc_object_t *);
static int EnumDevices (vlc_object_t *, char const *, char ***, char ***);
-static void GetDevices (vlc_object_t *, const char *);
#define AUDIO_DEV_TEXT N_("Audio output device")
#define AUDIO_DEV_LONGTEXT N_("Audio output device (using ALSA syntax).")
}
#define DumpDeviceStatus(o, p) DumpDeviceStatus(VLC_OBJECT(o), p)
-static int DeviceChanged (vlc_object_t *obj, const char *varname,
- vlc_value_t prev, vlc_value_t cur, void *data)
+static unsigned SetupChannelsUnknown (vlc_object_t *obj,
+ uint16_t *restrict mask)
{
- aout_ChannelsRestart (obj, varname, prev, cur, data);
+ uint16_t map = var_InheritInteger (obj, "alsa-audio-channels");
+ uint16_t chans = *mask & map;
- if (!var_Type (obj, "alsa-audio-device"))
- var_Create (obj, "alsa-audio-device", VLC_VAR_STRING);
- var_SetString (obj, "alsa-audio-device", cur.psz_string);
- return VLC_SUCCESS;
+ if (unlikely(chans == 0)) /* WTH? */
+ chans = AOUT_CHANS_STEREO;
+
+ if (popcount (chans) < popcount (*mask))
+ msg_Dbg (obj, "downmixing from %u to %u channels",
+ popcount (*mask), popcount (chans));
+ else
+ msg_Dbg (obj, "keeping %u channels", popcount (chans));
+ *mask = chans;
+ return 0;
+}
+
+#if (SND_LIB_VERSION >= 0x01001B)
+static const uint16_t vlc_chans[] = {
+ [SND_CHMAP_MONO] = AOUT_CHAN_CENTER,
+ [SND_CHMAP_FL] = AOUT_CHAN_LEFT,
+ [SND_CHMAP_FR] = AOUT_CHAN_RIGHT,
+ [SND_CHMAP_RL] = AOUT_CHAN_REARLEFT,
+ [SND_CHMAP_RR] = AOUT_CHAN_REARRIGHT,
+ [SND_CHMAP_FC] = AOUT_CHAN_CENTER,
+ [SND_CHMAP_LFE] = AOUT_CHAN_LFE,
+ [SND_CHMAP_SL] = AOUT_CHAN_MIDDLELEFT,
+ [SND_CHMAP_SR] = AOUT_CHAN_MIDDLERIGHT,
+ [SND_CHMAP_RC] = AOUT_CHAN_REARCENTER,
+};
+
+static int Map2Mask (vlc_object_t *obj, const snd_pcm_chmap_t *restrict map)
+{
+ uint16_t mask = 0;
+
+ for (unsigned i = 0; i < map->channels; i++)
+ {
+ const unsigned pos = map->pos[i];
+ uint_fast16_t vlc_chan = 0;
+
+ if (pos < sizeof (vlc_chans) / sizeof (vlc_chans[0]))
+ vlc_chan = vlc_chans[pos];
+ if (vlc_chan == 0)
+ {
+ msg_Dbg (obj, " %s channel %u position %u", "unsupported", i, pos);
+ return -1;
+ }
+ if (mask & vlc_chan)
+ {
+ msg_Dbg (obj, " %s channel %u position %u", "duplicate", i, pos);
+ return -1;
+ }
+ mask |= vlc_chan;
+ }
+ return mask;
+}
+
+/**
+ * Compares a fixed ALSA channels map with the VLC channels order.
+ */
+static unsigned SetupChannelsFixed(const snd_pcm_chmap_t *restrict map,
+ uint16_t *restrict mask, uint8_t *restrict tab)
+{
+ uint32_t chans_out[AOUT_CHAN_MAX];
+
+ for (unsigned i = 0; i < map->channels; i++)
+ {
+ uint_fast16_t vlc_chan = vlc_chans[map->pos[i]];
+
+ chans_out[i] = vlc_chan;
+ *mask |= vlc_chan;
+ }
+
+ return aout_CheckChannelReorder(NULL, chans_out, *mask, tab);
+}
+
+/**
+ * Negotiate channels mapping.
+ */
+static unsigned SetupChannels (vlc_object_t *obj, snd_pcm_t *pcm,
+ uint16_t *restrict mask, uint8_t *restrict tab)
+{
+ snd_pcm_chmap_query_t **maps = snd_pcm_query_chmaps (pcm);
+ if (tab == NULL)
+ { /* Fallback to manual configuration */
+ msg_Dbg(obj, "channels map not provided");
+ return SetupChannelsUnknown (obj, mask);
+ }
+
+ /* Find most appropriate available channels map */
+ unsigned best_offset;
+ unsigned best_score = 0;
+
+ for (snd_pcm_chmap_query_t *const *p = maps; *p != NULL; p++)
+ {
+ snd_pcm_chmap_query_t *map = *p;
+
+ switch (map->type)
+ {
+ case SND_CHMAP_TYPE_FIXED:
+ case SND_CHMAP_TYPE_PAIRED:
+ case SND_CHMAP_TYPE_VAR:
+ break;
+ default:
+ msg_Err (obj, "unknown channels map type %u", map->type);
+ continue;
+ }
+
+ int chans = Map2Mask (obj, &map->map);
+ if (chans == -1)
+ continue;
+
+ unsigned score = popcount (chans & *mask);
+ if (score > best_score)
+ {
+ best_offset = p - maps;
+ best_score = score;
+ }
+ }
+
+ if (best_score == 0)
+ {
+ msg_Err (obj, "cannot find supported channels map");
+ snd_pcm_free_chmaps (maps);
+ return SetupChannelsUnknown (obj, mask);
+ }
+
+ const snd_pcm_chmap_t *map = &maps[best_offset]->map;
+ msg_Dbg (obj, "using channels map %u, type %u, %u channel(s)", best_offset,
+ maps[best_offset]->type, best_score);
+
+ /* Setup channels map */
+ unsigned to_reorder = SetupChannelsFixed(map, mask, tab);
+
+ /* TODO: avoid reordering for PAIRED and VAR types */
+ //snd_pcm_set_chmap (pcm, ...)
+
+ snd_pcm_free_chmaps (maps);
+ return to_reorder;
}
+#else /* (SND_LIB_VERSION < 0x01001B) */
+# define SetupChannels(obj, pcm, mask, tab) \
+ SetupChannelsUnknown(obj, mask)
+#endif
static int TimeGet (audio_output_t *aout, mtime_t *);
static void Play (audio_output_t *, block_t *);
static void Pause (audio_output_t *, bool, mtime_t);
static void PauseDummy (audio_output_t *, bool, mtime_t);
static void Flush (audio_output_t *, bool);
-static void Reorder71 (void *, size_t, unsigned);
/** Initializes an ALSA playback stream */
static int Start (audio_output_t *aout, audio_sample_format_t *restrict fmt)
{
aout_sys_t *sys = aout->sys;
-
- /* Get device name */
- char *device = var_InheritString (aout, "alsa-audio-device");
- if (unlikely(device == NULL))
- return VLC_ENOMEM;
-
snd_pcm_format_t pcm_format; /* ALSA sample format */
- vlc_fourcc_t fourcc = fmt->i_format;
bool spdif = false;
- switch (fourcc)
+ switch (fmt->i_format)
{
- case VLC_CODEC_F64B:
- pcm_format = SND_PCM_FORMAT_FLOAT64_BE;
- break;
- case VLC_CODEC_F64L:
- pcm_format = SND_PCM_FORMAT_FLOAT64_LE;
+ case VLC_CODEC_FL64:
+ pcm_format = SND_PCM_FORMAT_FLOAT64;
break;
- case VLC_CODEC_F32B:
- pcm_format = SND_PCM_FORMAT_FLOAT_BE;
+ case VLC_CODEC_FL32:
+ pcm_format = SND_PCM_FORMAT_FLOAT;
break;
- case VLC_CODEC_F32L:
- pcm_format = SND_PCM_FORMAT_FLOAT_LE;
- break;
- case VLC_CODEC_S32B:
- pcm_format = SND_PCM_FORMAT_S32_BE;
- break;
- case VLC_CODEC_S32L:
- pcm_format = SND_PCM_FORMAT_S32_LE;
+ case VLC_CODEC_S32N:
+ pcm_format = SND_PCM_FORMAT_S32;
break;
case VLC_CODEC_S24B:
pcm_format = SND_PCM_FORMAT_S24_3BE;
case VLC_CODEC_U24L:
pcm_format = SND_PCM_FORMAT_U24_3LE;
break;
- case VLC_CODEC_S16B:
- pcm_format = SND_PCM_FORMAT_S16_BE;
- break;
- case VLC_CODEC_S16L:
- pcm_format = SND_PCM_FORMAT_S16_LE;
- break;
- case VLC_CODEC_U16B:
- pcm_format = SND_PCM_FORMAT_U16_BE;
- break;
- case VLC_CODEC_U16L:
- pcm_format = SND_PCM_FORMAT_U16_LE;
- break;
- case VLC_CODEC_S8:
- pcm_format = SND_PCM_FORMAT_S8;
+ case VLC_CODEC_S16N:
+ pcm_format = SND_PCM_FORMAT_S16;
break;
case VLC_CODEC_U8:
pcm_format = SND_PCM_FORMAT_U8;
spdif = var_InheritBool (aout, "spdif");
if (spdif)
{
- fourcc = VLC_CODEC_SPDIFL;
+ fmt->i_format = VLC_CODEC_SPDIFL;
pcm_format = SND_PCM_FORMAT_S16;
}
else
if (HAVE_FPU)
{
- fourcc = VLC_CODEC_FL32;
+ fmt->i_format = VLC_CODEC_FL32;
pcm_format = SND_PCM_FORMAT_FLOAT;
}
else
{
- fourcc = VLC_CODEC_S16N;
+ fmt->i_format = VLC_CODEC_S16N;
pcm_format = SND_PCM_FORMAT_S16;
}
}
- /* ALSA channels */
- /* XXX: maybe this should be shared with other dumb outputs */
- uint32_t map = var_InheritInteger (aout, "alsa-audio-channels");
- map &= fmt->i_physical_channels;
- if (unlikely(map == 0)) /* WTH? */
- map = AOUT_CHANS_STEREO;
-
- unsigned channels = popcount (map);
- if (channels < aout_FormatNbChannels (fmt))
- msg_Dbg (aout, "downmixing from %u to %u channels",
- aout_FormatNbChannels (fmt), channels);
- else
- msg_Dbg (aout, "keeping %u channels", channels);
-
- /* Choose the IEC device for S/PDIF output:
- if the device is overridden by the user then it will be the one.
- Otherwise we compute the default device based on the output format. */
+ const char *device = sys->device;
+ char *devbuf = NULL;
+ /* Choose the IEC device for S/PDIF output */
if (spdif && !strcmp (device, "default"))
{
unsigned aes3;
break;
}
- free (device);
- if (asprintf (&device,
+ if (asprintf (&devbuf,
"iec958:AES0=0x%x,AES1=0x%x,AES2=0x%x,AES3=0x%x",
IEC958_AES0_CON_EMPHASIS_NONE | IEC958_AES0_NONAUDIO,
IEC958_AES1_CON_ORIGINAL | IEC958_AES1_CON_PCM_CODER,
0, aes3) == -1)
return VLC_ENOMEM;
+ device = devbuf;
}
/* Open the device */
const int mode = SND_PCM_NO_AUTO_RESAMPLE;
int val = snd_pcm_open (&pcm, device, SND_PCM_STREAM_PLAYBACK, mode);
-#if (SND_LIB_VERSION <= 0x010015)
-# warning Please update alsa-lib to version > 1.0.21a.
- var_Create (aout->p_libvlc, "alsa-working", VLC_VAR_BOOL);
- if (val != 0 && var_GetBool (aout->p_libvlc, "alsa-working"))
- dialog_Fatal (aout, "ALSA version problem",
- "VLC failed to re-initialize your audio output device.\n"
- "Please update alsa-lib to version 1.0.22 or higher "
- "to fix this issue.");
- var_SetBool (aout->p_libvlc, "alsa-working", !val);
-#endif
+ free (devbuf);
if (val != 0)
{
-#if (SND_LIB_VERSION <= 0x010017)
-# warning Please update alsa-lib to version > 1.0.23.
- var_Create (aout->p_libvlc, "alsa-broken", VLC_VAR_BOOL);
- if (!var_GetBool (aout->p_libvlc, "alsa-broken"))
- {
- var_SetBool (aout->p_libvlc, "alsa-broken", true);
- dialog_Fatal (aout, "Potential ALSA version problem",
- "VLC failed to initialize your audio output device (if any).\n"
- "Please update alsa-lib to version 1.0.24 or higher "
- "to try to fix this issue.");
- }
-#endif
- msg_Err (aout, "cannot open ALSA device \"%s\": %s", device,
+ msg_Err (aout, "cannot open ALSA device \"%s\": %s", sys->device,
snd_strerror (val));
dialog_Fatal (aout, _("Audio output failed"),
_("The audio device \"%s\" could not be used:\n%s."),
- device, snd_strerror (val));
- free (device);
+ sys->device, snd_strerror (val));
return VLC_EGENERIC;
}
sys->pcm = pcm;
/* Print some potentially useful debug */
- msg_Dbg (aout, "using ALSA device: %s", device);
+ msg_Dbg (aout, "using ALSA device: %s", sys->device);
DumpDevice (VLC_OBJECT(aout), pcm);
/* Get Initial hardware parameters */
else
if (snd_pcm_hw_params_test_format (pcm, hw, SND_PCM_FORMAT_FLOAT) == 0)
{
- fourcc = VLC_CODEC_FL32;
+ fmt->i_format = VLC_CODEC_FL32;
pcm_format = SND_PCM_FORMAT_FLOAT;
}
else
if (snd_pcm_hw_params_test_format (pcm, hw, SND_PCM_FORMAT_S32) == 0)
{
- fourcc = VLC_CODEC_S32N;
+ fmt->i_format = VLC_CODEC_S32N;
pcm_format = SND_PCM_FORMAT_S32;
}
else
if (snd_pcm_hw_params_test_format (pcm, hw, SND_PCM_FORMAT_S16) == 0)
{
- fourcc = VLC_CODEC_S16N;
+ fmt->i_format = VLC_CODEC_S16N;
pcm_format = SND_PCM_FORMAT_S16;
}
else
}
/* Set channels count */
+ unsigned channels;
+ if (!spdif)
+ {
+ sys->chans_to_reorder = SetupChannels (VLC_OBJECT(aout), pcm,
+ &fmt->i_physical_channels, sys->chans_table);
+ channels = popcount (fmt->i_physical_channels);
+ }
+ else
+ channels = 2;
+ fmt->i_original_channels = fmt->i_physical_channels;
+
/* By default, ALSA plug will pad missing channels with zeroes, which is
* usually fine. However, it will also discard extraneous channels, which
* is not acceptable. Thus the user must configure the physically
}
/* Set sample rate */
- unsigned rate = fmt->i_rate;
- val = snd_pcm_hw_params_set_rate_near (pcm, hw, &rate, NULL);
+ val = snd_pcm_hw_params_set_rate_near (pcm, hw, &fmt->i_rate, NULL);
if (val)
{
msg_Err (aout, "cannot set sample rate: %s", snd_strerror (val));
goto error;
}
- if (fmt->i_rate != rate)
- msg_Dbg (aout, "resampling from %d Hz to %d Hz", fmt->i_rate, rate);
+ sys->rate = fmt->i_rate;
/* Set buffer size */
param = AOUT_MAX_ADVANCE_TIME;
}
/* Setup audio_output_t */
- fmt->i_format = fourcc;
- fmt->i_rate = rate;
- sys->reorder = NULL;
if (spdif)
{
fmt->i_bytes_per_frame = AOUT_SPDIF_SIZE;
}
else
{
- fmt->i_original_channels =
- fmt->i_physical_channels = map;
- switch (popcount (map))
- {
- case 8:
- sys->reorder = Reorder71;
- break;
- }
+ aout_FormatPrepare (fmt);
+ sys->bits = fmt->i_bitspersample;
}
- sys->format = *fmt;
aout->time_get = TimeGet;
aout->play = Play;
msg_Warn (aout, "device cannot be paused");
}
aout->flush = Flush;
-
- /* Setup audio-device choices */
- {
- vlc_value_t text;
-
- var_Create (aout, "audio-device", VLC_VAR_STRING | VLC_VAR_HASCHOICE);
- text.psz_string = _("Audio Device");
- var_Change (aout, "audio-device", VLC_VAR_SETTEXT, &text, NULL);
-
- GetDevices (VLC_OBJECT(aout), device);
- }
- var_AddCallback (aout, "audio-device", DeviceChanged, NULL);
-
- free (device);
aout_SoftVolumeStart (aout);
return 0;
error:
snd_pcm_close (pcm);
- free (device);
return VLC_EGENERIC;
}
-static int TimeGet (audio_output_t *aout, mtime_t *restrict pts)
+static int TimeGet (audio_output_t *aout, mtime_t *restrict delay)
{
aout_sys_t *sys = aout->sys;
- snd_pcm_t *pcm = sys->pcm;
- snd_pcm_status_t *status;
- int val;
+ snd_pcm_sframes_t frames;
- snd_pcm_status_alloca (&status);
- val = snd_pcm_status (pcm, status);
- if (val < 0)
+ int val = snd_pcm_delay (sys->pcm, &frames);
+ if (val)
{
- msg_Err (aout, "cannot get status: %s", snd_strerror (val));
+ msg_Err (aout, "cannot estimate delay: %s", snd_strerror (val));
return -1;
}
-
- if (snd_pcm_status_get_state (status) != SND_PCM_STATE_RUNNING)
- return -1;
-
- snd_pcm_sframes_t frames = snd_pcm_status_get_delay (status);
- *pts = mdate () + (frames * CLOCK_FREQ / sys->format.i_rate);
+ *delay = frames * CLOCK_FREQ / sys->rate;
return 0;
}
{
aout_sys_t *sys = aout->sys;
- if (sys->reorder != NULL)
- sys->reorder (block->p_buffer, block->i_nb_samples,
- sys->format.i_bitspersample / 8);
+ if (sys->chans_to_reorder != 0)
+ aout_ChannelReorder(block->p_buffer, block->i_buffer,
+ sys->chans_to_reorder, sys->chans_table, sys->bits);
snd_pcm_t *pcm = sys->pcm;
aout_sys_t *sys = aout->sys;
snd_pcm_t *pcm = sys->pcm;
- var_DelCallback (aout, "audio-device", DeviceChanged, NULL);
- var_Destroy (aout, "audio-device");
-
snd_pcm_drop (pcm);
snd_pcm_close (pcm);
}
-/**
- * Converts from VLC to ALSA order for 7.1.
- * VLC has middle channels in position 2 and 3, ALSA in position 6 and 7.
- */
-static void Reorder71 (void *p, size_t n, unsigned size)
-{
- switch (size)
- {
- case 4:
- for (uint64_t *ptr = p; n > 0; ptr += 4, n--)
- {
- uint64_t middle = ptr[1], c_lfe = ptr[2], rear = ptr[3];
- ptr[1] = c_lfe; ptr[2] = rear; ptr[3] = middle;
- }
- break;
- case 2:
- for (uint32_t *ptr = p; n > 0; ptr += 4, n--)
- {
- uint32_t middle = ptr[1], c_lfe = ptr[2], rear = ptr[3];
- ptr[1] = c_lfe; ptr[2] = rear; ptr[3] = middle;
- }
- break;
-
- default:
- for (uint16_t *ptr = p; n > 0; n--)
- {
- uint16_t middle[size];
- memcpy (middle, ptr + size, size * 2);
- ptr += size;
- memcpy (ptr, ptr + size, size * 2);
- ptr += size;
- memcpy (ptr, ptr + size, size * 2);
- ptr += size;
- memcpy (ptr, middle, size * 2);
- ptr += size;
- }
- break;
- }
-}
-
-
/**
* Enumerates ALSA output devices.
*/
static int EnumDevices(vlc_object_t *obj, char const *varname,
- char ***restrict vp, char ***restrict tp)
+ char ***restrict idp, char ***restrict namep)
{
- unsigned n = 0;
-
- char **names = xmalloc(sizeof (*names));
- char **descs = xmalloc(sizeof (*names));
- names[0] = strdup ("default");
- descs[0] = strdup (N_("Default"));
- n++;
- if (unlikely(names[0] == NULL || descs[0] == NULL))
- abort();
-
void **hints;
+
+ msg_Dbg (obj, "Available ALSA PCM devices:");
if (snd_device_name_hint(-1, "pcm", &hints) < 0)
- return n;
+ return -1;
+
+ char **ids = NULL, **names = NULL;
+ unsigned n = 0;
for (size_t i = 0; hints[i] != NULL; i++)
{
continue;
char *desc = snd_device_name_get_hint(hint, "DESC");
- if (desc == NULL)
- {
- free (name);
- continue;
- }
+ if (desc != NULL)
+ for (char *lf = strchr(desc, '\n'); lf; lf = strchr(lf, '\n'))
+ *lf = ' ';
+ msg_Dbg (obj, "%s (%s)", (desc != NULL) ? desc : name, name);
+ ids = xrealloc (ids, (n + 1) * sizeof (*ids));
names = xrealloc (names, (n + 1) * sizeof (*names));
- descs = xrealloc (descs, (n + 1) * sizeof (*descs));
- names[n] = name;
- descs[n] = desc;
+ ids[n] = name;
+ names[n] = desc;
n++;
}
- snd_device_name_free_hint(hints);
- (void) obj; (void) varname;
- *vp = names;
- *tp = descs;
+ snd_device_name_free_hint(hints);
+ *idp = ids;
+ *namep = names;
+ (void) varname;
return n;
}
-
-static void GetDevices(vlc_object_t *obj, const char *prefs_dev)
+static int DevicesEnum (audio_output_t *aout, char ***idp, char ***namep)
{
- void **hints;
-
- msg_Dbg(obj, "Available ALSA PCM devices:");
- if (snd_device_name_hint(-1, "pcm", &hints) < 0)
- return;
-
- vlc_value_t val, text;
- bool hinted_default = false;
- bool hinted_prefs = !strcmp (prefs_dev, "default");
-
- for (size_t i = 0; hints[i] != NULL; i++)
- {
- void *hint = hints[i];
-
- char *name = snd_device_name_get_hint(hint, "NAME");
- if (unlikely(name == NULL))
- continue;
-
- char *desc = snd_device_name_get_hint(hint, "DESC");
- if (desc != NULL)
- for (char *lf = strchr(desc, '\n'); lf; lf = strchr(lf, '\n'))
- *lf = ' ';
- msg_Dbg(obj, "%s (%s)", (desc != NULL) ? desc : name, name);
-
- if (!strcmp (name, "default"))
- hinted_default = true;
- if (!strcmp (name, prefs_dev))
- hinted_prefs = true;
-
- val.psz_string = name;
- text.psz_string = desc;
- var_Change(obj, "audio-device", VLC_VAR_ADDCHOICE, &val, &text);
- free(desc);
- free(name);
- }
+ return EnumDevices (VLC_OBJECT(aout), NULL, idp, namep);
+}
- snd_device_name_free_hint(hints);
+static int DeviceSelect (audio_output_t *aout, const char *id)
+{
+ aout_sys_t *sys = aout->sys;
- if (!hinted_default)
- {
- val.psz_string = (char *)"default";
- text.psz_string = (char *)N_("Default");
- var_Change(obj, "audio-device", VLC_VAR_ADDCHOICE, &val, &text);
- }
+ char *device = strdup (id ? id : "default");
+ if (unlikely(device == NULL))
+ return -1;
- val.psz_string = (char *)prefs_dev;
- if (!hinted_prefs)
- {
- text.psz_string = (char *)N_("VLC preferences");
- var_Change(obj, "audio-device", VLC_VAR_ADDCHOICE, &val, &text);
- }
- var_Change(obj, "audio-device", VLC_VAR_SETVALUE, &val, NULL);
+ free (sys->device);
+ sys->device = device;
+ aout_DeviceReport (aout, device);
+ aout_RestartRequest (aout, AOUT_RESTART_OUTPUT);
+ return 0;
}
static int Open(vlc_object_t *obj)
if (unlikely(sys == NULL))
return VLC_ENOMEM;
+ sys->device = var_InheritString (aout, "alsa-audio-device");
+ if (unlikely(sys->device == NULL))
+ goto error;
+
aout->sys = sys;
aout->start = Start;
aout->stop = Stop;
aout_SoftVolumeInit (aout);
+ aout->device_enum = DevicesEnum;
+ aout->device_select = DeviceSelect;
+ aout_DeviceReport (aout, sys->device);
return VLC_SUCCESS;
+error:
+ free (sys);
+ return VLC_ENOMEM;
}
static void Close(vlc_object_t *obj)
audio_output_t *aout = (audio_output_t *)obj;
aout_sys_t *sys = aout->sys;
- free(sys);
+ free (sys->device);
+ free (sys);
}