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
@@ -12,32 +12,34 @@
 
 /*
  * $Id$
  */
 // Sun 03-17-2002  L.J.Potter <ljp@llornkcor.com>
 #include <sys/utsname.h>
 #include <sys/time.h>
 #include <sys/types.h>
 #include <unistd.h>
 #include <stdio.h>
 #include <sys/stat.h>
 #include <fcntl.h>
 #include <sys/ioctl.h>
 #include <linux/soundcard.h>
 #include <string.h>
 #include <stdlib.h>
+#include <errno.h>
+#include <string.h>
 
 typedef struct _waveheader {
   u_long  main_chunk; /* 'RIFF'  */
   u_long  length;     /* filelen */
   u_long  chunk_type; /* 'WAVE'  */
   u_long  sub_chunk;  /* 'fmt '                                               */
   u_long  sc_len;     /* length of sub_chunk, =16     
                          (chunckSize) format len */
   u_short format;     /* should be 1 for PCM-code        (formatTag)          */
 
   u_short modus;      /* 1 Mono, 2 Stereo                (channels)           */
   u_long  sample_fq;  /* samples per second              (samplesPerSecond)   */
   u_long  byte_p_sec; /* avg bytes per second            (avgBytePerSecond)   */
   u_short byte_p_spl; /* samplesize; 1 or 2 bytes        (blockAlign)         */
   u_short bit_p_spl;  /* 8, 12 or 16 bit                 (bitsPerSample)      */
 
@@ -193,41 +195,41 @@ static char * vmemo_xpm[] = {
 
 VMemo::VMemo( QWidget *parent, const char *_name )
     : QWidget( parent, _name )
 {
   setFixedHeight( 18 );
   setFixedWidth( 14 );
   
   recording = FALSE;
   
   struct utsname name; /* check for embedix kernel running on the zaurus*/
   if (uname(&name) != -1) {
       QString release=name.release;
       if(release.find("embedix",0,TRUE) !=-1)
           systemZaurus=TRUE;
       else {
           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()");
       }
   }
 }
 
 VMemo::~VMemo()
 {
 }
 
 void VMemo::receive( const QCString &msg, const QByteArray &data )
 {
   QDataStream stream( data, IO_ReadOnly );
   if (msg == "toggleRecord()")  {
     if (recording)
     mouseReleaseEvent(NULL);
     else
     mousePressEvent(NULL);
@@ -239,78 +241,90 @@ void VMemo::paintEvent( QPaintEvent* )
   QPainter p(this);
   p.drawPixmap( 0, 1,( const char** )  vmemo_xpm );
 }
 
 void VMemo::mousePressEvent( QMouseEvent *me )
 {
   // just to be safe
     if (recording) {
         recording = FALSE;
         return;
     }
 
       /* 
          No mousePress/mouseRelease recording on the iPAQ. The REC button on the iPAQ calls these functions
          mousePressEvent and mouseReleaseEvent with a NULL parameter.
       */
-     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;
 //     } else {
     QSound::play(Resource::findSound("vmemob"));
 //     }
     qDebug("Start recording");
     recording = TRUE;
     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;
         return;
     }
   
   Config vmCfg("VMemo");
   vmCfg.setGroup("Defaults");
   
   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
   fileName.replace(QRegExp("'"),"");
   fileName.replace(QRegExp(" "),"_");
   fileName.replace(QRegExp(":"),".");
   fileName.replace(QRegExp(","),"");
   
   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");
     close(dsp);
     return;
   }
   
   QArray<int> cats(1);
   cats[0] = vmCfg.readNumEntry("Category", 0);
   
   QString dlName("vm_");
   dlName += dt.toString();
   DocLnk l;
   l.setFile(fileName);
   l.setName(dlName);
   l.setType("audio/x-wav");
   l.setCategories(cats);
   l.writeLink();
@@ -321,74 +335,83 @@ void VMemo::mousePressEvent( QMouseEvent *me )
 void VMemo::mouseReleaseEvent( QMouseEvent * )
 {
   recording = FALSE;
 }
 
 int VMemo::openDSP()
 {
     Config cfg("Sound");
     cfg.setGroup("Record");
   
     speed = cfg.readNumEntry("SampleRate", 22050);
     channels = cfg.readNumEntry("Stereo", 1) ? 2 : 1; // 1 = stereo(2), 0 = mono(1)
     if (cfg.readNumEntry("SixteenBit", 1)==1)  {
         format = AFMT_S16_LE;
         resolution = 16;
     } else {
-        format = AFMT_S8;
+        format = AFMT_U8;
         resolution = 8;
     }
-  
+
+    qDebug("samplerate: %d, channels %d, resolution %d", speed, channels, resolution);  
 
     if(systemZaurus) {
         dsp = open("/dev/dsp1", O_RDWR); //Zaurus needs /dev/dsp1
         channels=1; //zaurus has one input channel
     }  else {
         dsp = open("/dev/dsp", O_RDWR);
     }
   
     if(dsp == -1)  {
         perror("open(\"/dev/dsp\")");
+        
+        errorMsg="open(\"/dev/dsp\")\n "+(QString)strerror(errno);
         return -1;
     }
   
     if(ioctl(dsp, SNDCTL_DSP_SETFMT , &format)==-1)  {
         perror("ioctl(\"SNDCTL_DSP_SETFMT\")");
+        errorMsg="ioctl(\"SNDCTL_DSP_SETFMT\")\n%d\n"+(QString)strerror(errno),format;
         return -1;
     }
     if(ioctl(dsp, SNDCTL_DSP_CHANNELS , &channels)==-1)  {
         perror("ioctl(\"SNDCTL_DSP_CHANNELS\")");
+        errorMsg="ioctl(\"SNDCTL_DSP_CHANNELS\")\n%d\n"+(QString)strerror(errno),channels;
         return -1;
     }
     if(ioctl(dsp, SNDCTL_DSP_SPEED , &speed)==-1)  {
         perror("ioctl(\"SNDCTL_DSP_SPEED\")");
+        errorMsg="ioctl(\"SNDCTL_DSP_SPEED\")\n%d\n"+(QString)strerror(errno),speed;
         return -1;
     }
     if(ioctl(dsp, SOUND_PCM_READ_RATE , &rate)==-1)  {
         perror("ioctl(\"SOUND_PCM_READ_RATE\")");
+        errorMsg="ioctl(\"SOUND_PCM_READ_RATE\")\n%d\n"+(QString)strerror(errno),rate;
         return -1;
     }
   
     return 1;
 }
 
 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;
+  }
 
   wav=track.handle();
   
   WaveHeader wh;
 
   wh.main_chunk = RIFF;
   wh.length=0; 
   wh.chunk_type = WAVE;
   wh.sub_chunk  = FMT;
   wh.sc_len     = 16;
   wh.format     = PCM_CODE;
   wh.modus      = channels;
   wh.sample_fq  = speed;
   wh.byte_p_sec = speed * channels *  resolution/8;
   wh.byte_p_spl = channels * (resolution / 8); 
   wh.bit_p_spl  = resolution;
@@ -417,53 +440,53 @@ void VMemo::record(void)
                     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);
                 }
                 printf("%d\r",length);
                 fflush(stdout);
             }
