more informative error messages

2 files changed, 70 insertions, 47 deletions

diff --git a/core/applets/vmemo/vmemo.cpp b/core/applets/vmemo/vmemo.cpp
index 4d4d598..4c55dc7 100644
--- a/core/applets/vmemo/vmemo.cpp
+++ b/core/applets/vmemo/vmemo.cpp
@@ -27,2 +27,4 @@
 #include <stdlib.h>
+#include <errno.h>
+#include <string.h>
 
@@ -208,11 +210,11 @@ VMemo::VMemo( QWidget *parent, const char *_name )
           systemZaurus=FALSE;
-            myChannel = new QCopChannel( "QPE/VMemo", this );
-            connect( myChannel, SIGNAL(received(const QCString&, const QByteArray&)),
-                     this, SLOT(receive(const QCString&, const QByteArray&)) );
+          myChannel = new QCopChannel( "QPE/VMemo", this );
+          connect( myChannel, SIGNAL(received(const QCString&, const QByteArray&)),
+                   this, SLOT(receive(const QCString&, const QByteArray&)) );
 
 //              // Register the REC key press, for ipaq only
-            QCopEnvelope e("QPE/Desktop", "keyRegister(int key, QString channel, QString message)");
-            e << 4096;
-            e << QString("QPE/VMemo");
-            e << QString("toggleRecord()");
+          QCopEnvelope e("QPE/Desktop", "keyRegister(int key, QString channel, QString message)");
+          e << 4096;
+          e << QString("QPE/VMemo");
+          e << QString("toggleRecord()");
       }
@@ -254,11 +256,11 @@ void VMemo::mousePressEvent( QMouseEvent *me )
       */
-     if (!systemZaurus && me != NULL)
-         return;
+       if (!systemZaurus && me != NULL)
+           return;
 
-//     Config config( "Sound" );
-//     config.setGroup( "System" );
-//     useAlerts = config.readBoolEntry("Alert");
+     Config config( "Sound" );
+     config.setGroup( "System" );
+     useAlerts = config.readBoolEntry("Alert");
 
-//     if(useAlerts) {
-//         if( QMessageBox::warning(this,"VMemo","Really Record?","Yes","No",0,0,1) ==1)
+//      if(useAlerts) 
+//  QMessageBox::message("VMemo","Really Record?");//) ==1)
 //             return;
