On Tue, Feb 10, 2009 at 09:35:22AM +0100, Alexandre Ratchov wrote:
> On Tue, Feb 10, 2009 at 08:55:14AM +0100, David Coppa wrote:
> > After aucat changes that go in last week, is this piece of code wrong?
> >
> > PStringArray PSoundChannelLIBSNDIO::GetDeviceNames(Directions)
> > {
> > static const char * const devices[] = {
> > "default",
> > "/tmp/aucat.sock",
> > "/dev/audio0",
> > "/dev/audio1",
> > "/dev/audio2"
> > };
> >
> > return PStringArray(PARRAYSIZE(devices), devices);
> > }
> >
>
> /tmp/aucat.sock is no more used, the other entries are correct.
>
> We don't have a standard way to enumerate valid devices; hardcoding
> a device list this way will stop working the day we change the
> device naming. In order to avoid name collisions, the device naming
> will have to change as soon as we add new device types to libsndio.
> The problem is still unsolved, sorry.
>
> the safest for now is to ask the user to type the device string. If
> an application depends on the device list, then provide only one
> entry, that is mapped to the NULL string.
>
oh, i didn't answer the main question: that's what "default" does
here, it's the one mapped to NULL. So it's ok, besides the unused
entry that should be fixed.
-- Alexandre