-        }  else { //AFMT_S8 
+        }  else { //AFMT_U8 
 // 8bit unsigned
             unsigned short sound[512], monoBuffer[512];
             while(recording)   {
                 result = read(dsp, sound, 512); // 8192
                 qApp->processEvents();
                 int j=0;
                 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;
                 printf("%d\r",length);
                 fflush(stdout);
 
                 qApp->processEvents();
             }
 
             qApp->processEvents();
         }
 
     } else {
 
         signed short sound[512], monoBuffer[512];
 
         while(recording)  {
             result = read(dsp, sound, 512); // 8192
@@ -478,21 +501,20 @@ void VMemo::record(void)
         fflush(stdout);
           //  qDebug("file has length of %d lasting %d seconds",
           //         length, (( length / speed) / channels) / 2 );
           // medialplayer states wrong length in secs
     }
 
     value = length+36;
     lseek(wav, 4, SEEK_SET);
     write(wav, &value, 4);
     lseek(wav, 40, SEEK_SET);
     write(wav, &length, 4);
     track.close();
   
     if( ioctl( dsp, SNDCTL_DSP_RESET,0) == -1)
         perror("ioctl(\"SNDCTL_DSP_RESET\")");
     ::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
@@ -17,32 +17,33 @@
 #ifndef __VMEMO_H__
 #define __VMEMO_H__
 
 #include <qwidget.h>
 #include <qpixmap.h>
 #include <qpe/applnk.h>
 #include <qfile.h>
 #include <qpe/qcopenvelope_qws.h>
 
 class VMemo : public QWidget
 {
   Q_OBJECT
 public:
   VMemo( QWidget *parent, const char *name = NULL);
   ~VMemo();
   QFile track;  
+  QString fileName, errorMsg;
 
 public slots:
   void record();
   void mousePressEvent( QMouseEvent * );
   void mouseReleaseEvent( QMouseEvent * );
   void receive( const QCString &msg, const QByteArray &data );
 
 private:
   bool useAlerts;
   void paintEvent( QPaintEvent* );
 
   int openDSP();
   int openWAV(const char *filename);
 
   QPixmap vmemoPixmap;
   QCopChannel *myChannel;