]> www.fi.muni.cz Git - evince.git/blobdiff - backend/comics/comics-document.c
[windows] Use \r\n as delimiter to split lines on Windows
[evince.git] / backend / comics / comics-document.c
index 6ea6a7a1cdc741a165834c8eac6e1d22217ed1e3..b66f4ce7187d4a663610e4882ada6ca7c915c962 100644 (file)
@@ -1,5 +1,6 @@
 /* -*- Mode: C; tab-width: 8; indent-tabs-mode: t; c-basic-offset: 8; c-indent-level: 8 -*- */
 /*
+ * Copyright (C) 2009, Juanjo Marín <juanj.marin@juntadeandalucia.es>
  * Copyright (C) 2005, Teemu Tervo <teemu.tervo@gmx.net>
  *
  * This program is free software; you can redistribute it and/or modify
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
  */
 
-#include "config.h"
-
 #include <config.h>
+
 #include <unistd.h>
 #include <string.h>
-#include <glib/gi18n.h>
+#include <stdlib.h>
+#include <errno.h>
+
+#include <glib.h>
+#include <glib/gi18n-lib.h>
+#include <glib/gstdio.h>
 #include <gio/gio.h>
 
+#ifdef G_OS_WIN32
+# define WIFEXITED(x) ((x) != 3)
+# define WEXITSTATUS(x) (x)
+#else
+# include <sys/wait.h>
+#endif
+
 #include "comics-document.h"
 #include "ev-document-misc.h"
 #include "ev-document-thumbnails.h"
 #include "ev-file-helpers.h"
 
+#ifdef G_OS_WIN32
+/* On windows g_spawn_command_line_sync reads stdout in O_BINARY mode, not in O_TEXT mode.
+ * As a consequence, newlines are in a platform dependent representation (\r\n). This
+ * might be considered a bug in glib.
+ */
+#define EV_EOL "\r\n"
+#else
+#define EV_EOL "\n"
+#endif
+
+typedef enum
+{
+       RARLABS,
+       GNAUNRAR,
+       UNZIP,
+       P7ZIP,
+       TAR
+} ComicBookDecompressType;
+
+typedef struct _ComicsDocumentClass ComicsDocumentClass;
+
 struct _ComicsDocumentClass
 {
-       GObjectClass parent_class;
+       EvDocumentClass parent_class;
 };
 
 struct _ComicsDocument
 {
-       GObject parent_instance;
-
-       gchar  *archive;
-       GSList *page_names;
-       int     n_pages;
-       char   *extract_command;
-       gboolean regex_arg;
+       EvDocument parent_instance;
+
+       gchar    *archive, *dir;
+       GPtrArray *page_names;
+       gchar    *selected_command;
+       gchar    *extract_command, *list_command, *decompress_tmp;
+       gint     offset;
+       ComicBookDecompressType command_usage;
 };
 
-typedef struct _ComicsDocumentClass ComicsDocumentClass;
+#define OFFSET_7Z 53
+#define NO_OFFSET 0
+
+/* For perfomance reasons of 7z* we've choosen to decompress on the temporary 
+ * directory instead of decompressing on the stdout */
+
+/**
+ * @extract: command line arguments to pass to extract a file from the archive
+ *   to stdout.
+ * @list: command line arguments to list the archive contents
+ * @decompress_tmp: command line arguments to pass to extract the archive
+ *   into a directory.
+ * @offset: the position offset of the filename on each line in the output of
+ *   running the @list command
+ */
+typedef struct {
+        char *extract;
+        char *list;
+        char *decompress_tmp;
+        gint offset;
+} ComicBookDecompressCommand;
+
+static const ComicBookDecompressCommand command_usage_def[] = {
+        /* RARLABS unrar */
+       {"%s p -c- -ierr --", "%s vb -c- -- %s", NULL             , NO_OFFSET},
+
+        /* GNA! unrar */
+       {NULL               , "%s t %s"        , "%s -xf %s %s"   , NO_OFFSET},
+
+        /* unzip */
+       {"%s -p -C --"      , "%s -Z -1 -- %s" , NULL             , NO_OFFSET},
+
+        /* 7zip */
+       {NULL               , "%s l -- %s"     , "%s x -y %s -o%s", OFFSET_7Z},
+
+        /* tar */
+       {"%s -xOf"          , "%s -tf %s"      , NULL             , NO_OFFSET}
+};
 
