From: pjrm <pjrm@users.sourceforge.net>
Date: Tue, 7 Apr 2009 04:04:51 +0000 (+0000)
Subject: functional noop: Clarify g_strcanon behaviour by not redundantly reassigning back... 
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=403701fecda3ae3813c8bdb2748d9291c70bf5d0;p=inkscape.git

functional noop: Clarify g_strcanon behaviour by not redundantly reassigning back to argument.
---

diff --git a/src/selection-chemistry.cpp b/src/selection-chemistry.cpp
index 891e958a5..2ea1de143 100644
--- a/src/selection-chemistry.cpp
+++ b/src/selection-chemistry.cpp
@@ -2519,7 +2519,7 @@ sp_selection_create_bitmap_copy (SPDesktop *desktop)
     gchar *filename = g_strdup_printf ("%s-%s-%u.png", document->name, SP_OBJECT_REPR(items->data)->attribute("id"), current);
     // Imagemagick is known not to handle spaces in filenames, so we replace anything but letters,
     // digits, and a few other chars, with "_"
-    filename = g_strcanon (filename, "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789-_.=+~$#@^&!?", '_');
+    g_strcanon(filename, "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789-_.=+~$#@^&!?", '_');
 
     // Build the complete path by adding document base dir, if set, otherwise home dir
     gchar * directory = NULL;