use exec except of system() to start external viewer

src/qtgui/rclmain_w.cpp

@@ -1037,12 +1037,6 @@ void RclMain::startNativeViewer(Rcl::Doc doc)
 	// addition to the path. We store a copy of xdg-open there, to be 
 	// used as last resort
 	cmdpath = rclconfig->findFilter(lcmd.front());
-	// Substitute path for cmd
-	if (!cmdpath.empty()) {
-	    lcmd.front() = cmdpath;
-	    cmd.erase();
-	    stringsToString(lcmd, cmd);
-	}
     } else {
 	ExecCmd::which(lcmd.front(), cmdpath);
     }
@@ -1050,7 +1044,9 @@ void RclMain::startNativeViewer(Rcl::Doc doc)
     // Specialcase text/html because of the help browser need
     if (cmdpath.empty() && !doc.mimetype.compare("text/html")) {
 	if (lookForHtmlBrowser(cmdpath)) {
-	    cmd = cmdpath + " %u";
+	    lcmd.clear();
+            lcmd.push_back(cmdpath);
+            lcmd.push_back("%u");
 	}
     }
 
@@ -1128,7 +1124,10 @@ void RclMain::startNativeViewer(Rcl::Doc doc)
         }
     }
 
-    // Substitute %xx inside prototype command
+    // Get rid of the command name. lcmd is now argv[1...n]
+    lcmd.pop_front();
+
+    // Substitute %xx inside arguments
     string efftime;
     if (!doc.dmtime.empty() || !doc.fmtime.empty()) {
         efftime = doc.dmtime.empty() ? doc.fmtime : doc.dmtime;
@@ -1137,22 +1136,28 @@ void RclMain::startNativeViewer(Rcl::Doc doc)
     }
     // Try to keep the letters used more or less consistent with the reslist
     // paragraph format.
-    string ncmd;
     map<string, string> subs;
-    subs["D"] = escapeShell(efftime);
-    subs["f"] = escapeShell(fn);
-    subs["i"] = escapeShell(doc.ipath);
-    subs["M"] = escapeShell(doc.mimetype);
-    subs["U"] = escapeShell(url);
-    subs["u"] = escapeShell(url);
+    subs["D"] = efftime;
+    subs["f"] = fn;
+    subs["i"] = doc.ipath;
+    subs["M"] = doc.mimetype;
+    subs["U"] = url;
+    subs["u"] = url;
     // Let %(xx) access all metadata.
     for (map<string,string>::const_iterator it = doc.meta.begin();
          it != doc.meta.end(); it++) {
-        subs[it->first] = escapeShell(it->second);
+        subs[it->first] = it->second;
+    }
+    string ncmd;
+    for (list<string>::iterator it = lcmd.begin(); 
+         it != lcmd.end(); it++) {
+        pcSubst(*it, ncmd, subs);
+        LOGDEB(("%s->%s\n", it->c_str(), ncmd.c_str()));
+        *it = ncmd;
     }
 
+    // Also substitute inside the unsplitted command line for displaying
     pcSubst(cmd, ncmd, subs);
-
     ncmd += " &";
     QStatusBar *stb = statusBar();
     if (stb) {
@@ -1166,11 +1171,11 @@ void RclMain::startNativeViewer(Rcl::Doc doc)
 
     if (!istempfile)
 	historyEnterDoc(g_dynconf, doc.meta[Rcl::Doc::keyudi]);
-    
-    // We should actually monitor these processes so that we can
-    // delete the temp files when they exit
-    LOGDEB(("Executing: [%s]\n", ncmd.c_str()));
-    system(ncmd.c_str());
+
+    // We keep pushing back and never deleting. This can't be good...
+    ExecCmd *ecmd = new ExecCmd;
+    m_viewers.push_back(ecmd);
+    ecmd->startExec(cmdpath, lcmd, false, false);
 }
 
 void RclMain::startManual()

src/qtgui/rclmain_w.h

@@ -31,6 +31,8 @@
 #include "refcntr.h"
 #include "pathut.h"
 
+class ExecCmd;
+
 #if QT_VERSION < 0x040000
 #include "rclmain.h"
 #else
@@ -130,6 +132,7 @@ private:
     QTimer         *periodictimer;
 
     vector<TempFile>  m_tempfiles;
+    vector<ExecCmd*>  m_viewers;
     map<QString, int> m_stemLangToId;
     vector<string>    m_catgbutvec;
     int               m_idNoStem;

src/utils/execmd.cpp

@@ -127,7 +127,7 @@ public:
     ~ExecCmdRsrc() {
 	if (!m_active || !m_parent)
 	    return;
-	LOGDEB0(("~ExecCmdRsrc: working\n"));
+	LOGDEB1(("~ExecCmdRsrc: working. mypid: %d\n", (int)getpid()));
 	int status;
 	if (m_parent->m_pid > 0) {
 	    LOGDEB(("ExecCmd: killing cmd\n"));

src/utils/execmd.h

@@ -199,6 +199,9 @@ class ExecCmd {
     // Child process code
     void dochild(const string &cmd, const list<string>& args, 
 		 bool has_input, bool has_output);
+    /* Copyconst and assignment private and forbidden */
+    ExecCmd(const ExecCmd &) {}
+    ExecCmd& operator=(const ExecCmd &) {return *this;};
 };