remove all use of access(2) (this was done a bit stupidly, we could have used g_access)

git-svn-id: svn://localhost/ardour2/branches/3.0@9340 d708f5d6-7413-0410-9779-e7cbd77b26cf

3 files changed, 5 insertions, 15 deletions

diff --git a/libs/ardour/audiosource.cc b/libs/ardour/audiosource.cc
index b993a093b5..ac98837183 100644
--- a/libs/ardour/audiosource.cc
+++ b/libs/ardour/audiosource.cc
@@ -197,7 +197,7 @@ AudioSource::rename_peakfile (string newpath)
 
 	string oldpath = peakpath;
 
-	if (access (oldpath.c_str(), F_OK) == 0) {
+	if (Glib::file_test (oldpath, Glib::FILE_TEST_EXISTS)) {
 		if (rename (oldpath.c_str(), newpath.c_str()) != 0) {
 			error << string_compose (_("cannot rename peakfile for %1 from %2 to %3 (%4)"), _name, oldpath, newpath, strerror (errno)) << endmsg;
 			return -1;
diff --git a/libs/ardour/file_source.cc b/libs/ardour/file_source.cc
index 5522031d69..e6990aa487 100644
--- a/libs/ardour/file_source.cc
+++ b/libs/ardour/file_source.cc
@@ -191,7 +191,7 @@ FileSource::move_to_trash (const string& trash_dir_name)
 		snprintf (buf, sizeof (buf), "%s.%d", newpath.c_str(), version);
 		newpath_v = buf;
 
-		while (access (newpath_v.c_str(), F_OK) == 0 && version < 999) {
+		while (Glib::file_test (newpath_v, Glib::FILE_TEST_EXISTS) && version < 999) {
 			snprintf (buf, sizeof (buf), "%s.%d", newpath.c_str(), ++version);
 			newpath_v = buf;
 		}
diff --git a/libs/ardour/session_state.cc b/libs/ardour/session_state.cc
index b5b713cbd4..616f19c50a 100644
--- a/libs/ardour/session_state.cc
+++ b/libs/ardour/session_state.cc
@@ -130,6 +130,7 @@ using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
 
+
 void
 Session::first_stage_init (string fullpath, string snapshot_name)
 {
@@ -151,11 +152,7 @@ Session::first_stage_init (string fullpath, string snapshot_name)
 		_path += G_DIR_SEPARATOR;
 	}
 
-	if (Glib::file_test (_path, Glib::FILE_TEST_EXISTS) && ::access (_path.c_str(), W_OK)) {
-		_writable = false;
-	} else {
-		_writable = true;
-	}
+	_writable = exists_and_writable (sys::path (_path));
 
 	/* these two are just provisional settings. set_state()
 	   will likely override them.
@@ -898,14 +895,7 @@ Session::load_state (string snapshot_name)
 
 	set_dirty();
 
-	/* writable() really reflects the whole folder, but if for any
-	   reason the session state file can't be written to, still
-	   make us unwritable.
-	*/
-
-	if (::access (xmlpath.to_string().c_str(), W_OK) != 0) {
-		_writable = false;
-	}
+	_writable = exists_and_writable (xmlpath);
 
 	if (!state_tree->read (xmlpath.to_string())) {
 		error << string_compose(_("Could not understand ardour file %1"), xmlpath.to_string()) << endmsg;