improve mixer code

4 files changed, 72 insertions, 91 deletions

diff --git a/examples/opiemm/osoundsystemdemo/osoundsystemdemo.cpp b/examples/opiemm/osoundsystemdemo/osoundsystemdemo.cpp
index f74a1b9..b18d719 100644
--- a/examples/opiemm/osoundsystemdemo/osoundsystemdemo.cpp
+++ b/examples/opiemm/osoundsystemdemo/osoundsystemdemo.cpp
@@ -50,4 +50,4 @@ int main( int argc, char** argv )
     OSoundCard* card = it.current();
-
     OMixerInterface* mixer = card->mixer();
+    odebug << "This device " << ( mixer->hasMultipleRecording() ? "does" : "does not" ) << " feature multiple recording sources." << oendl;
 
@@ -58,5 +58,11 @@ int main( int argc, char** argv )
         bool stereo = mixer->isStereo( *it );
-        odebug << "OSSDEMO: Mixer has channel " << *it << ( stereo ? "[stereo]" : "[mono]" ) << oendl;
-        odebug << "OSSDEMO:              +--- volume " << ( mixer->volume( *it ) & 0xff )
-               << " (left) | " << ( mixer->volume( *it ) >> 8 ) << " (right)" << oendl;
+        bool recsrc = mixer->isRecordable( *it );
+        QString line = "OSSDEMO: Mixer has channel " + *it + " ";
+        line = line.leftJustify( 50 ) + ( stereo ? "[stereo]" : "[mono]" );
+        line = line.leftJustify( 60 ) + ( recsrc ? "[recsrc]" : "[      ]" );
+        line = line.leftJustify( 70 );
+        line += " [ " + QString::number( mixer->volume( *it ) & 0xff ).rightJustify( 3 );
+        if ( stereo ) line += " | " + QString::number( mixer->volume( *it ) >> 8 ).rightJustify( 3 );
+        line += " ]";
+        odebug << line << oendl;
     }
diff --git a/libopie2/config.in b/libopie2/config.in
deleted file mode 100644
index 5f1bed3..0000000
--- a/libopie2/config.in
+++ b/dev/null
@@ -1,12 +0,0 @@
-#menu "libopie2"
-  comment ""
-  source libopie2/opiecore/config.in
-  source libopie2/opiedb/config.in
-  source libopie2/opienet/config.in
-  source libopie2/opiepim/config.in
-  source libopie2/opieui/config.in
-  source libopie2/opiemm/config.in
-  source libopie2/opiesecurity/config.in
-  comment ""
-#endmenu
-
diff --git a/libopie2/opiemm/osoundsystem.cpp b/libopie2/opiemm/osoundsystem.cpp
index 09fd28f..6c03203 100644
--- a/libopie2/opiemm/osoundsystem.cpp
+++ b/libopie2/opiemm/osoundsystem.cpp
@@ -2,4 +2,3 @@
                              This file is part of the Opie Project
-
-                             (C) 2003 Michael 'Mickey' Lauer <mickey@tm.informatik.uni-frankfurt.de>
+                             (C) 2003-2005 Michael 'Mickey' Lauer <mickey@Vanille.de>
               =.
@@ -42,2 +41,4 @@
 
+static const char* device_label[] = SOUND_DEVICE_LABELS;
+static int max_device_nr = sizeof device_label / sizeof (const char*);
 
@@ -66,3 +67,2 @@ void OSoundSystem::synchronize()
 