-static void       comics_document_document_iface_init (EvDocumentIface *iface);
 static void       comics_document_document_thumbnails_iface_init (EvDocumentThumbnailsIface *iface);
 
 static GSList*    get_supported_image_extensions (void);
@@ -68,38 +138,232 @@ EV_BACKEND_REGISTER_WITH_CODE (ComicsDocument, comics_document,
                                                comics_document_document_thumbnails_iface_init);
        } );
 
-static char *
-comics_regex_quote (const char *s)
+
+/* This function manages the command for decompressing a comic book */
+static gboolean 
+comics_decompress_temp_dir (const gchar *command_decompress_tmp,
+                           const gchar *command, 
+                           GError      **error)
 {
-    char *ret, *d;
-
-    d = ret = g_malloc (strlen (s) * 4 + 3);
-    
-    *d++ = '\'';
-
-    for (; *s; s++, d++) {
-       switch (*s) {
-       case '?':
-       case '|':
-       case '[':
-       case ']':
-       case '*':
-       case '\\':
-           *d++ = '\\';
-           break;
-       case '\'':
-           *d++ = '\'';
-           *d++ = '\\';
-           *d++ = '\'';
-           break;
+       gboolean success;
+       gchar *std_out, *basename;
+       GError *err = NULL;
+       gint retval;
+       
+       success = g_spawn_command_line_sync (command_decompress_tmp, &std_out, 
+                                            NULL, &retval, &err);
+       basename = g_path_get_basename (command);
+       if (!success) {
+               g_set_error (error,
+                            EV_DOCUMENT_ERROR, 
+                            EV_DOCUMENT_ERROR_INVALID,
+                            _("Error launching the command “%s” in order to "
+                            "decompress the comic book: %s"),
+                            basename,
+                            err->message);
+               g_error_free (err);
+       } else if (WIFEXITED (retval)) {
+               if (WEXITSTATUS (retval) == EXIT_SUCCESS) {
+                       g_free (std_out);
+                       g_free (basename);
+                       return TRUE;
+               } else {
+                       g_set_error (error,
+                                    EV_DOCUMENT_ERROR,
+                                    EV_DOCUMENT_ERROR_INVALID,
+                                    _("The command “%s” failed at "
+                                    "decompressing the comic book."),
+                                    basename);
+                       g_free (std_out);
+               }
+       } else {
+               g_set_error (error,
+                            EV_DOCUMENT_ERROR,
+                            EV_DOCUMENT_ERROR_INVALID,
+                            _("The command “%s” did not end normally."),
+                            basename);
+               g_free (std_out);
        }
-       *d = *s;
-    }
-    
-    *d++ = '\'';
-    *d = '\0';
+       g_free (basename);
+       return FALSE;
+}
 
-    return ret;
+/* This function shows how to use the choosen command for decompressing a
+ * comic book file. It modifies fields of the ComicsDocument struct with 
+ * this information */
+static gboolean 
+comics_generate_command_lines (ComicsDocument *comics_document, 
+                              GError         **error)
+{
+       gchar *quoted_file;
+       gchar *quoted_command;
+       ComicBookDecompressType type;
+       
+       type = comics_document->command_usage;
+       quoted_file = g_shell_quote (comics_document->archive);
+       quoted_command = g_shell_quote (comics_document->selected_command);
+
+       comics_document->extract_command =
+                           g_strdup_printf (command_usage_def[type].extract,
+                                            quoted_command);
+       comics_document->list_command =
+                           g_strdup_printf (command_usage_def[type].list,
+                                            quoted_command, quoted_file);
+       comics_document->offset = command_usage_def[type].offset;
+       if (command_usage_def[type].decompress_tmp) {
+               comics_document->dir = ev_mkdtemp ("evince-comics-XXXXXX", error);
+                if (comics_document->dir == NULL)
+                        return FALSE;
+
+               /* unrar-free can't create directories, but ev_mkdtemp already created the dir */
+
+               comics_document->decompress_tmp =
+                       g_strdup_printf (command_usage_def[type].decompress_tmp, 
+                                        quoted_command, quoted_file,
+                                        comics_document->dir);
+               g_free (quoted_file);
+               g_free (quoted_command);
+
+               if (!comics_decompress_temp_dir (comics_document->decompress_tmp,
+                   comics_document->selected_command, error))
+                       return FALSE;
+               else
+                       return TRUE;
+       } else {
+               g_free (quoted_file);
+               g_free (quoted_command);
+               return TRUE;
+       }
+
+}
+
+/* This function chooses an external command for decompressing a comic 
+ * book based on its mime tipe. */
+static gboolean 
+comics_check_decompress_command        (gchar          *mime_type, 
+                                ComicsDocument *comics_document,
+                                GError         **error)
+{
+       gboolean success;
+       gchar *std_out, *std_err;
+       gint retval;
+       GError *err = NULL;
+       
+       /* FIXME, use proper cbr/cbz mime types once they're
+        * included in shared-mime-info */
+       
+       if (!strcmp (mime_type, "application/x-cbr") ||
+           !strcmp (mime_type, "application/x-rar")) {
+               /* The RARLAB provides a no-charge proprietary (freeware) 
+               * decompress-only client for Linux called unrar. Another 
+               * option is a GPLv2-licensed command-line tool developed by 
+               * the Gna! project. Confusingly enough, the free software RAR 
+               * decoder is also named unrar. For this reason we need to add 
+               * some lines for disambiguation. Sorry for the added the 
+               * complexity but it's life :)
+               * Finally, some distributions, like Debian, rename this free 
+               * option as unrar-free. 
+               * */
+               comics_document->selected_command = 
+                                       g_find_program_in_path ("unrar");
+               if (comics_document->selected_command) {
+                       /* We only use std_err to avoid printing useless error 
+                        * messages on the terminal */
+                       success = 
+                               g_spawn_command_line_sync (
+                                             comics_document->selected_command, 
+                                                          &std_out, &std_err,
+                                                          &retval, &err);
+                       if (!success) {
+                               g_propagate_error (error, err);
+                               g_error_free (err);
+                               return FALSE;
+                       /* I don't check retval status because RARLAB unrar 
+                        * doesn't have a way to return 0 without involving an 
+                        * operation with a file*/
+                       } else if (WIFEXITED (retval)) {
+                               if (g_strrstr (std_out,"freeware") != NULL)
+                                       /* The RARLAB freeware client */
+                                       comics_document->command_usage = RARLABS;
+                               else
+                                       /* The Gna! free software client */
+                                       comics_document->command_usage = GNAUNRAR;
+
+                               g_free (std_out);
+                               g_free (std_err);
+                               return TRUE;
+                       }
+               }
+               /* The Gna! free software client with Debian naming convention */
+               comics_document->selected_command = 
+                               g_find_program_in_path ("unrar-free");
+               if (comics_document->selected_command) {
+                       comics_document->command_usage = GNAUNRAR;
+                       return TRUE;
+               }
+
+       } else if (!strcmp (mime_type, "application/x-cbz") ||
+                  !strcmp (mime_type, "application/zip")) {
+               /* InfoZIP's unzip program */
+               comics_document->selected_command = 
+                               g_find_program_in_path ("unzip");
+               if (comics_document->selected_command) {
+                       comics_document->command_usage = UNZIP;
+                       return TRUE;
+               }
+
+       } else if (!strcmp (mime_type, "application/x-cb7") ||
+                  !strcmp (mime_type, "application/x-7z-compressed")) {
+               /* 7zr, 7za and 7z are the commands from the p7zip project able 
+                * to decompress .7z files */ 
+                       comics_document->selected_command = 
+                               g_find_program_in_path ("7zr");
+                       if (comics_document->selected_command) {
+                               comics_document->command_usage = P7ZIP;
+                               return TRUE;
+                       }
+                       comics_document->selected_command = 
+                               g_find_program_in_path ("7za");
+                       if (comics_document->selected_command) {
+                               comics_document->command_usage = P7ZIP;
+                               return TRUE;
+                       }
+                       comics_document->selected_command = 
+                               g_find_program_in_path ("7z");
+                       if (comics_document->selected_command) {
+                               comics_document->command_usage = P7ZIP;
+                               return TRUE;
+                       }
+       } else if (!strcmp (mime_type, "application/x-cbt") ||
+                  !strcmp (mime_type, "application/x-tar")) {
+               /* tar utility (Tape ARchive) */
+               comics_document->selected_command =
+                               g_find_program_in_path ("tar");
+               if (comics_document->selected_command) {
+                       comics_document->command_usage = TAR;
+                       return TRUE;
+               }
+       } else {
+               g_set_error (error,
+                            EV_DOCUMENT_ERROR,
+                            EV_DOCUMENT_ERROR_INVALID,
+                            _("Not a comic book MIME type: %s"),
+                            mime_type);
+                            return FALSE;
+       }
+       g_set_error_literal (error,
+                            EV_DOCUMENT_ERROR,
+                            EV_DOCUMENT_ERROR_INVALID,
+                            _("Can't find an appropriate command to "
+                            "decompress this type of comic book"));
+       return FALSE;
+}
+
+static int
+sort_page_names (gconstpointer a,
+                 gconstpointer b)
+{
+  return strcmp (* (const char **) a, * (const char **) b);
 }
 
 static gboolean
@@ -109,87 +373,101 @@ comics_document_load (EvDocument *document,
 {
        ComicsDocument *comics_document = COMICS_DOCUMENT (document);
        GSList *supported_extensions;
-       gchar *list_files_command = NULL, *std_out, *quoted_file;
+       gchar *std_out;
        gchar *mime_type;
-       gchar **cbr_files;
+       gchar **cb_files, *cb_file;
        gboolean success;
        int i, retval;
+       GError *err = NULL;
 
        comics_document->archive = g_filename_from_uri (uri, NULL, error);
-       g_return_val_if_fail (comics_document->archive != NULL, FALSE);
+       if (!comics_document->archive)
+               return FALSE;
 
-       quoted_file = g_shell_quote (comics_document->archive);
-       mime_type = ev_file_get_mime_type (uri, FALSE, NULL);
+       mime_type = ev_file_get_mime_type (uri, FALSE, &err);
+       if (!mime_type) {
+               if (err) {
+                       g_propagate_error (error, err);
+               } else {
+                       g_set_error_literal (error,
+                                            EV_DOCUMENT_ERROR,
+                                            EV_DOCUMENT_ERROR_INVALID,
+                                            _("Unknown MIME Type"));
+               }
 
-       /* FIXME, use proper cbr/cbz mime types once they're
-        * included in shared-mime-info */
-       if (!strcmp (mime_type, "application/x-cbr") ||
-           !strcmp (mime_type, "application/x-rar")) {
-               comics_document->extract_command =
-                       g_strdup ("unrar p -c- -ierr");
-               list_files_command =
-                       g_strdup_printf ("unrar vb -c- -- %s", quoted_file);
-               comics_document->regex_arg = FALSE;
-       } else if (!strcmp (mime_type, "application/x-cbz") ||
-                  !strcmp (mime_type, "application/x-zip")) {
-               comics_document->extract_command =
-                       g_strdup ("unzip -p -C");
-               list_files_command = 
-                       g_strdup_printf ("zipinfo -1 -- %s", quoted_file);
-               comics_document->regex_arg = TRUE;
-       } else if (!strcmp (mime_type, "application/x-cb7")) {
-               comics_document->extract_command =
-                       g_strdup ("7zr x -so");
-               list_files_command = 
-                       g_strdup_printf ("7zr l -- %s", quoted_file);
-               comics_document->regex_arg = TRUE;
+               return FALSE;
+       }
+       
+       if (!comics_check_decompress_command (mime_type, comics_document, 
+       error)) {       
+               g_free (mime_type);
+               return FALSE;
+       } else if (!comics_generate_command_lines (comics_document, error)) {
+                  g_free (mime_type);
+               return FALSE;
        }
 
        g_free (mime_type);
-       g_free (quoted_file);
 
        /* Get list of files in archive */
-       success = g_spawn_command_line_sync (list_files_command,
+       success = g_spawn_command_line_sync (comics_document->list_command,
                                             &std_out, NULL, &retval, error);
-       g_free (list_files_command);
 
        if (!success) {
                return FALSE;
-       } else if (retval != 0) {
-               g_set_error (error,
-                            EV_DOCUMENT_ERROR,
-                            EV_DOCUMENT_ERROR_INVALID,
-                            _("File corrupted."));
+       } else if (!WIFEXITED(retval) || WEXITSTATUS(retval) != EXIT_SUCCESS) {
+               g_set_error_literal (error,
+                                     EV_DOCUMENT_ERROR,
+                                     EV_DOCUMENT_ERROR_INVALID,
+                                     _("File corrupted"));
+               return FALSE;
+       }
+
+       /* FIXME: is this safe against filenames containing \n in the archive ? */
+       cb_files = g_strsplit (std_out, EV_EOL, 0);
+
+       g_free (std_out);
+
+       if (!cb_files) {
+               g_set_error_literal (error,
+                                    EV_DOCUMENT_ERROR,
+                                    EV_DOCUMENT_ERROR_INVALID,
+                                    _("No files in archive"));
                return FALSE;
        }
 
-       cbr_files = g_strsplit (std_out, "\n", 0);
+        comics_document->page_names = g_ptr_array_sized_new (64);
+
        supported_extensions = get_supported_image_extensions ();
-       for (i = 0; cbr_files[i] != NULL; i++) {
-               gchar *suffix = g_strrstr (cbr_files[i], ".");
+       for (i = 0; cb_files[i] != NULL; i++) {
+               if (comics_document->offset != NO_OFFSET) {
+                       if (g_utf8_strlen (cb_files[i],-1) > 
+                           comics_document->offset) {
+                               cb_file = 
+                                       g_utf8_offset_to_pointer (cb_files[i], 
+                                                      comics_document->offset);
+                       } else {
+                               continue;
+                       }
+               } else {
+                       cb_file = cb_files[i];
+               }
+               gchar *suffix = g_strrstr (cb_file, ".");
                if (!suffix)
                        continue;
                suffix = g_ascii_strdown (suffix + 1, -1);
-
                if (g_slist_find_custom (supported_extensions, suffix,
                                         (GCompareFunc) strcmp) != NULL) {
-                       comics_document->page_names =
-                               g_slist_insert_sorted (
-                                       comics_document->page_names,
-                                       g_strdup (g_strstrip (cbr_files[i])),
-                                       (GCompareFunc) strcmp);
-                       comics_document->n_pages++;
+                        g_ptr_array_add (comics_document->page_names,
+                                         g_strstrip (g_strdup (cb_file)));
                }
-
                g_free (suffix);
        }
-
-       g_free (std_out);
-       g_strfreev (cbr_files);
+       g_strfreev (cb_files);
        g_slist_foreach (supported_extensions, (GFunc) g_free, NULL);
        g_slist_free (supported_extensions);
 
-       if (comics_document->n_pages == 0) {
+       if (comics_document->page_names->len == 0) {
                g_set_error (error,
                             EV_DOCUMENT_ERROR,
                             EV_DOCUMENT_ERROR_INVALID,
@@ -198,6 +476,9 @@ comics_document_load (EvDocument *document,
                return FALSE;
        }
 
+        /* Now sort the pages */
+        g_ptr_array_sort (comics_document->page_names, sort_page_names);
+
        return TRUE;
 }
 
@@ -215,7 +496,12 @@ comics_document_save (EvDocument *document,
 static int
 comics_document_get_n_pages (EvDocument *document)
 {
-       return COMICS_DOCUMENT (document)->n_pages;
+       ComicsDocument *comics_document = COMICS_DOCUMENT (document);
+
+        if (comics_document->page_names == NULL)
+                return 0;
+
+       return comics_document->page_names->len;
 }
 
 static void
@@ -229,44 +515,61 @@ comics_document_get_page_size (EvDocument *document,
        guchar buf[1024];
        gboolean success, got_size = FALSE;
        gint outpipe = -1;
-       GPid child_pid = -1;
-
-       argv = extract_argv (document, page->index);
-       success = g_spawn_async_with_pipes (NULL, argv, NULL,
-                                           G_SPAWN_SEARCH_PATH | G_SPAWN_STDERR_TO_DEV_NULL,
-                                           NULL, NULL,
-                                           &child_pid,
-                                           NULL, &outpipe, NULL, NULL);
-       g_strfreev (argv);
-       g_return_if_fail (success == TRUE);
-
-       loader = gdk_pixbuf_loader_new ();
-       g_signal_connect (loader, "area-prepared",
-                         G_CALLBACK (get_page_size_area_prepared_cb),
-                         &got_size);
-
-       while (outpipe >= 0) {
-               gssize bytes = read (outpipe, buf, 1024);
+       GPid child_pid;
+       gssize bytes;
+       GdkPixbuf *pixbuf;
+       gchar *filename;
+       ComicsDocument *comics_document = COMICS_DOCUMENT (document);
+       
+       if (!comics_document->decompress_tmp) {
+               argv = extract_argv (document, page->index);
+               success = g_spawn_async_with_pipes (NULL, argv, NULL,
+                                                   G_SPAWN_SEARCH_PATH | 
+                                                   G_SPAWN_STDERR_TO_DEV_NULL,
+                                                   NULL, NULL,
+                                                   &child_pid,
+                                                   NULL, &outpipe, NULL, NULL);
+               g_strfreev (argv);
+               g_return_if_fail (success == TRUE);
+
+               loader = gdk_pixbuf_loader_new ();
+               g_signal_connect (loader, "area-prepared",
+                                 G_CALLBACK (get_page_size_area_prepared_cb),
+                                 &got_size);
+
+               while (outpipe >= 0) {
+                       bytes = read (outpipe, buf, 1024);
                
-               if (bytes > 0)
+                       if (bytes > 0)
                        gdk_pixbuf_loader_write (loader, buf, bytes, NULL);
-               if (bytes <= 0 || got_size) {
-                       close (outpipe);
-                       outpipe = -1;
-                       gdk_pixbuf_loader_close (loader, NULL);
+                       if (bytes <= 0 || got_size) {
+                               close (outpipe);
+                               outpipe = -1;
+                               gdk_pixbuf_loader_close (loader, NULL);
+                       }
                }
-       }
 
-       if (gdk_pixbuf_loader_get_pixbuf (loader)) {
-               GdkPixbuf *pixbuf = gdk_pixbuf_loader_get_pixbuf (loader);
+               if (gdk_pixbuf_loader_get_pixbuf (loader)) {
+                       pixbuf = gdk_pixbuf_loader_get_pixbuf (loader);
+                       if (width)
+                               *width = gdk_pixbuf_get_width (pixbuf);
+                       if (height)
+                               *height = gdk_pixbuf_get_height (pixbuf);
+               }
+
+               g_spawn_close_pid (child_pid);
+               g_object_unref (loader);
+       } else {
+               filename = g_build_filename (comics_document->dir,
+                                             (char *) comics_document->page_names->pdata[page->index],
+                                            NULL);
+               pixbuf = gdk_pixbuf_new_from_file (filename, NULL);
+               g_free (filename);
                if (width)
                        *width = gdk_pixbuf_get_width (pixbuf);
                if (height)
                        *height = gdk_pixbuf_get_height (pixbuf);
        }
-
-       g_spawn_close_pid (child_pid);
-       g_object_unref (loader);
 }
 
 static void
@@ -287,39 +590,62 @@ comics_document_render_pixbuf (EvDocument      *document,
        guchar buf[4096];
        gboolean success;
        gint outpipe = -1;
-       GPid child_pid = -1;
-
-       argv = extract_argv (document, rc->page->index);
-       success = g_spawn_async_with_pipes (NULL, argv, NULL,
-                                           G_SPAWN_SEARCH_PATH
-                                           | G_SPAWN_STDERR_TO_DEV_NULL,
-                                           NULL, NULL,
-                                           &child_pid,
-                                           NULL, &outpipe, NULL, NULL);
-       g_strfreev (argv);
-       g_return_val_if_fail (success == TRUE, NULL);
-
-       loader = gdk_pixbuf_loader_new ();
-       g_signal_connect (loader, "size-prepared",
-                         G_CALLBACK (render_pixbuf_size_prepared_cb), &rc->scale);
-
-       while (outpipe >= 0) {
-               gssize bytes = read (outpipe, buf, 4096);
-
-               if (bytes > 0) {
-                       gdk_pixbuf_loader_write (loader, buf, bytes, NULL);
-               } else if (bytes <= 0) {
-                       close (outpipe);
-                       gdk_pixbuf_loader_close (loader, NULL);
-                       outpipe = -1;
+       GPid child_pid;
+       gssize bytes;
+       gint width, height;
+       gchar *filename;
+       ComicsDocument *comics_document = COMICS_DOCUMENT (document);
+       
+       if (!comics_document->decompress_tmp) {
+               argv = extract_argv (document, rc->page->index);
+               success = g_spawn_async_with_pipes (NULL, argv, NULL,
+                                                   G_SPAWN_SEARCH_PATH | 
+                                                   G_SPAWN_STDERR_TO_DEV_NULL,
+                                                   NULL, NULL,
+                                                   &child_pid,
+                                                   NULL, &outpipe, NULL, NULL);
+               g_strfreev (argv);
+               g_return_val_if_fail (success == TRUE, NULL);
+
+               loader = gdk_pixbuf_loader_new ();
+               g_signal_connect (loader, "size-prepared",
+                                 G_CALLBACK (render_pixbuf_size_prepared_cb), 
+                                 &rc->scale);
+
+               while (outpipe >= 0) {
+                       bytes = read (outpipe, buf, 4096);
+
+                       if (bytes > 0) {
+                               gdk_pixbuf_loader_write (loader, buf, bytes, 
+                               NULL);
+                       } else if (bytes <= 0) {
+                               close (outpipe);
+                               gdk_pixbuf_loader_close (loader, NULL);
+                               outpipe = -1;
+                       }
                }
-       }
-
-       rotated_pixbuf = gdk_pixbuf_rotate_simple (gdk_pixbuf_loader_get_pixbuf (loader),
-                                                  360 - rc->rotation);
-       g_spawn_close_pid (child_pid);
-       g_object_unref (loader);
 
+               rotated_pixbuf = gdk_pixbuf_rotate_simple (
+                                       gdk_pixbuf_loader_get_pixbuf (loader),
+                                       360 - rc->rotation);
+               g_spawn_close_pid (child_pid);
+               g_object_unref (loader);
+       } else {
+               filename = 
+                       g_build_filename (comics_document->dir,
+                                          (char *) comics_document->page_names->pdata[rc->page->index],
+                                         NULL);
+          
+               gdk_pixbuf_get_file_info (filename, &width, &height);
+               
+               rotated_pixbuf = 
+                 gdk_pixbuf_rotate_simple (gdk_pixbuf_new_from_file_at_size (
+                                           filename, width * (rc->scale) + 0.5,
+                                           height * (rc->scale) + 0.5, NULL),
+                                           360 - rc->rotation);
+               g_free (filename);
+       
+       }
        return rotated_pixbuf;
 }
 
@@ -350,22 +676,59 @@ render_pixbuf_size_prepared_cb (GdkPixbufLoader *loader,
        gdk_pixbuf_loader_set_size (loader, w, h);
 }
 
+/**
+ * comics_remove_dir: Removes a directory recursively. 
+ * Returns:
+ *     0 if it was successfully deleted,
+ *     -1 if an error occurred                 
+ */
+static int 
+comics_remove_dir (gchar *path_name) 
+{
+       GDir  *content_dir;
+       const gchar *filename;
+       gchar *filename_with_path;
+       
+       if (g_file_test (path_name, G_FILE_TEST_IS_DIR)) {
+               content_dir = g_dir_open  (path_name, 0, NULL);
+               filename  = g_dir_read_name (content_dir);
+               while (filename) {
+                       filename_with_path = 
+                               g_build_filename (path_name, 
+                                                 filename, NULL);
+                       comics_remove_dir (filename_with_path);
+                       g_free (filename_with_path);
+                       filename = g_dir_read_name (content_dir);
+               }
+               g_dir_close (content_dir);
+       }
+       /* Note from g_remove() documentation: on Windows, it is in general not 
+        * possible to remove a file that is open to some process, or mapped 
+        * into memory.*/
+       return (g_remove (path_name));
+}
+
 static void
 comics_document_finalize (GObject *object)
 {
        ComicsDocument *comics_document = COMICS_DOCUMENT (object);
-
-       if (comics_document->archive)
-               g_free (comics_document->archive);
-
+       
+       if (comics_document->decompress_tmp) {
+               if (comics_remove_dir (comics_document->dir) == -1)
+                       g_warning (_("There was an error deleting “%s”."),
+                                  comics_document->dir);
+               g_free (comics_document->dir);
+       }
+       
        if (comics_document->page_names) {
-               g_slist_foreach (comics_document->page_names,
-                                (GFunc) g_free, NULL);
-               g_slist_free (comics_document->page_names);
+                g_ptr_array_foreach (comics_document->page_names, (GFunc) g_free, NULL);
+                g_ptr_array_free (comics_document->page_names, TRUE);
        }
 
-       if (comics_document->extract_command)
-               g_free (comics_document->extract_command);
+       g_free (comics_document->archive);
+       g_free (comics_document->selected_command);
+       g_free (comics_document->extract_command);
+       g_free (comics_document->list_command);
 
        G_OBJECT_CLASS (comics_document_parent_class)->finalize (object);
 }
@@ -373,27 +736,16 @@ comics_document_finalize (GObject *object)
 static void
 comics_document_class_init (ComicsDocumentClass *klass)
 {
-       GObjectClass *gobject_class = G_OBJECT_CLASS (klass);
-       gobject_class->finalize = comics_document_finalize;
-}
+       GObjectClass    *gobject_class = G_OBJECT_CLASS (klass);
+       EvDocumentClass *ev_document_class = EV_DOCUMENT_CLASS (klass);
 
-static EvDocumentInfo *
-comics_document_get_info (EvDocument *document)
-{
-       EvDocumentInfo *info;
-       info = g_new0 (EvDocumentInfo, 1);
-       return info;
-}
+       gobject_class->finalize = comics_document_finalize;
 
-static void
-comics_document_document_iface_init (EvDocumentIface *iface)
-{
-       iface->load = comics_document_load;
-       iface->save = comics_document_save;
-       iface->get_n_pages = comics_document_get_n_pages;
-       iface->get_page_size = comics_document_get_page_size;
-       iface->render = comics_document_render;
-       iface->get_info = comics_document_get_info;
+       ev_document_class->load = comics_document_load;
+       ev_document_class->save = comics_document_save;
+       ev_document_class->get_n_pages = comics_document_get_n_pages;
+       ev_document_class->get_page_size = comics_document_get_page_size;
+       ev_document_class->render = comics_document_render;
 }
 
 static void
@@ -402,7 +754,6 @@ comics_document_init (ComicsDocument *comics_document)
        comics_document->archive = NULL;
        comics_document->page_names = NULL;
        comics_document->extract_command = NULL;
-       comics_document->n_pages = 0;
 }
 
 /* Returns a list of file extensions supported by gdk-pixbuf */
@@ -483,27 +834,24 @@ extract_argv (EvDocument *document, gint page)
        char *command_line, *quoted_archive, *quoted_filename;
        GError *err = NULL;
 
+        if (page >= comics_document->page_names->len)
+                return NULL;
+
        quoted_archive = g_shell_quote (comics_document->archive);
-       if (comics_document->regex_arg) {
-               quoted_filename = comics_regex_quote (
-                       g_slist_nth_data (comics_document->page_names, page));
-       } else {
-               quoted_filename = g_shell_quote (
-                       g_slist_nth_data (comics_document->page_names, page));
-       }
+       quoted_filename = g_shell_quote (comics_document->page_names->pdata[page]);
 
-       command_line = g_strdup_printf ("%s -- %s %s",
+       command_line = g_strdup_printf ("%s %s %s",
                                        comics_document->extract_command,
                                        quoted_archive,
                                        quoted_filename);
-
        g_shell_parse_argv (command_line, NULL, &argv, &err);
-       
+
        if (err) {
-               g_warning ("Error %s", err->message);
+               g_warning (_("Error %s"), err->message);
+               g_error_free (err);
                return NULL;
        }
-       
+
        g_free (command_line);
        g_free (quoted_archive);
        g_free (quoted_filename);