Changeset 86 for psi/trunk/src/tools/openpgp/gpgproc/gpgproc.cpp

Timestamp:

01/06/07 19:03:56 (20 months ago)

Author:

dmik

Message:

Psi: OpenPGP: replaced printf() debug statements with qDebug() to redirect debug logs to stderr; added some minor OS/2-related corrections.

Files:

• psi/trunk/src/tools/openpgp/gpgproc/gpgproc.cpp (modified) (12 diffs)

psi/trunk/src/tools/openpgp/gpgproc/gpgproc.cpp

@@ -106 +106 @@
         if(!d->pipeAux.open()) {
 #ifdef GPG_DEBUG
-                printf("Error creating pipeAux\n");
+                qDebug("### Error creating pipeAux");
 #endif
                 closePipes();
@@ -113 +113 @@
         if(!d->pipeCommand.open()) {
 #ifdef GPG_DEBUG
-                printf("Error creating pipeCommand\n");
+                qDebug("### Error creating pipeCommand");
 #endif
                 closePipes();
@@ -120 +120 @@
         if(!d->pipeStatus.open()) {
 #ifdef GPG_DEBUG
-                printf("Error creating pipeStatus\n");
+                qDebug("### Error creating pipeStatus");
 #endif
                 closePipes();
@@ -130 +130 @@
         if(!d->pipeAux.writeEnd().winDupHandle()) {
 #ifdef GPG_DEBUG
-                printf("Error dup'ing pipeAux\n");
+                qDebug("### Error dup'ing pipeAux");
 #endif
                 closePipes();
@@ -137 +137 @@
         if(!d->pipeCommand.writeEnd().winDupHandle()) {
 #ifdef GPG_DEBUG
-                printf("Error dup'ing pipeCommand\n");
+                qDebug("### Error dup'ing pipeCommand");
 #endif
                 closePipes();
@@ -144 +144 @@
         if(!d->pipeStatus.readEnd().winDupHandle()) {
 #ifdef GPG_DEBUG
-                printf("Error dup'ing pipeStatus\n");
+                qDebug("### Error dup'ing pipeStatus");
 #endif
                 closePipes();
@@ -168 +168 @@
                 QCString cs2 = buf.data();
                 if(cs != cs2)
-                        printf("Pipe test failed.\n");
+                        qDebug("### Pipe test failed.");
                 else
-                        printf("Pipe test succeeded.\n");
+                        qDebug("### Pipe test succeeded.");
         }*/
 
@@ -180 +180 @@
 
 #ifdef GPG_DEBUG
-        printf("Pipe setup complete.\n");
+        qDebug("### Pipe setup complete.");
 #endif
 
@@ -188 +188 @@
 
         if(useExtra) {
-                fullargs += "--enable-special-filenames";
-
                 fullargs += "--status-fd";
                 fullargs += d->pipeStatus.writeEnd().toString();
@@ -197 +195 @@
         }
 
+        bool seen_special_filenames = false;        
         for(QStringList::ConstIterator it = args.begin(); it != args.end(); ++it) {
                 const QString &a = (*it);
-                if(a == "-&?")
-                        fullargs += (QString("-&") + d->pipeAux.readEnd().toString());
-                else
+                if(a == "-&?") {
+                        seen_special_filenames = true;
+                        fullargs += (QString("\"-&") + d->pipeAux.readEnd().toString() + "\"");
+                } else {
                         fullargs += a;
-        }
+                }
+        }
+        // insert --enable-special-filenames after --no-tty
+        if(seen_special_filenames)
+                fullargs.insert(fullargs.at(2), "--enable-special-filenames");
 
         // show full command
 #ifdef GPG_DEBUG
-        printf("GPGProc: ");
-        for(QStringList::ConstIterator it2 = fullargs.begin(); it2 != fullargs.end(); ++it2) {
-                printf("%s ", (*it2).latin1());
-        }
-        printf("\n");
+        qDebug("### GPGProc: %s", fullargs.join(" ").latin1());
 #endif
 
@@ -223 +223 @@
         d->proc->setClosePipeList(plist);
 #endif
-        /// @todo (dmik)
-        //  On Unix, it is done when SProcess is destructed. On OS/2 as well as on
-        //  Win32, SProcess is not used, but we use the same QPipe implelentation
-        //  on OS/2 as Unix does, so we need to use the similar technuque to close
-        //  them... Need to investigate this file (i.e. usage) in detail.
 
         connect(d->proc, SIGNAL(readyReadStdout()), SLOT(proc_readyReadStdout()));
@@ -327 +322 @@
 #endif
         processExited();
+
+#if defined(Q_OS_OS2)
+        closePipes();
+#endif
 }