-
     /*
@@ -178,3 +178,3 @@ void OAudioInterface::init()
 OMixerInterface::OMixerInterface( QObject* parent, const char* name )
-                 :QObject( parent, name )
+                 :QObject( parent, name ), _devmask( 0 ), _recmask( 0 ), _stmask( 0 )
 {
@@ -201,57 +201,29 @@ void OMixerInterface::init()
     // construct the device capabilities
-    int devmask = 0;
-    if ( ioctl( _fd, SOUND_MIXER_READ_DEVMASK, &devmask ) != -1 )
-    {
-        if ( devmask & ( 1 << SOUND_MIXER_VOLUME ) )    _channels.insert( "PlayVolume", SOUND_MIXER_VOLUME );
-        if ( devmask & ( 1 << SOUND_MIXER_BASS ) )      _channels.insert( "PlayBass", SOUND_MIXER_BASS );
-        if ( devmask & ( 1 << SOUND_MIXER_TREBLE ) )    _channels.insert( "PlayTreble", SOUND_MIXER_TREBLE );
-        if ( devmask & ( 1 << SOUND_MIXER_SYNTH ) )     _channels.insert( "PlaySynth", SOUND_MIXER_SYNTH );
-        if ( devmask & ( 1 << SOUND_MIXER_PCM ) )       _channels.insert( "PlayPCM", SOUND_MIXER_PCM );
-        if ( devmask & ( 1 << SOUND_MIXER_SPEAKER ) )   _channels.insert( "PlaySpeaker", SOUND_MIXER_SPEAKER );
-        if ( devmask & ( 1 << SOUND_MIXER_LINE ) )      _channels.insert( "PlayLine", SOUND_MIXER_LINE );
-        if ( devmask & ( 1 << SOUND_MIXER_MIC ) )       _channels.insert( "PlayMic", SOUND_MIXER_MIC );
-        if ( devmask & ( 1 << SOUND_MIXER_CD ) )        _channels.insert( "PlayCD", SOUND_MIXER_CD );
-        if ( devmask & ( 1 << SOUND_MIXER_IMIX ) )      _channels.insert( "PlayInputMix", SOUND_MIXER_IMIX );
-        if ( devmask & ( 1 << SOUND_MIXER_ALTPCM ) )    _channels.insert( "PlayAltPCM", SOUND_MIXER_ALTPCM );
-        if ( devmask & ( 1 << SOUND_MIXER_RECLEV ) )    _channels.insert( "PlayRecord", SOUND_MIXER_RECLEV );
-        if ( devmask & ( 1 << SOUND_MIXER_IGAIN ) )     _channels.insert( "PlayInputGain", SOUND_MIXER_IGAIN );
-        if ( devmask & ( 1 << SOUND_MIXER_OGAIN ) )     _channels.insert( "PlayOutputGain", SOUND_MIXER_OGAIN );
-        //odebug << "devmask available and constructed." << oendl;
-    }
-
-    devmask = 0;
-    if ( ioctl( _fd, SOUND_MIXER_READ_RECMASK, &devmask ) != -1 )
-    {
-        if ( devmask & ( 1 << SOUND_MIXER_VOLUME ) )    _channels.insert( "RecVolume", SOUND_MIXER_VOLUME );
-        if ( devmask & ( 1 << SOUND_MIXER_BASS ) )      _channels.insert( "RecBass", SOUND_MIXER_BASS );
-        if ( devmask & ( 1 << SOUND_MIXER_TREBLE ) )    _channels.insert( "RecTreble", SOUND_MIXER_TREBLE );
-        if ( devmask & ( 1 << SOUND_MIXER_SYNTH ) )     _channels.insert( "RecSynth", SOUND_MIXER_SYNTH );
-        if ( devmask & ( 1 << SOUND_MIXER_PCM ) )       _channels.insert( "RecPCM", SOUND_MIXER_PCM );
-        if ( devmask & ( 1 << SOUND_MIXER_SPEAKER ) )   _channels.insert( "RecSpeaker", SOUND_MIXER_SPEAKER );
-        if ( devmask & ( 1 << SOUND_MIXER_LINE ) )      _channels.insert( "RecLine", SOUND_MIXER_LINE );
-        if ( devmask & ( 1 << SOUND_MIXER_MIC ) )       _channels.insert( "RecMic", SOUND_MIXER_MIC );
-        if ( devmask & ( 1 << SOUND_MIXER_CD ) )        _channels.insert( "RecCD", SOUND_MIXER_CD );
-        if ( devmask & ( 1 << SOUND_MIXER_IMIX ) )      _channels.insert( "RecInputMix", SOUND_MIXER_IMIX );
-        if ( devmask & ( 1 << SOUND_MIXER_ALTPCM ) )    _channels.insert( "RecAltPCM", SOUND_MIXER_ALTPCM );
-        if ( devmask & ( 1 << SOUND_MIXER_RECLEV ) )    _channels.insert( "RecRecord", SOUND_MIXER_RECLEV );
-        if ( devmask & ( 1 << SOUND_MIXER_IGAIN ) )     _channels.insert( "RecInputGain", SOUND_MIXER_IGAIN );
-        if ( devmask & ( 1 << SOUND_MIXER_OGAIN ) )     _channels.insert( "RecOutputGain", SOUND_MIXER_OGAIN );
-        //odebug << "recmask available and constructed." << oendl;
-    }
-
-    devmask = 0;
-    if ( ioctl( _fd, SOUND_MIXER_READ_STEREODEVS, &devmask ) != -1 )
-    {
-        odebug << "stereomask = " << devmask << oendl;
-    }
-
-/*    ChannelIterator it;
-    for ( it = _channels.begin(); it != _channels.end(); ++it )
-    {
-        odebug << "Channel " <<  it.key() << " available (bit " << it.data() << ")" << oendl;
-        odebug << " +--- Volume: " << volume( it.key() ) & 0xff << " | " << volume( it.key() ) >> 8 << "" << oendl;
+    if ( ioctl( _fd, SOUND_MIXER_READ_CAPS, &_capmask ) != -1 )
+    {
+        odebug << "OMixerInterface::init() - capmask = " << _capmask << oendl;
+    }
+    if ( ioctl( _fd, SOUND_MIXER_READ_DEVMASK, &_devmask ) != -1 )
+    {
+        odebug << "OMixerInterface::init() - devmask = " << _devmask << oendl;
+    }
+    if ( ioctl( _fd, SOUND_MIXER_READ_RECMASK, &_recmask ) != -1 )
+    {
+        odebug << "OMixerInterface::init() - recmask = " << _recmask << oendl;
+    }
+    if ( ioctl( _fd, SOUND_MIXER_READ_STEREODEVS, &_stmask ) != -1 )
+    {
+        odebug << "OMixerInterface::init() - stereomask = " << _stmask << oendl;
+    }
+    for ( int i = 0; i < max_device_nr; ++i )
+    {
+        if ( _devmask & ( 1 << i ) ) 
+        {
+            _channels.insert( QString( device_label[i] ).stripWhiteSpace(), i );
+            odebug << "OMixerInterface::init() - channel '" << device_label[i] << "'" << oendl;
+        }
     }
-*/
 }
 
