From 87b89a605f0c4e5fcabf9061d1a13c9caa65a824 Mon Sep 17 00:00:00 2001
From: Robin Gareus <robin@gareus.org>
Date: Thu, 16 Jul 2015 19:29:45 +0200
Subject: [PATCH] Revert "Possible fix for
 http://tracker.ardour.org/view.php?id=6332"

This reverts commit 1a619472ca0d7514831476bb9be9980ffbd91f46.

On Unix systems "#define g_open open" interferes with class member function
IMHO this is the wrong approach, the filename should be converted using
glib::filename_from_utf8().
---
 gtk2_ardour/sfdb_ui.cc           |  9 +++++----
 libs/ardour/sndfileimportable.cc |  9 +--------
 libs/ardour/sndfilesource.cc     | 14 ++------------
 3 files changed, 8 insertions(+), 24 deletions(-)

diff --git a/gtk2_ardour/sfdb_ui.cc b/gtk2_ardour/sfdb_ui.cc
index 1ab1faa206..6996273650 100644
--- a/gtk2_ardour/sfdb_ui.cc
+++ b/gtk2_ardour/sfdb_ui.cc
@@ -29,6 +29,7 @@
 
 #include <unistd.h>
 #include <limits.h>
+#include <sys/stat.h>
 
 #include <gtkmm/box.h>
 #include <gtkmm/stock.h>
@@ -1254,8 +1255,8 @@ SoundFileBrowser::get_paths ()
 		vector<string>::iterator i;
 
 		for (i = filenames.begin(); i != filenames.end(); ++i) {
-			GStatBuf buf;
-			if ((!g_stat((*i).c_str(), &buf)) && S_ISREG(buf.st_mode)) {
+			struct stat buf;
+			if ((!stat((*i).c_str(), &buf)) && S_ISREG(buf.st_mode)) {
 				results.push_back (*i);
 			}
 		}
@@ -1589,7 +1590,7 @@ SoundFileOmega::check_link_status (const Session* s, const vector<string>& paths
 	std::string tmpdir(Glib::build_filename (s->session_directory().sound_path(), "linktest"));
 	bool ret = false;
 
-	if (g_mkdir (tmpdir.c_str(), 0744)) {
+	if (mkdir (tmpdir.c_str(), 0744)) {
 		if (errno != EEXIST) {
 			return false;
 		}
@@ -1613,7 +1614,7 @@ SoundFileOmega::check_link_status (const Session* s, const vector<string>& paths
 	ret = true;
 
   out:
-	g_rmdir (tmpdir.c_str());
+	rmdir (tmpdir.c_str());
 	return ret;
 #endif
 }
diff --git a/libs/ardour/sndfileimportable.cc b/libs/ardour/sndfileimportable.cc
index ffc894b2d4..5ccab2c0d2 100644
--- a/libs/ardour/sndfileimportable.cc
+++ b/libs/ardour/sndfileimportable.cc
@@ -24,9 +24,6 @@
 #include "pbd/error.h"
 #include "ardour/sndfileimportable.h"
 
-#include <fcntl.h>
-#include <glib/gstdio.h>
-
 using namespace ARDOUR;
 using namespace std;
 
@@ -70,12 +67,8 @@ SndFileImportableSource::get_timecode_info (SNDFILE* sf, SF_BROADCAST_INFO* binf
 
 SndFileImportableSource::SndFileImportableSource (const string& path)
 {
-	int fd;
-	if ((-1) == (fd = g_open (path.c_str(), O_RDONLY, 0664)))
-		throw failed_constructor();
-
 	memset(&sf_info, 0 , sizeof(sf_info));
-	in.reset( sf_open_fd(fd, SFM_READ, &sf_info, true), sf_close);
+	in.reset( sf_open(path.c_str(), SFM_READ, &sf_info), sf_close);
 	if (!in) throw failed_constructor();
 
 	SF_BROADCAST_INFO binfo;
diff --git a/libs/ardour/sndfilesource.cc b/libs/ardour/sndfilesource.cc
index ca2c41d582..5acfe7b119 100644
--- a/libs/ardour/sndfilesource.cc
+++ b/libs/ardour/sndfilesource.cc
@@ -28,8 +28,6 @@
 
 #include <sys/stat.h>
 
-#include <glib/gstdio.h>
-
 #ifdef PLATFORM_WINDOWS
 #include <glibmm/convert.h>
 #endif
@@ -41,8 +39,6 @@
 #include "ardour/utils.h"
 #include "ardour/session.h"
 
-#include <fcntl.h>
-
 #include "i18n.h"
 
 using namespace std;
@@ -925,17 +921,11 @@ SndFileSource::get_soundfile_info (const string& path, SoundFileInfo& info, stri
 	SNDFILE *sf;
 	SF_INFO sf_info;
 	BroadcastInfo binfo;
-	char errbuf[1024];
-	int  fd;
 
 	sf_info.format = 0; // libsndfile says to clear this before sf_open().
 
-	if ((-1) == (fd = g_open (path.c_str(), O_RDONLY, 0664))) {
-		sprintf (errbuf, "SndFileSource::get_soundfile_info - cannot open file \"%s\"", path.c_str()); 
-		return false;
-	}
-
-	if ((sf = sf_open_fd (fd, SFM_READ, &sf_info, true)) == 0) { 
+	if ((sf = sf_open (const_cast<char*>(path.c_str()), SFM_READ, &sf_info)) == 0) {
+		char errbuf[256];
 		error_msg = sf_error_str (0, errbuf, sizeof (errbuf) - 1);
 		return false;
 	}
-- 
2.30.2