Clean up misc::isValidFileSystemName() function

src/misc.cpp

@@ -575,13 +575,10 @@ QString misc::toValidFileSystemName(QString filename) {
   return filename;
 }
 
-bool misc::isValidFileSystemName(QString filename) {
+bool misc::isValidFileSystemName(const QString& filename) {
-  filename.replace("\\", "/").trimmed();
   if(filename.isEmpty()) return false;
-  const QRegExp regex("[/:?\"*<>|]");
+  const QRegExp regex("[\\\\/:?\"*<>|]");
-  if(filename.contains(regex))
+  return !filename.contains(regex);
-    return false;
-  return true;
 }
 
 #ifndef DISABLE_GUI

src/misc.h

@@ -140,7 +140,7 @@ public:
   static bool isUrl(const QString &s);
   static void copyDir(QString src_path, QString dst_path);
   static QString toValidFileSystemName(QString filename);
-  static bool isValidFileSystemName(QString filename);
+  static bool isValidFileSystemName(const QString& filename);
 
   /* Ported from Qt4 to drop dependency on QtGui */
   static QString QDesktopServicesDataLocation();