+
 QStringList OMixerInterface::allChannels() const
@@ -262,4 +234,3 @@ QStringList OMixerInterface::allChannels() const
     {
-        channels += it.key();
-        it++;
+        channels += it++.key();
     }
@@ -271,4 +242,9 @@ QStringList OMixerInterface::recChannels() const
 {
-    owarn << "NYI" << oendl;
-    return QStringList();
+    ChannelIterator it = _channels.begin();
+    QStringList channels;
+    while ( it != _channels.end() )
+    {
+        if ( _recmask & ( 1 << _channels[it.key()] ) ) channels += it++.key();
+    }
+    return channels;
 }
@@ -278,4 +254,9 @@ QStringList OMixerInterface::playChannels() const
 {
-    owarn << "NYI" << oendl;
-    return QStringList();
+    return allChannels();
+}
+
+
+bool OMixerInterface::hasMultipleRecording() const
+{
+    return !( _capmask & SOUND_CAP_EXCL_INPUT );
 }
@@ -291,12 +272,9 @@ bool OMixerInterface::isStereo( const QString& channel ) const
 {
-    bool result = false;
-    if ( _channels.contains( channel ) )
-    {
-        int devmask = 0;
-        if ( ioctl( _fd, SOUND_MIXER_READ_STEREODEVS, &devmask ) != -1 )
-        {
-            result = devmask & ( 1 << _channels[channel] );
+    return _channels.contains( channel ) && ( _stmask & ( 1 << _channels[channel] ) );
         }
-    }
-    return result;
+
+
+bool OMixerInterface::isRecordable( const QString& channel ) const
+{
+    return _channels.contains( channel ) && ( _recmask & ( 1 << _channels[channel] ) );
 }
diff --git a/libopie2/opiemm/osoundsystem.h b/libopie2/opiemm/osoundsystem.h
index ac7a5a7..cce90c0 100644
--- a/libopie2/opiemm/osoundsystem.h
+++ b/libopie2/opiemm/osoundsystem.h
@@ -2,4 +2,3 @@
                              This file is part of the Opie Project
-
-                             (C) 2003 Michael 'Mickey' Lauer <mickey@tm.informatik.uni-frankfurt.de>
+                             (C) 2003-2005 Michael 'Mickey' Lauer <mickey@Vanille.de>
               =.
@@ -200,3 +199,6 @@ class OMixerInterface : public QObject
     QStringList playChannels() const;
-
+    /**
+     * @returns true, if the device features multiple recording sources.
+     */
+    bool hasMultipleRecording() const;
     /**
@@ -209,3 +211,6 @@ class OMixerInterface : public QObject
     bool isStereo( const QString& channel ) const;
-
+    /**
+     * @returns tru, if @a channel is a possible recording source.
+     */
+    bool isRecordable( const QString& channel ) const;
     /**
@@ -223,2 +228,6 @@ class OMixerInterface : public QObject
     int _fd;
+    int _capmask;
+    int _devmask;
+    int _recmask;
+    int _stmask;
     QMap<QString, int> _channels;