@@ -270,3 +272,3 @@ void VMemo::mousePressEvent( QMouseEvent *me )
     if (openDSP() == -1)  {
-        QMessageBox::critical(0, "VMemo", "Could not open dsp device.", "Abort");
+        QMessageBox::critical(0, "VMemo", "Could not open dsp device.\n"+errorMsg, "Abort");
         recording = FALSE;
@@ -279,13 +281,26 @@ void VMemo::mousePressEvent( QMouseEvent *me )
   QDateTime dt = QDateTime::currentDateTime();
-  QString fileName;
 
-  if(systemZaurus)
-  fileName=vmCfg.readEntry("Dir", "/mnt/cf/"); // zaurus does not have /mnt/ramfs
-  else
-  fileName=vmCfg.readEntry("Dir", "/mnt/ramfs/");
-  
-  fileName += "vm_";
-  fileName += dt.toString();
-  fileName += ".wav";
+  QString fName;
+  Config cfg( "Sound" );
+  cfg.setGroup( "System" );
+  fileName = cfg.readEntry("RecLocation",QPEApplication::documentDir() );
+
+  int s;
+  s=fileName.find(':');
+  if(s)
+     fileName=fileName.right(fileName.length()-s-2)+"/";
+     
+//   if( !fileName.right(1).find('/') == -1)
+//           fileName+="/audio/";
+//       else
+//           fileName+="audio/";
+
+//   if(systemZaurus) 
+//   fileName=vmCfg.readEntry("Dir", "/mnt/cf/"); // zaurus does not have /mnt/ramfs
+//   else
+//   fileName=vmCfg.readEntry("Dir", "/mnt/ramfs/");
   
+  fName = "vm_"+ dt.toString()+ ".wav";
+  fileName+=fName;
+     qDebug("filename is "+fileName);
   // No spaces in the filename
@@ -297,5 +312,4 @@ void VMemo::mousePressEvent( QMouseEvent *me )
   if(openWAV(fileName.latin1()) == -1)  {
-    QString err("Could not open the output file: ");
+    QString err("Could not open the output file\n");
     err += fileName;
-
     QMessageBox::critical(0, "VMemo", err, "Abort");
@@ -336,6 +350,7 @@ int VMemo::openDSP()
     } else {
-        format = AFMT_S8;
+        format = AFMT_U8;
         resolution = 8;
     }
-  
+
+    qDebug("samplerate: %d, channels %d, resolution %d", speed, channels, resolution);  
 
@@ -350,2 +365,4 @@ int VMemo::openDSP()
         perror("open(\"/dev/dsp\")");
+        
+        errorMsg="open(\"/dev/dsp\")\n "+(QString)strerror(errno);
         return -1;
@@ -355,2 +372,3 @@ int VMemo::openDSP()
         perror("ioctl(\"SNDCTL_DSP_SETFMT\")");
+        errorMsg="ioctl(\"SNDCTL_DSP_SETFMT\")\n%d\n"+(QString)strerror(errno),format;
         return -1;
@@ -359,2 +377,3 @@ int VMemo::openDSP()
         perror("ioctl(\"SNDCTL_DSP_CHANNELS\")");
+        errorMsg="ioctl(\"SNDCTL_DSP_CHANNELS\")\n%d\n"+(QString)strerror(errno),channels;
         return -1;
@@ -363,2 +382,3 @@ int VMemo::openDSP()
         perror("ioctl(\"SNDCTL_DSP_SPEED\")");
+        errorMsg="ioctl(\"SNDCTL_DSP_SPEED\")\n%d\n"+(QString)strerror(errno),speed;
         return -1;
@@ -367,2 +387,3 @@ int VMemo::openDSP()
         perror("ioctl(\"SOUND_PCM_READ_RATE\")");
+        errorMsg="ioctl(\"SOUND_PCM_READ_RATE\")\n%d\n"+(QString)strerror(errno),rate;
         return -1;
@@ -376,4 +397,6 @@ int VMemo::openWAV(const char *filename)
   track.setName(filename);
-  if(!track.open(IO_WriteOnly|IO_Truncate|IO_Raw)) 
+  if(!track.open(IO_WriteOnly|IO_Truncate|IO_Raw)) {
+      errorMsg=filename;
     return -1;
+  }
 
@@ -432,3 +455,3 @@ void VMemo::record(void)
             }
-        }  else { //AFMT_S8 
+        }  else { //AFMT_U8 
 // 8bit unsigned
@@ -440,15 +463,15 @@ void VMemo::record(void)
                 if(systemZaurus) {
-                      for (int i = 0; i < result; i++) { //since Z is mono do normally
-                          monoBuffer[i] = sound[i];
-                      }
-                      qApp->processEvents();
-                      length+=write(wav, monoBuffer, result);
-                  } else { //ipaq /stereo inputs
-                      for (int i = 0; i < result; i+=2) {
-                          monoBuffer[j] = (sound[i]+sound[i+1])/2;
-                          j++;
-                      }
-                      qApp->processEvents();
-                      length+=write(wav, monoBuffer, result/2);
-                  }
+                    for (int i = 0; i < result; i++) { //since Z is mono do normally
+                        monoBuffer[i] = sound[i];
+                    }
+                    qApp->processEvents();
+                    length+=write(wav, monoBuffer, result);
+                } else { //ipaq /stereo inputs
+                    for (int i = 0; i < result; i+=2) {
+                        monoBuffer[j] = (sound[i]+sound[i+1])/2;
+                        j++;
+                    }
+                    qApp->processEvents();
+                    length+=write(wav, monoBuffer, result/2);
+                }
                 length += result;
@@ -493,6 +516,5 @@ void VMemo::record(void)
     ::close(dsp);
-//     if(useAlerts) 
-//        QMessageBox::message("Vmemo"," Done recording");
-//     else
-        QSound::play(Resource::findSound("vmemoe"));
+    if(useAlerts) 
+        QMessageBox::message("Vmemo"," Done recording\n"+ fileName);
+    QSound::play(Resource::findSound("vmemoe"));
 }
diff --git a/core/applets/vmemo/vmemo.h b/core/applets/vmemo/vmemo.h
index f30476f..3972877 100644
--- a/core/applets/vmemo/vmemo.h
+++ b/core/applets/vmemo/vmemo.h
@@ -32,2 +32,3 @@ public:
   QFile track;  
+  QString fileName, errorMsg;