![[tokkee]](http://tokkee.org/images/avatar.png){kind=avatar}
index 8f121800fa90b4e34f1dd3ab0ec4f552767b43b5..eabf147f64980afa2acbb18027dc682c961da6a3 100644 (file)
* Bryce Harrington <bryce@osdl.org>
* Ted Gould <ted@gould.cx>
*
- * Copyright (C) 2002-2005 Authors
+ * Copyright (C) 2002-2005,2007 Authors
*
* Released under GNU GPL, read the file 'COPYING' for more information
*/
#include <unistd.h>
#include <errno.h>
-#include <gtkmm/textview.h>
-#include <gtkmm/scrolledwindow.h>
+#include <gtkmm.h>
#include "ui/view/view.h"
#include "desktop-handles.h"
+#include "desktop.h"
#include "selection.h"
#include "sp-namedview.h"
#include "io/sys.h"
-#include "prefs-utils.h"
+#include "preferences.h"
#include "../system.h"
#include "extension/effect.h"
+#include "extension/output.h"
+#include "extension/input.h"
#include "extension/db.h"
#include "script.h"
+#include "dialogs/dialog-events.h"
+#include "application/application.h"
+#include "xml/node.h"
+#include "xml/attribute-record.h"
#include "util/glib-list-iterators.h"
+
+
#ifdef WIN32
#include <windows.h>
+#include <sys/stat.h>
+#include "registrytool.h"
#endif
+
+
/** This is the command buffer that gets allocated from the stack */
#define BUFSIZE (255)
+
+
/* Namespaces */
namespace Inkscape {
namespace Extension {
namespace Implementation {
-/* Real functions */
-/**
- \return A script object
- \brief This function creates a script object and sets up the
+/** \brief Make GTK+ events continue to come through a little bit
+
+ This just keeps coming the events through so that we'll make the GUI
+ update and look pretty.
+*/
+void
+Script::pump_events (void) {
+ while( Gtk::Main::events_pending() )
+ Gtk::Main::iteration();
+ return;
+}
+
+
+/** \brief A table of what interpreters to call for a given language
+
+ This table is used to keep track of all the programs to execute a
+ given script. It also tracks the preference to use to overwrite
+ the given interpreter to a custom one per user.
+*/
+Script::interpreter_t const Script::interpreterTab[] = {
+ {"perl", "perl-interpreter", "perl" },
+#ifdef WIN32
+ {"python", "python-interpreter", "pythonw" },
+#else
+ {"python", "python-interpreter", "python" },
+#endif
+ {"ruby", "ruby-interpreter", "ruby" },
+ {"shell", "shell-interpreter", "sh" },
+ { NULL, NULL, NULL }
+};
+
+
+
+/** \brief Look up an interpreter name, and translate to something that
+ is executable
+ \param interpNameArg The name of the interpreter that we're looking
+ for, should be an entry in interpreterTab
+*/
+Glib::ustring
+Script::resolveInterpreterExecutable(const Glib::ustring &interpNameArg)
+{
+
+ Glib::ustring interpName = interpNameArg;
+
+ interpreter_t const *interp;
+ bool foundInterp = false;
+ for (interp = interpreterTab ; interp->identity ; interp++ ){
+ if (interpName == interp->identity) {
+ foundInterp = true;
+ break;
+ }
+ }
+
+ // Do we have a supported interpreter type?
+ if (!foundInterp)
+ return "";
+ interpName = interp->defaultval;
+
+ // 1. Check preferences
+ Inkscape::Preferences *prefs = Inkscape::Preferences::get();
+ Glib::ustring prefInterp = prefs->getString("/extensions/" + Glib::ustring(interp->prefstring));
+
+ if (!prefInterp.empty()) {
+ interpName = prefInterp;
+ return interpName;
+ }
+
+#ifdef WIN32
+
+ // 2. Windows. Try looking relative to inkscape.exe
+ RegistryTool rt;
+ Glib::ustring fullPath;
+ Glib::ustring path;
+ Glib::ustring exeName;
+ if (rt.getExeInfo(fullPath, path, exeName)) {
+ Glib::ustring interpPath = path;
+ interpPath.append("\\");
+ interpPath.append(interpNameArg);
+ interpPath.append("\\");
+ interpPath.append(interpName);
+ interpPath.append(".exe");
+ struct stat finfo;
+ if (stat(interpPath .c_str(), &finfo) ==0) {
+ g_message("Found local interpreter, '%s', Size: %d",
+ interpPath .c_str(),
+ (int)finfo.st_size);
+ return interpPath;
+ }
+ }
+
+ // 3. Try searching the path
+ char szExePath[MAX_PATH];
+ char szCurrentDir[MAX_PATH];
+ GetCurrentDirectory(sizeof(szCurrentDir), szCurrentDir);
+ unsigned int ret = (unsigned int)FindExecutable(
+ interpName.c_str(), szCurrentDir, szExePath);
+ if (ret > 32) {
+ interpName = szExePath;
+ return interpName;
+ }
+
+#endif // win32
+
+
+ return interpName;
+}
+
+/** \brief This function creates a script object and sets up the
variables.
+ \return A script object
This function just sets the command to NULL. It should get built
officially in the load function. This allows for less allocation
of memory in the unloaded state.
*/
Script::Script() :
- Implementation(),
- command(NULL),
- helper_extension(NULL)
+ Implementation()
{
}
+/**
+ * brief Destructor
+ */
+Script::~Script()
+{
+}
+
+
+
/**
\return A string with the complete string with the relative directory expanded
\brief This function takes in a Repr that contains a reldir entry
string. This means that the caller of this function can always
free what they are given (and should do it too!).
*/
-gchar *
+Glib::ustring
Script::solve_reldir(Inkscape::XML::Node *reprin) {
- gchar const *reldir = reprin->attribute("reldir");
- if (reldir == NULL) {
- return g_strdup(sp_repr_children(reprin)->content());
+ gchar const *s = reprin->attribute("reldir");
+
+ if (!s) {
+ Glib::ustring str = sp_repr_children(reprin)->content();
+ return str;
}
- if (!strcmp(reldir, "extensions")) {
- for(unsigned int i=0; i<Inkscape::Extension::Extension::search_path.size(); i++) {
- gchar * filename = g_build_filename(Inkscape::Extension::Extension::search_path[i], sp_repr_children(reprin)->content(), NULL);
- if ( Inkscape::IO::file_test(filename, G_FILE_TEST_EXISTS) ) {
+ Glib::ustring reldir = s;
+
+ if (reldir == "extensions") {
+
+ for (unsigned int i=0;
+ i < Inkscape::Extension::Extension::search_path.size();
+ i++) {
+
+ gchar * fname = g_build_filename(
+ Inkscape::Extension::Extension::search_path[i],
+ sp_repr_children(reprin)->content(),
+ NULL);
+ Glib::ustring filename = fname;
+ g_free(fname);
+
+ if ( Inkscape::IO::file_test(filename.c_str(), G_FILE_TEST_EXISTS) )
return filename;
- }
- g_free(filename);
+
}
} else {
- return g_strdup(sp_repr_children(reprin)->content());
+ Glib::ustring str = sp_repr_children(reprin)->content();
+ return str;
}
- return NULL;
+ return "";
}
+
+
/**
\return Whether the command given exists, including in the path
\brief This function is used to find out if something exists for
then a FALSE is returned, the command could not be found.
*/
bool
-Script::check_existance(gchar const *command)
+Script::check_existance(const Glib::ustring &command)
{
- if (*command == '\0') {
- /* We check the simple case first. */
- return FALSE;
+
+ // Check the simple case first
+ if (command.size() == 0) {
+ return false;
}
- if (g_utf8_strchr(command, -1, G_DIR_SEPARATOR) != NULL) {
- /* Don't search when it contains a slash. */
- if (Inkscape::IO::file_test(command, G_FILE_TEST_EXISTS))
- return TRUE;
+ //Don't search when it contains a slash. */
+ if (command.find(G_DIR_SEPARATOR) != command.npos) {
+ if (Inkscape::IO::file_test(command.c_str(), G_FILE_TEST_EXISTS))
+ return true;
else
- return FALSE;
+ return false;
}
- gchar *path = g_strdup(g_getenv("PATH"));
- if (path == NULL) {
- /* There is no `PATH' in the environment.
+ Glib::ustring path;
+ gchar *s = (gchar *) g_getenv("PATH");
+ if (s)
+ path = s;
+ else
+ /* There is no `PATH' in the environment.
The default search path is the current directory */
- path = g_strdup(G_SEARCHPATH_SEPARATOR_S);
- }
- gchar *orig_path = path;
+ path = G_SEARCHPATH_SEPARATOR_S;
- for (; path != NULL;) {
- gchar *const local_path = path;
- path = g_utf8_strchr(path, -1, G_SEARCHPATH_SEPARATOR);
- if (path == NULL) {
- break;
- }
- /* Not sure whether this is UTF8 happy, but it would seem
- like it considering that I'm searching (and finding)
- the ':' character */
- if (path != local_path && path != NULL) {
- path[0] = '\0';
- path++;
- } else {
- path = NULL;
- }
+ std::string::size_type pos = 0;
+ std::string::size_type pos2 = 0;
+ while ( pos < path.size() ) {
+
+ Glib::ustring localPath;
- gchar *final_name;
- if (local_path == '\0') {
- final_name = g_strdup(command);
+ pos2 = path.find(G_SEARCHPATH_SEPARATOR, pos);
+ if (pos2 == path.npos) {
+ localPath = path.substr(pos);
+ pos = path.size();
} else {
- final_name = g_build_filename(local_path, command, NULL);
+ localPath = path.substr(pos, pos2-pos);
+ pos = pos2+1;
}
- if (Inkscape::IO::file_test(final_name, G_FILE_TEST_EXISTS)) {
- g_free(final_name);
- g_free(orig_path);
- return TRUE;
+ //printf("### %s\n", localPath.c_str());
+ Glib::ustring candidatePath =
+ Glib::build_filename(localPath, command);
+
+ if (Inkscape::IO::file_test(candidatePath .c_str(),
+ G_FILE_TEST_EXISTS)) {
+ return true;
}
- g_free(final_name);
}
- return FALSE;
+ return false;
}
+
+
+
+
/**
\return none
\brief This function 'loads' an extention, basically it determines
bool
Script::load(Inkscape::Extension::Extension *module)
{
- if (module->loaded()) {
- return TRUE;
- }
+ if (module->loaded())
+ return true;
- helper_extension = NULL;
+ helper_extension = "";
/* This should probably check to find the executable... */
Inkscape::XML::Node *child_repr = sp_repr_children(module->get_repr());
- gchar *command_text = NULL;
while (child_repr != NULL) {
- if (!strcmp(child_repr->name(), "script")) {
+ if (!strcmp(child_repr->name(), INKSCAPE_EXTENSION_NS "script")) {
child_repr = sp_repr_children(child_repr);
while (child_repr != NULL) {
- if (!strcmp(child_repr->name(), "command")) {
- command_text = solve_reldir(child_repr);
-
- const gchar * interpretstr = child_repr->attribute("interpreter");
+ if (!strcmp(child_repr->name(), INKSCAPE_EXTENSION_NS "command")) {
+ const gchar *interpretstr = child_repr->attribute("interpreter");
if (interpretstr != NULL) {
- struct interpreter_t {
- gchar * identity;
- gchar * prefstring;
- gchar * defaultval;
- };
- const interpreter_t interpreterlst[] = {
- {"perl", "perl-interpreter", "perl"},
- {"python", "python-interpreter", "python"},
- {"ruby", "ruby-interpreter", "ruby"},
- {"shell", "shell-interpreter", "sh"}
- }; /* Change count below if you change structure */
- for (unsigned int i = 0; i < 4; i++) {
- if (!strcmp(interpretstr, interpreterlst[i].identity)) {
- const gchar * insertText = interpreterlst[i].defaultval;
- if (prefs_get_string_attribute("extensions", interpreterlst[i].prefstring) != NULL)
- insertText = prefs_get_string_attribute("extensions", interpreterlst[i].prefstring);
-#ifdef _WIN32
- else {
- char szExePath[MAX_PATH];
- char szCurrentDir[MAX_PATH];
- GetCurrentDirectory(sizeof(szCurrentDir), szCurrentDir);
- if (reinterpret_cast<unsigned>(FindExecutable(command_text, szCurrentDir, szExePath)) > 32)
- insertText = szExePath;
- }
-#endif
-
- gchar * temp = command_text;
- command_text = g_strconcat(insertText, " ", temp, NULL);
- g_free(temp);
-
- break;
- }
- }
+ Glib::ustring interpString =
+ resolveInterpreterExecutable(interpretstr);
+ //g_message("Found: %s and %s",interpString.c_str(),interpretstr);
+ command.insert(command.end(), interpretstr);
}
+ Glib::ustring tmp = "\"";
+ tmp += solve_reldir(child_repr);
+ tmp += "\"";
+
+ command.insert(command.end(), tmp);
}
- if (!strcmp(child_repr->name(), "helper_extension")) {
- helper_extension = g_strdup(sp_repr_children(child_repr)->content());
+ if (!strcmp(child_repr->name(), INKSCAPE_EXTENSION_NS "helper_extension")) {
+ helper_extension = sp_repr_children(child_repr)->content();
}
child_repr = sp_repr_next(child_repr);
}
child_repr = sp_repr_next(child_repr);
}
- g_return_val_if_fail(command_text != NULL, FALSE);
+ //g_return_val_if_fail(command.length() > 0, false);
- if (command != NULL)
- g_free(command);
- command = command_text;
-
- return TRUE;
+ return true;
}
+
/**
\return None.
\brief Unload this puppy!
command if it has been allocated.
*/
void
-Script::unload(Inkscape::Extension::Extension *module)
+Script::unload(Inkscape::Extension::Extension */*module*/)
{
- if (command != NULL) {
- g_free(command);
- command = NULL;
- }
- if (helper_extension != NULL) {
- g_free(helper_extension);
- helper_extension = NULL;
- }
-
- return;
+ command.clear();
+ helper_extension = "";
}
+
+
+
/**
\return Whether the check passed or not
\brief Check every dependency that was given to make sure we should keep this extension
bool
Script::check(Inkscape::Extension::Extension *module)
{
+ int script_count = 0;
Inkscape::XML::Node *child_repr = sp_repr_children(module->get_repr());
while (child_repr != NULL) {
- if (!strcmp(child_repr->name(), "script")) {
+ if (!strcmp(child_repr->name(), INKSCAPE_EXTENSION_NS "script")) {
+ script_count++;
child_repr = sp_repr_children(child_repr);
while (child_repr != NULL) {
- if (!strcmp(child_repr->name(), "check")) {
- gchar *command_text = solve_reldir(child_repr);
- if (command_text != NULL) {
+ if (!strcmp(child_repr->name(), INKSCAPE_EXTENSION_NS "check")) {
+ Glib::ustring command_text = solve_reldir(child_repr);
+ if (command_text.size() > 0) {
/* I've got the command */
- bool existance;
-
- existance = check_existance(command_text);
- g_free(command_text);
+ bool existance = check_existance(command_text);
if (!existance)
- return FALSE;
+ return false;
}
}
- if (!strcmp(child_repr->name(), "helper_extension")) {
+ if (!strcmp(child_repr->name(), INKSCAPE_EXTENSION_NS "helper_extension")) {
gchar const *helper = sp_repr_children(child_repr)->content();
if (Inkscape::Extension::db.get(helper) == NULL) {
- return FALSE;
+ return false;
}
}
child_repr = sp_repr_next(child_repr);
}
- return TRUE;
+ if (script_count == 0) {
+ return false;
+ }
+
+ return true;
}
-/**
- \return A dialog for preferences
- \brief A stub funtion right now
- \param module Module who's preferences need getting
- \param filename Hey, the file you're getting might be important
+class ScriptDocCache : public ImplementationDocumentCache {
+ friend class Script;
+protected:
+ std::string _filename;
+ int _tempfd;
+public:
+ ScriptDocCache (Inkscape::UI::View::View * view);
+ ~ScriptDocCache ( );
+};
- This function should really do something, right now it doesn't.
-*/
-Gtk::Widget *
-Script::prefs_input(Inkscape::Extension::Input *module, gchar const *filename)
+ScriptDocCache::ScriptDocCache (Inkscape::UI::View::View * view) :
+ ImplementationDocumentCache(view),
+ _filename(""),
+ _tempfd(0)
{
- /*return module->autogui(); */
- return NULL;
+ try {
+ _tempfd = Inkscape::IO::file_open_tmp(_filename, "ink_ext_XXXXXX.svg");
+ } catch (...) {
+ /// \todo Popup dialog here
+ return;
+ }
+
+ SPDesktop *desktop = (SPDesktop *) view;
+ sp_namedview_document_from_window(desktop);
+
+ Inkscape::Extension::save(
+ Inkscape::Extension::db.get(SP_MODULE_KEY_OUTPUT_SVG_INKSCAPE),
+ view->doc(), _filename.c_str(), false, false, false);
+
+ return;
}
+ScriptDocCache::~ScriptDocCache ( )
+{
+ close(_tempfd);
+ unlink(_filename.c_str());
+}
+
+ImplementationDocumentCache *
+Script::newDocCache( Inkscape::Extension::Extension * /*ext*/, Inkscape::UI::View::View * view ) {
+ return new ScriptDocCache(view);
+}
+
+
/**
\return A dialog for preferences
\brief A stub funtion right now
- \param module Module whose preferences need getting
+ \param module Module who's preferences need getting
+ \param filename Hey, the file you're getting might be important
This function should really do something, right now it doesn't.
*/
Gtk::Widget *
-Script::prefs_output(Inkscape::Extension::Output *module)
+Script::prefs_input(Inkscape::Extension::Input *module,
+ const gchar */*filename*/)
{
- /*return module->autogui();*/
- return NULL;
+ return module->autogui(NULL, NULL);
}
+
+
/**
\return A dialog for preferences
\brief A stub funtion right now
- \param module Module who's preferences need getting
+ \param module Module whose preferences need getting
This function should really do something, right now it doesn't.
*/
Gtk::Widget *
-Script::prefs_effect(Inkscape::Extension::Effect *module, Inkscape::UI::View::View *view)
+Script::prefs_output(Inkscape::Extension::Output *module)
{
- return module->autogui();
+ return module->autogui(NULL, NULL);
}
/**
@@ -405,58 +546,55 @@ Script::prefs_effect(Inkscape::Extension::Effect *module, Inkscape::UI::View::Vi
That document is then returned from this function.
*/
SPDocument *
-Script::open(Inkscape::Extension::Input *module, gchar const *filename)
+Script::open(Inkscape::Extension::Input *module,
+ const gchar *filenameArg)
{
- int data_read = 0;
- gint tempfd;
- gchar *tempfilename_out;
-
- // FIXME: process the GError instead of passing NULL
- if ((tempfd = g_file_open_tmp("ink_ext_XXXXXX", &tempfilename_out, NULL)) == -1) {
- /* Error, couldn't create temporary filename */
- if (errno == EINVAL) {
- /* The last six characters of template were not XXXXXX. Now template is unchanged. */
- perror("Extension::Script: template for filenames is misconfigured.\n");
- exit(-1);
- } else if (errno == EEXIST) {
- /* Now the contents of template are undefined. */
- perror("Extension::Script: Could not create a unique temporary filename\n");
- return NULL;
- } else {
- perror("Extension::Script: Unknown error creating temporary filename\n");
- exit(-1);
- }
+ std::list<std::string> params;
+ module->paramListString(params);
+
+ std::string tempfilename_out;
+ int tempfd_out = 0;
+ try {
+ tempfd_out = Inkscape::IO::file_open_tmp(tempfilename_out, "ink_ext_XXXXXX.svg");
+ } catch (...) {
+ /// \todo Popup dialog here
+ return NULL;
}
- gsize bytesRead = 0;
- gsize bytesWritten = 0;
- GError *error = NULL;
- gchar *local_filename = g_filename_from_utf8( filename,
- -1, &bytesRead, &bytesWritten, &error);
+ std::string lfilename = Glib::filename_from_utf8(filenameArg);
- data_read = execute(command, local_filename, tempfilename_out);
- g_free(local_filename);
+ file_listener fileout;
+ int data_read = execute(command, params, lfilename, fileout);
+ fileout.toFile(tempfilename_out);
- SPDocument *mydoc = NULL;
+ SPDocument * mydoc = NULL;
if (data_read > 10) {
- if (helper_extension == NULL) {
- mydoc = Inkscape::Extension::open(Inkscape::Extension::db.get(SP_MODULE_KEY_INPUT_SVG), tempfilename_out);
+ if (helper_extension.size()==0) {
+ mydoc = Inkscape::Extension::open(
+ Inkscape::Extension::db.get(SP_MODULE_KEY_INPUT_SVG),
+ tempfilename_out.c_str());
} else {
- mydoc = Inkscape::Extension::open(Inkscape::Extension::db.get(helper_extension), tempfilename_out);
+ mydoc = Inkscape::Extension::open(
+ Inkscape::Extension::db.get(helper_extension.c_str()),
+ tempfilename_out.c_str());
}
- }
+ } // data_read
- if (mydoc != NULL)
- sp_document_set_uri(mydoc, (const gchar *)filename);
+ if (mydoc != NULL) {
+ g_free(mydoc->base);
+ mydoc->base = NULL;
+ sp_document_change_uri_and_hrefs(mydoc, filenameArg);
+ }
// make sure we don't leak file descriptors from g_file_open_tmp
- close(tempfd);
- // FIXME: convert to utf8 (from "filename encoding") and unlink_utf8name
- unlink(tempfilename_out);
- g_free(tempfilename_out);
+ close(tempfd_out);
+
+ unlink(tempfilename_out.c_str());
return mydoc;
-}
+} // open
+
+
/**
\return none
delete the temporary file.
*/
void
-Script::save(Inkscape::Extension::Output *module, SPDocument *doc, gchar const *filename)
+Script::save(Inkscape::Extension::Output *module,
+ SPDocument *doc,
+ const gchar *filenameArg)
{
- gint tempfd;
- gchar *tempfilename_in;
- // FIXME: process the GError instead of passing NULL
- if ((tempfd = g_file_open_tmp("ink_ext_XXXXXX", &tempfilename_in, NULL)) == -1) {
- /* Error, couldn't create temporary filename */
- if (errno == EINVAL) {
- /* The last six characters of template were not XXXXXX. Now template is unchanged. */
- perror("Extension::Script: template for filenames is misconfigured.\n");
- exit(-1);
- } else if (errno == EEXIST) {
- /* Now the contents of template are undefined. */
- perror("Extension::Script: Could not create a unique temporary filename\n");
- return;
- } else {
- perror("Extension::Script: Unknown error creating temporary filename\n");
- exit(-1);
- }
+ std::list<std::string> params;
+ module->paramListString(params);
+
+ std::string tempfilename_in;
+ int tempfd_in = 0;
+ try {
+ tempfd_in = Inkscape::IO::file_open_tmp(tempfilename_in, "ink_ext_XXXXXX.svg");
+ } catch (...) {
+ /// \todo Popup dialog here
+ return;
}
- if (helper_extension == NULL) {
- Inkscape::Extension::save(Inkscape::Extension::db.get(SP_MODULE_KEY_OUTPUT_SVG_INKSCAPE), doc, tempfilename_in, FALSE, FALSE, FALSE);
+ if (helper_extension.size() == 0) {
+ Inkscape::Extension::save(
+ Inkscape::Extension::db.get(SP_MODULE_KEY_OUTPUT_SVG_INKSCAPE),
+ doc, tempfilename_in.c_str(), false, false, false);
} else {
- Inkscape::Extension::save(Inkscape::Extension::db.get(helper_extension), doc, tempfilename_in, FALSE, FALSE, FALSE);
+ Inkscape::Extension::save(
+ Inkscape::Extension::db.get(helper_extension.c_str()),
+ doc, tempfilename_in.c_str(), false, false, false);
}
- gsize bytesRead = 0;
- gsize bytesWritten = 0;
- GError *error = NULL;
- gchar *local_filename = g_filename_from_utf8( filename,
- -1, &bytesRead, &bytesWritten, &error);
- execute(command, tempfilename_in, local_filename);
+ file_listener fileout;
+ execute(command, params, tempfilename_in, fileout);
- g_free(local_filename);
+ std::string lfilename = Glib::filename_from_utf8(filenameArg);
+ fileout.toFile(lfilename);
// make sure we don't leak file descriptors from g_file_open_tmp
- close(tempfd);
+ close(tempfd_in);
// FIXME: convert to utf8 (from "filename encoding") and unlink_utf8name
- unlink(tempfilename_in);
- g_free(tempfilename_in);
+ unlink(tempfilename_in.c_str());
+
+ return;
}
+
+
/**
\return none
\brief This function uses an extention as a effect on a document.
@@ -536,7 +673,7 @@ Script::save(Inkscape::Extension::Output *module, SPDocument *doc, gchar const *
This function is a little bit trickier than the previous two. It
needs two temporary files to get it's work done. Both of these
files have random names created for them using the g_file_open_temp function
- with the sp_ext_ prefix in the temporary directory. Like the other
+ with the ink_ext_ prefix in the temporary directory. Like the other
functions, the temporary files are deleted at the end.
To save/load the two temporary documents (both are SVG) the internal
@@ -556,97 +693,97 @@ Script::save(Inkscape::Extension::Output *module, SPDocument *doc, gchar const *
point both should be full, and the second one is loaded.
*/
void
-Script::effect(Inkscape::Extension::Effect *module, Inkscape::UI::View::View *doc)
+Script::effect(Inkscape::Extension::Effect *module,
+ Inkscape::UI::View::View *doc,
+ ImplementationDocumentCache * docCache)
{
- int data_read = 0;
- SPDocument * mydoc = NULL;
- gint tempfd_in;
- gchar *tempfilename_in;
-
- // FIXME: process the GError instead of passing NULL
- if ((tempfd_in = g_file_open_tmp("ink_ext_XXXXXX", &tempfilename_in, NULL)) == -1) {
- /* Error, couldn't create temporary filename */
- if (errno == EINVAL) {
- /* The last six characters of template were not XXXXXX. Now template is unchanged. */
- perror("Extension::Script: template for filenames is misconfigured.\n");
- exit(-1);
- } else if (errno == EEXIST) {
- /* Now the contents of template are undefined. */
- perror("Extension::Script: Could not create a unique temporary filename\n");
- return;
- } else {
- perror("Extension::Script: Unknown error creating temporary filename\n");
- exit(-1);
- }
+ if (docCache == NULL) {
+ docCache = newDocCache(module, doc);
}
-
- gint tempfd_out;
- gchar *tempfilename_out;
- // FIXME: process the GError instead of passing NULL
- if ((tempfd_out = g_file_open_tmp("ink_ext_XXXXXX", &tempfilename_out, NULL)) == -1) {
- /* Error, couldn't create temporary filename */
- if (errno == EINVAL) {
- /* The last six characters of template were not XXXXXX. Now template is unchanged. */
- perror("Extension::Script: template for filenames is misconfigured.\n");
- exit(-1);
- } else if (errno == EEXIST) {
- /* Now the contents of template are undefined. */
- perror("Extension::Script: Could not create a unique temporary filename\n");
- return;
- } else {
- perror("Extension::Script: Unknown error creating temporary filename\n");
- exit(-1);
- }
+ ScriptDocCache * dc = dynamic_cast<ScriptDocCache *>(docCache);
+ if (dc == NULL) {
+ printf("TOO BAD TO LIVE!!!");
+ exit(1);
}
- Inkscape::Extension::save(Inkscape::Extension::db.get(SP_MODULE_KEY_OUTPUT_SVG_INKSCAPE),
- doc->doc(), tempfilename_in, FALSE, FALSE, FALSE);
+ SPDesktop *desktop = (SPDesktop *)doc;
+ sp_namedview_document_from_window(desktop);
- Glib::ustring local_command(command);
+ gchar * orig_output_extension = g_strdup(sp_document_repr_root(desktop->doc())->attribute("inkscape:output_extension"));
+
+ std::list<std::string> params;
+ module->paramListString(params);
+
+ if (module->no_doc) {
+ // this is a no-doc extension, e.g. a Help menu command;
+ // just run the command without any files, ignoring errors
+
+ Glib::ustring empty;
+ file_listener outfile;
+ execute(command, params, empty, outfile);
+
+ return;
+ }
+
+ std::string tempfilename_out;
+ int tempfd_out = 0;
+ try {
+ tempfd_out = Inkscape::IO::file_open_tmp(tempfilename_out, "ink_ext_XXXXXX.svg");
+ } catch (...) {
+ /// \todo Popup dialog here
+ return;
+ }
- /* fixme: Should be some sort of checking here. Don't know how to do this with structs instead
- * of classes. */
- SPDesktop *desktop = (SPDesktop *) doc;
if (desktop != NULL) {
- using Inkscape::Util::GSListConstIterator;
- GSListConstIterator<SPItem *> selected = sp_desktop_selection(desktop)->itemList();
+ Inkscape::Util::GSListConstIterator<SPItem *> selected =
+ sp_desktop_selection(desktop)->itemList();
while ( selected != NULL ) {
- local_command += " --id=";
- local_command += SP_OBJECT_ID(*selected);
+ Glib::ustring selected_id;
+ selected_id += "--id=";
+ selected_id += SP_OBJECT_ID(*selected);
+ params.insert(params.begin(), selected_id);
++selected;
}
}
- Glib::ustring * paramString = module->paramString();
- local_command += *paramString;
- delete paramString;
+ file_listener fileout;
+ int data_read = execute(command, params, dc->_filename, fileout);
+ fileout.toFile(tempfilename_out);
- // std::cout << local_command << std::endl;
+ pump_events();
- data_read = execute(local_command.c_str(), tempfilename_in, tempfilename_out);
+ SPDocument * mydoc = NULL;
+ if (data_read > 10) {
+ mydoc = Inkscape::Extension::open(
+ Inkscape::Extension::db.get(SP_MODULE_KEY_INPUT_SVG),
+ tempfilename_out.c_str());
+ } // data_read
- if (data_read > 10)
- mydoc = Inkscape::Extension::open(Inkscape::Extension::db.get(SP_MODULE_KEY_INPUT_SVG), tempfilename_out);
+ pump_events();
// make sure we don't leak file descriptors from g_file_open_tmp
- close(tempfd_in);
close(tempfd_out);
+
// FIXME: convert to utf8 (from "filename encoding") and unlink_utf8name
- unlink(tempfilename_in);
- g_free(tempfilename_in);
- unlink(tempfilename_out);
- g_free(tempfilename_out);
+ unlink(tempfilename_out.c_str());
/* Do something with mydoc.... */
- if (mydoc != NULL) {
+ if (mydoc) {
doc->doc()->emitReconstructionStart();
copy_doc(doc->doc()->rroot, mydoc->rroot);
doc->doc()->emitReconstructionFinish();
mydoc->release();
+ sp_namedview_update_layers_from_document(desktop);
+
+ sp_document_repr_root(desktop->doc())->setAttribute("inkscape:output_extension", orig_output_extension);
}
+ g_free(orig_output_extension);
+
+ return;
}
+
/**
\brief A function to take all the svg elements from one document
and put them in another.
Script::copy_doc (Inkscape::XML::Node * oldroot, Inkscape::XML::Node * newroot)
{
std::vector<Inkscape::XML::Node *> delete_list;
+ Inkscape::XML::Node * oldroot_namedview = NULL;
+
for (Inkscape::XML::Node * child = oldroot->firstChild();
child != NULL;
child = child->next()) {
- if (!strcmp("sodipodi:namedview", child->name()))
- continue;
- if (!strcmp("svg:defs", child->name()))
- continue;
- delete_list.push_back(child);
+ if (!strcmp("sodipodi:namedview", child->name())) {
+ oldroot_namedview = child;
+ for (Inkscape::XML::Node * oldroot_namedview_child = child->firstChild();
+ oldroot_namedview_child != NULL;
+ oldroot_namedview_child = oldroot_namedview_child->next()) {
+ delete_list.push_back(oldroot_namedview_child);
+ }
+ } else {
+ delete_list.push_back(child);
+ }
}
for (unsigned int i = 0; i < delete_list.size(); i++)
sp_repr_unparent(delete_list[i]);
@@ -681,177 +825,42 @@ Script::copy_doc (Inkscape::XML::Node * oldroot, Inkscape::XML::Node * newroot)
for (Inkscape::XML::Node * child = newroot->firstChild();
child != NULL;
child = child->next()) {
- if (!strcmp("sodipodi:namedview", child->name()))
- continue;
- if (!strcmp("svg:defs", child->name()))
- continue;
- oldroot->appendChild(child->duplicate());
- }
-
- /** \todo Restore correct layer */
- /** \todo Restore correct selection */
-}
-
-/* Helper class used by Script::execute */
-class pipe_t {
-public:
- /* These functions set errno if they return false.
- I'm not sure whether that's a good idea or not, but it should be reasonably
- straightforward to change it if needed. */
- bool open(char *command, char const *errorFile, int mode);
- bool close();
-
- /* These return the number of bytes read/written. */
- size_t read(void *buffer, size_t size);
- size_t write(void const *buffer, size_t size);
-
- enum {
- mode_read = 1 << 0,
- mode_write = 1 << 1,
- };
-
-private:
-#ifdef WIN32
- /* This is used to translate win32 errors into errno errors.
- It only recognizes a few win32 errors for the moment though. */
- static int translate_error(DWORD err);
-
- HANDLE hpipe;
-#else
- FILE *ppipe;
-#endif
-};
-
-/**
- \return none
- \brief This is the core of the extension file as it actually does
- the execution of the extension.
- \param in_command The command to be executed
- \param filein Filename coming in
- \param fileout Filename of the out file
- \return Number of bytes that were read into the output file.
-
- The first thing that this function does is build the command to be
- executed. This consists of the first string (in_command) and then
- the filename for input (filein). This file is put on the command
- line.
-
- The next thing is that this function does is open a pipe to the
- command and get the file handle in the ppipe variable. It then
- opens the output file with the output file handle. Both of these
- operations are checked extensively for errors.
-
- After both are opened, then the data is copied from the output
- of the pipe into the file out using fread and fwrite. These two
- functions are used because of their primitive nature they make
- no assumptions about the data. A buffer is used in the transfer,
- but the output of fread is stored so the exact number of bytes
- is handled gracefully.
-
- At the very end (after the data has been copied) both of the files
- are closed, and we return to what we were doing.
-*/
-int
-Script::execute (const gchar * in_command, const gchar * filein, const gchar * fileout)
-{
- g_return_val_if_fail(in_command != NULL, 0);
- // printf("Executing: %s\n", in_command);
-
- gchar * errorFile;
- gint errorFileNum;
- errorFileNum = g_file_open_tmp("ink_ext_stderr_XXXXXX", &errorFile, NULL);
- if (errorFileNum != 0) {
- close(errorFileNum);
- } else {
- g_free(errorFile);
- errorFile = NULL;
- }
-
- char *command = g_strdup_printf("%s \"%s\"", in_command, filein);
- // std::cout << "Command to run: " << command << std::endl;
-
- pipe_t pipe;
- bool open_success = pipe.open(command, errorFile, pipe_t::mode_read);
- g_free(command);
-
- /* Run script */
- if (!open_success) {
- /* Error - could not open pipe - check errno */
- if (errno == EINVAL) {
- perror("Extension::Script: Invalid mode argument in popen\n");
- } else if (errno == ECHILD) {
- perror("Extension::Script: Cannot obtain child extension status in popen\n");
+ if (!strcmp("sodipodi:namedview", child->name())) {
+ if (oldroot_namedview != NULL) {
+ for (Inkscape::XML::Node * newroot_namedview_child = child->firstChild();
+ newroot_namedview_child != NULL;
+ newroot_namedview_child = newroot_namedview_child->next()) {
+ oldroot_namedview->appendChild(newroot_namedview_child->duplicate(oldroot->document()));
+ }
+ }
} else {
- perror("Extension::Script: Unknown error for popen\n");
+ oldroot->appendChild(child->duplicate(oldroot->document()));
}
- return 0;
}
- Inkscape::IO::dump_fopen_call(fileout, "J");
- FILE *pfile = Inkscape::IO::fopen_utf8name(fileout, "w");
+ {
+ using Inkscape::Util::List;
+ using Inkscape::XML::AttributeRecord;
+ std::vector<gchar const *> attribs;
- if (pfile == NULL) {
- /* Error - could not open file */
- if (errno == EINVAL) {
- perror("Extension::Script: The mode provided to fopen was invalid\n");
- } else {
- perror("Extension::Script: Unknown error attempting to open temporary file\n");
+ // Make a list of all attributes of the old root node.
+ for (List<AttributeRecord const> iter = oldroot->attributeList(); iter; ++iter) {
+ attribs.push_back(g_quark_to_string(iter->key));
}
- return 0;
- }
- /* Copy pipe output to a temporary file */
- int amount_read = 0;
- char buf[BUFSIZE];
- int num_read;
- while ((num_read = pipe.read(buf, BUFSIZE)) != 0) {
- amount_read += num_read;
- fwrite(buf, 1, num_read, pfile);
- }
+ // Delete the attributes of the old root nodes.
+ for (std::vector<gchar const *>::const_iterator it = attribs.begin(); it != attribs.end(); it++)
+ oldroot->setAttribute(*it, NULL);
- /* Close file */
- if (fclose(pfile) == EOF) {
- if (errno == EBADF) {
- perror("Extension::Script: The filedescriptor for the temporary file is invalid\n");
- return 0;
- } else {
- perror("Extension::Script: Unknown error closing temporary file\n");
+ // Set the new attributes.
+ for (List<AttributeRecord const> iter = newroot->attributeList(); iter; ++iter) {
+ gchar const *name = g_quark_to_string(iter->key);
+ oldroot->setAttribute(name, newroot->attribute(name));
}
}
- /* Close pipe */
- if (!pipe.close()) {
- if (errno == EINVAL) {
- perror("Extension::Script: Invalid mode set for pclose\n");
- } else if (errno == ECHILD) {
- perror("Extension::Script: Could not obtain child status for pclose\n");
- } else {
- if (errorFile != NULL) {
- checkStderr(errorFile, Gtk::MESSAGE_ERROR,
- _("Inkscape has received an error from the script that it called. "
- "The text returned with the error is included below. "
- "Inkscape will continue working, but the action you requested has been cancelled."));
- } else {
- perror("Extension::Script: Unknown error for pclose\n");
- }
- }
- /* Could be a lie, but if there is an error, we don't want
- * to count on what was read being good */
- amount_read = 0;
- } else {
- if (errorFile != NULL) {
- checkStderr(errorFile, Gtk::MESSAGE_INFO,
- _("Inkscape has received additional data from the script executed. "
- "The script did not return an error, but this may indicate the results will not be as expected."));
- }
- }
-
- if (errorFile != NULL) {
- unlink(errorFile);
- g_free(errorFile);
- }
-
- return amount_read;
+ /** \todo Restore correct layer */
+ /** \todo Restore correct selection */
}
/** \brief This function checks the stderr file, and if it has data,
@@ -859,21 +868,14 @@ Script::execute (const gchar * in_command, const gchar * filein, const gchar * f
\param filename Filename of the stderr file
*/
void
-Script::checkStderr (gchar * filename, Gtk::MessageType type, gchar * message)
+Script::checkStderr (const Glib::ustring &data,
+ Gtk::MessageType type,
+ const Glib::ustring &message)
{
- // magic win32 crlf->lf conversion means the file length is not the same as
- // the text length, but luckily gtk will accept crlf in textviews so we can
- // just use binary mode
- std::ifstream stderrf (filename, std::ios_base::in | std::ios_base::binary);
- if (!stderrf.is_open()) return;
-
- stderrf.seekg(0, std::ios::end);
- int length = stderrf.tellg();
- if (0 == length) return;
- stderrf.seekg(0, std::ios::beg);
-
Gtk::MessageDialog warning(message, false, type, Gtk::BUTTONS_OK, true);
warning.set_resizable(true);
+ GtkWidget *dlg = GTK_WIDGET(warning.gobj());
+ sp_transientize(dlg);
Gtk::VBox * vbox = warning.get_vbox();
textview->set_wrap_mode(Gtk::WRAP_WORD);
textview->show();
- char * buffer = new char [length];
- stderrf.read(buffer, length);
- textview->get_buffer()->set_text(buffer, buffer + length);
- delete buffer;
- stderrf.close();
+ textview->get_buffer()->set_text(data.c_str());
Gtk::ScrolledWindow * scrollwindow = new Gtk::ScrolledWindow();
scrollwindow->add(*textview);
@@ -902,144 +900,180 @@ Script::checkStderr (gchar * filename, Gtk::MessageType type, gchar * message)
return;
}
-#ifdef WIN32
+bool
+Script::cancelProcessing (void) {
+ _canceled = true;
+ _main_loop->quit();
+ Glib::spawn_close_pid(_pid);
-bool pipe_t::open(char *command, char const *errorFile, int mode_p) {
- HANDLE pipe_write;
+ return true;
+}
- // Create pipe
- {
- SECURITY_ATTRIBUTES secattrs;
- ZeroMemory(&secattrs, sizeof(secattrs));
- secattrs.nLength = sizeof(secattrs);
- secattrs.lpSecurityDescriptor = 0;
- secattrs.bInheritHandle = TRUE;
- HANDLE t_pipe_read = 0;
- if ( !CreatePipe(&t_pipe_read, &pipe_write, &secattrs, 0) ) {
- errno = translate_error(GetLastError());
- return false;
- }
- // This duplicate handle makes the read pipe uninheritable
- if ( !DuplicateHandle(GetCurrentProcess(), t_pipe_read, GetCurrentProcess(), &hpipe, 0, FALSE, DUPLICATE_CLOSE_SOURCE | DUPLICATE_SAME_ACCESS) ) {
- int en = translate_error(GetLastError());
- CloseHandle(t_pipe_read);
- CloseHandle(pipe_write);
- errno = en;
- return false;
- }
- }
- // Open stderr file
- HANDLE hStdErrFile = CreateFile(errorFile, GENERIC_WRITE, FILE_SHARE_READ | FILE_SHARE_WRITE, NULL, CREATE_ALWAYS, 0, NULL);
- HANDLE hInheritableStdErr;
- DuplicateHandle(GetCurrentProcess(), hStdErrFile, GetCurrentProcess(), &hInheritableStdErr, 0, TRUE, DUPLICATE_CLOSE_SOURCE | DUPLICATE_SAME_ACCESS);
- // Create process
- {
- PROCESS_INFORMATION procinfo;
- STARTUPINFO startupinfo;
- ZeroMemory(&procinfo, sizeof(procinfo));
- ZeroMemory(&startupinfo, sizeof(startupinfo));
- startupinfo.cb = sizeof(startupinfo);
- //startupinfo.lpReserved = 0;
- //startupinfo.lpDesktop = 0;
- //startupinfo.lpTitle = 0;
- startupinfo.dwFlags = STARTF_USESTDHANDLES;
- startupinfo.hStdInput = GetStdHandle(STD_INPUT_HANDLE);
- startupinfo.hStdOutput = pipe_write;
- startupinfo.hStdError = hInheritableStdErr;
-
- if ( !CreateProcess(NULL, command, NULL, NULL, TRUE, 0, NULL, NULL, &startupinfo, &procinfo) ) {
- errno = translate_error(GetLastError());
- return false;
- }
- CloseHandle(procinfo.hThread);
- CloseHandle(procinfo.hProcess);
- }
+/** \brief This is the core of the extension file as it actually does
+ the execution of the extension.
+ \param in_command The command to be executed
+ \param filein Filename coming in
+ \param fileout Filename of the out file
+ \return Number of bytes that were read into the output file.
- // Close our copy of the write handle
- CloseHandle(hInheritableStdErr);
- CloseHandle(pipe_write);
+ The first thing that this function does is build the command to be
+ executed. This consists of the first string (in_command) and then
+ the filename for input (filein). This file is put on the command
+ line.
- return true;
-}
+ The next thing is that this function does is open a pipe to the
+ command and get the file handle in the ppipe variable. It then
+ opens the output file with the output file handle. Both of these
+ operations are checked extensively for errors.
-bool pipe_t::close() {
- BOOL retval = CloseHandle(hpipe);
- if ( !retval ) {
- errno = translate_error(GetLastError());
- }
- return retval != FALSE;
-}
+ After both are opened, then the data is copied from the output
+ of the pipe into the file out using fread and fwrite. These two
+ functions are used because of their primitive nature they make
+ no assumptions about the data. A buffer is used in the transfer,
+ but the output of fread is stored so the exact number of bytes
+ is handled gracefully.
-size_t pipe_t::read(void *buffer, size_t size) {
- DWORD bytes_read = 0;
- ReadFile(hpipe, buffer, size, &bytes_read, 0);
- return bytes_read;
-}
+ At the very end (after the data has been copied) both of the files
+ are closed, and we return to what we were doing.
+*/
+int
+Script::execute (const std::list<std::string> &in_command,
+ const std::list<std::string> &in_params,
+ const Glib::ustring &filein,
+ file_listener &fileout)
+{
+ g_return_val_if_fail(in_command.size() > 0, 0);
+ // printf("Executing\n");
-size_t pipe_t::write(void const *buffer, size_t size) {
- DWORD bytes_written = 0;
- WriteFile(hpipe, buffer, size, &bytes_written, 0);
- return bytes_written;
-}
+ std::vector <std::string> argv;
-int pipe_t::translate_error(DWORD err) {
- switch (err) {
- case ERROR_FILE_NOT_FOUND:
- return ENOENT;
- case ERROR_INVALID_HANDLE:
- case ERROR_INVALID_PARAMETER:
- return EINVAL;
- default:
- return 0;
+/*
+ for (std::list<std::string>::const_iterator i = in_command.begin();
+ i != in_command.end(); i++) {
+ argv.push_back(*i);
}
-}
+*/
+ // according to http://www.gtk.org/api/2.6/glib/glib-Spawning-Processes.html spawn quotes parameter containing spaces
+ // we tokenize so that spwan does not need to quote over all params
+ for (std::list<std::string>::const_iterator i = in_command.begin();
+ i != in_command.end(); i++) {
+ std::string param_str = *i;
+ do {
+ //g_message("param: %s", param_str.c_str());
+ size_t first_space = param_str.find_first_of(' ');
+ size_t first_quote = param_str.find_first_of('"');
+ //std::cout << "first space " << first_space << std::endl;
+ //std::cout << "first quote " << first_quote << std::endl;
+
+ if((first_quote != std::string::npos) && (first_quote == 0)) {
+ size_t next_quote = param_str.find_first_of('"', first_quote + 1);
+ //std::cout << "next quote " << next_quote << std::endl;
+
+ if(next_quote != std::string::npos) {
+ //std::cout << "now split " << next_quote << std::endl;
+ //std::cout << "now split " << param_str.substr(1, next_quote - 1) << std::endl;
+ //std::cout << "now split " << param_str.substr(next_quote + 1) << std::endl;
+ std::string part_str = param_str.substr(1, next_quote - 1);
+ if(part_str.size() > 0)
+ argv.push_back(part_str);
+ param_str = param_str.substr(next_quote + 1);
+
+ }
+ else {
+ if(param_str.size() > 0)
+ argv.push_back(param_str);
+ param_str = "";
+ }
-#else // Win32
+ }
+ else if(first_space != std::string::npos) {
+ //std::cout << "now split " << first_space << std::endl;
+ //std::cout << "now split " << param_str.substr(0, first_space) << std::endl;
+ //std::cout << "now split " << param_str.substr(first_space + 1) << std::endl;
+ std::string part_str = param_str.substr(0, first_space);
+ if(part_str.size() > 0)
+ argv.push_back(part_str);
+ param_str = param_str.substr(first_space + 1);
+ }
+ else {
+ if(param_str.size() > 0)
+ argv.push_back(param_str);
+ param_str = "";
+ }
+ } while(param_str.size() > 0);
+ }
-bool pipe_t::open(char *command, char const *errorFile, int mode_p) {
- char popen_mode[4] = {0,0,0,0};
- char *popen_mode_cur = popen_mode;
+ for (std::list<std::string>::const_iterator i = in_params.begin();
+ i != in_params.end(); i++) {
+ //g_message("Script parameter: %s",(*i)g.c_str());
+ argv.push_back(*i);
+ }
- if ( (mode_p & mode_read) != 0 ) {
- *popen_mode_cur++ = 'r';
+ if (!(filein.empty())) {
+ argv.push_back(filein);
}
- if ( (mode_p & mode_write) != 0 ) {
- *popen_mode_cur++ = 'w';
+ int stdout_pipe, stderr_pipe;
+
+ try {
+ Inkscape::IO::spawn_async_with_pipes(Glib::get_current_dir(), // working directory
+ argv, // arg v
+ Glib::SPAWN_SEARCH_PATH /*| Glib::SPAWN_DO_NOT_REAP_CHILD*/,
+ sigc::slot<void>(),
+ &_pid, // Pid
+ NULL, // STDIN
+ &stdout_pipe, // STDOUT
+ &stderr_pipe); // STDERR
+ } catch (Glib::SpawnError e) {
+ printf("Can't Spawn!!! spawn returns: %d\n", e.code());
+ return 0;
}
- /* Get the commandline to be run */
- if (errorFile != NULL) {
- char * temp;
- temp = g_strdup_printf("%s 2> %s", command, errorFile);
- ppipe = popen(temp, popen_mode);
- g_free(temp);
- } else
- ppipe = popen(command, popen_mode);
+ _main_loop = Glib::MainLoop::create(false);
- return ppipe != NULL;
-}
+ file_listener fileerr;
+ fileout.init(stdout_pipe, _main_loop);
+ fileerr.init(stderr_pipe, _main_loop);
-bool pipe_t::close() {
- return fclose(ppipe) == 0;
-}
+ _canceled = false;
+ _main_loop->run();
-size_t pipe_t::read(void *buffer, size_t size) {
- return fread(buffer, 1, size, ppipe);
-}
+ // Ensure all the data is out of the pipe
+ while (!fileout.isDead())
+ fileout.read(Glib::IO_IN);
+ while (!fileerr.isDead())
+ fileerr.read(Glib::IO_IN);
+
+ if (_canceled) {
+ // std::cout << "Script Canceled" << std::endl;
+ return 0;
+ }
+
+ Glib::ustring stderr_data = fileerr.string();
+ if (stderr_data.length() != 0 &&
+ Inkscape::NSApplication::Application::getUseGui()
+ ) {
+ checkStderr(stderr_data, Gtk::MESSAGE_INFO,
+ _("Inkscape has received additional data from the script executed. "
+ "The script did not return an error, but this may indicate the results will not be as expected."));
+ }
+
+ Glib::ustring stdout_data = fileout.string();
+ if (stdout_data.length() == 0) {
+ return 0;
+ }
-size_t pipe_t::write(void const *buffer, size_t size) {
- return fwrite(buffer, 1, size, ppipe);
+ // std::cout << "Finishing Execution." << std::endl;
+ return stdout_data.length();
}
-#endif // (Non-)Win32
-} /* Inkscape */
-} /* module */
-} /* Implementation */
+} // namespace Implementation
+} // namespace Extension
+} // namespace Inkscape
/*
Local Variables: