From 427b7f8553c745f7598c81aa75aeed7503a37ec3 Mon Sep 17 00:00:00 2001
From: Cory Bloor <cgbloor@ucalgary.ca>
Date: Thu, 12 Oct 2017 02:38:54 -0600
Subject: [PATCH] Revert "Set default extensions for most save files"

This reverts commit 6f21bacd82b107b03db829c96567e638b94ada81.
---
 qrenderdoc/Code/QRDUtils.cpp                               | 5 ++---
 qrenderdoc/Code/QRDUtils.h                                 | 1 -
 qrenderdoc/Windows/Dialogs/CaptureDialog.cpp               | 2 +-
 qrenderdoc/Windows/Dialogs/PerformanceCounterSelection.cpp | 5 ++---
 qrenderdoc/Windows/Dialogs/TextureSaveDialog.cpp           | 4 ++--
 qrenderdoc/Windows/MainWindow.cpp                          | 4 ++--
 qrenderdoc/Windows/PythonShell.cpp                         | 2 +-
 7 files changed, 10 insertions(+), 13 deletions(-)

diff --git a/qrenderdoc/Code/QRDUtils.cpp b/qrenderdoc/Code/QRDUtils.cpp
index e2e11606d..43791add8 100644
--- a/qrenderdoc/Code/QRDUtils.cpp
+++ b/qrenderdoc/Code/QRDUtils.cpp
@@ -537,12 +537,11 @@ QString RDDialog::getExecutableFileName(QWidget *parent, const QString &caption,
 }
 
 QString RDDialog::getSaveFileName(QWidget *parent, const QString &caption, const QString &dir,
-                                  const QString &filter, const QString &defaultExt,
-                                  QString *selectedFilter, QFileDialog::Options options)
+                                  const QString &filter, QString *selectedFilter,
+                                  QFileDialog::Options options)
 {
   QFileDialog fd(parent, caption, dir, filter);
   fd.setAcceptMode(QFileDialog::AcceptSave);
-  fd.setDefaultSuffix(defaultExt);
   fd.setOptions(options);
   show(&fd);
 
diff --git a/qrenderdoc/Code/QRDUtils.h b/qrenderdoc/Code/QRDUtils.h
index ecfd91b03..d4808fbc0 100644
--- a/qrenderdoc/Code/QRDUtils.h
+++ b/qrenderdoc/Code/QRDUtils.h
@@ -877,7 +877,6 @@ struct RDDialog
 
   static QString getSaveFileName(QWidget *parent = NULL, const QString &caption = QString(),
                                  const QString &dir = QString(), const QString &filter = QString(),
-                                 const QString &defaultExt = QString(),
                                  QString *selectedFilter = NULL,
                                  QFileDialog::Options options = QFileDialog::Options());
 };
diff --git a/qrenderdoc/Windows/Dialogs/CaptureDialog.cpp b/qrenderdoc/Windows/Dialogs/CaptureDialog.cpp
index c53cc2e16..6a548c081 100644
--- a/qrenderdoc/Windows/Dialogs/CaptureDialog.cpp
+++ b/qrenderdoc/Windows/Dialogs/CaptureDialog.cpp
@@ -857,7 +857,7 @@ void CaptureDialog::on_toggleGlobal_clicked()
 void CaptureDialog::on_saveSettings_clicked()
 {
   QString filename = RDDialog::getSaveFileName(this, tr("Save Settings As"), QString(),
-                                               tr("Capture settings (*.cap)"), lit("cap"));
+                                               tr("Capture settings (*.cap)"));
 
   if(!filename.isEmpty())
   {
diff --git a/qrenderdoc/Windows/Dialogs/PerformanceCounterSelection.cpp b/qrenderdoc/Windows/Dialogs/PerformanceCounterSelection.cpp
index 2a8501f7a..85dd0c78a 100644
--- a/qrenderdoc/Windows/Dialogs/PerformanceCounterSelection.cpp
+++ b/qrenderdoc/Windows/Dialogs/PerformanceCounterSelection.cpp
@@ -353,9 +353,8 @@ void PerformanceCounterSelection::SetSelectedCounters(const QList<GPUCounter> &c
 
 void PerformanceCounterSelection::Save()
 {
-  QString filename =
-      RDDialog::getSaveFileName(this, tr("Save File"), QDir::homePath(),
-                                tr("Performance Counter Settings (*.json)"), lit("json"));
+  QString filename = RDDialog::getSaveFileName(this, tr("Save File"), QDir::homePath(),
+                                               tr("Performance Counter Settings (*.json)"));
 
   if(filename.isEmpty())
     return;
diff --git a/qrenderdoc/Windows/Dialogs/TextureSaveDialog.cpp b/qrenderdoc/Windows/Dialogs/TextureSaveDialog.cpp
index 211b2b1ff..716abb84d 100644
--- a/qrenderdoc/Windows/Dialogs/TextureSaveDialog.cpp
+++ b/qrenderdoc/Windows/Dialogs/TextureSaveDialog.cpp
@@ -526,8 +526,8 @@ void TextureSaveDialog::on_browse_clicked()
 
   QString *selectedFilter = NULL;
 
-  QString filename = RDDialog::getSaveFileName(this, tr("Save Texture As"), QString(), filter,
-                                               QString(), selectedFilter);
+  QString filename =
+      RDDialog::getSaveFileName(this, tr("Save Texture As"), QString(), filter, selectedFilter);
 
   QFileInfo checkFile(filename);
   if(!filename.isEmpty())
diff --git a/qrenderdoc/Windows/MainWindow.cpp b/qrenderdoc/Windows/MainWindow.cpp
index fecb94171..f0c5ae899 100644
--- a/qrenderdoc/Windows/MainWindow.cpp
+++ b/qrenderdoc/Windows/MainWindow.cpp
@@ -571,8 +571,8 @@ QString MainWindow::GetSavePath()
       dir = m_LastSaveCapturePath;
   }
 
-  QString filename = RDDialog::getSaveFileName(this, tr("Save Capture As"), dir,
-                                               tr("Capture Files (*.rdc)"), lit("rdc"));
+  QString filename =
+      RDDialog::getSaveFileName(this, tr("Save Capture As"), dir, tr("Capture Files (*.rdc)"));
 
   if(!filename.isEmpty())
   {
diff --git a/qrenderdoc/Windows/PythonShell.cpp b/qrenderdoc/Windows/PythonShell.cpp
index 27ed350d3..2dd990604 100644
--- a/qrenderdoc/Windows/PythonShell.cpp
+++ b/qrenderdoc/Windows/PythonShell.cpp
@@ -511,7 +511,7 @@ void PythonShell::on_openScript_clicked()
 void PythonShell::on_saveScript_clicked()
 {
   QString filename = RDDialog::getSaveFileName(this, tr("Save Python Script"), QString(),
-                                               tr("Python scripts (*.py)"), lit("py"));
+                                               tr("Python scripts (*.py)"));
 
   if(!filename.isEmpty())
   {