[tracker/tracker-0.12] libtracker-common, tracker-extract: Change the internal tracker_file_open API



commit 54c03b0fa55cb687a6ee3ca456573ebdca4ed09b
Author: Philip Van Hoof <philip codeminded be>
Date:   Wed Sep 21 17:35:48 2011 +0200

    libtracker-common, tracker-extract: Change the internal tracker_file_open API

 src/libtracker-common/tracker-file-utils.c   |   14 ++++----------
 src/libtracker-common/tracker-file-utils.h   |    4 +---
 src/tracker-extract/tracker-extract-jpeg.c   |    2 +-
 src/tracker-extract/tracker-extract-png.c    |    2 +-
 src/tracker-extract/tracker-extract-ps.c     |    2 +-
 src/tracker-extract/tracker-extract-vorbis.c |    2 +-
 src/tracker-extract/tracker-gsf.c            |    2 +-
 7 files changed, 10 insertions(+), 18 deletions(-)
---
diff --git a/src/libtracker-common/tracker-file-utils.c b/src/libtracker-common/tracker-file-utils.c
index e6a0280..f60e624 100644
--- a/src/libtracker-common/tracker-file-utils.c
+++ b/src/libtracker-common/tracker-file-utils.c
@@ -51,30 +51,24 @@
 static GHashTable *file_locks = NULL;
 
 FILE *
-tracker_file_open (const gchar *path,
-                   const gchar *how,
-                   gboolean     sequential)
+tracker_file_open (const gchar *path)
 {
 	FILE *file;
-	gboolean readonly;
 	int fd;
 
 	g_return_val_if_fail (path != NULL, NULL);
-	g_return_val_if_fail (how != NULL, NULL);
-
-	readonly = !strstr (how, "r+") && strchr (how, 'r');
 
 #if defined(__linux__)
-	fd = g_open (path, (readonly ? O_RDONLY : O_RDWR) | O_NOATIME);
+	fd = g_open (path, O_RDONLY | O_NOATIME);
 #else
-	fd = g_open (path, readonly ? O_RDONLY : O_RDWR);
+	fd = g_open (path, O_RDONLY);
 #endif
 
 	if (fd == -1) {
 		return NULL;
 	}
 
-	file = fdopen (fd, how);
+	file = fdopen (fd, "r");
 
 	if (!file) {
 		return NULL;
diff --git a/src/libtracker-common/tracker-file-utils.h b/src/libtracker-common/tracker-file-utils.h
index 6c5be1e..d3a8bc8 100644
--- a/src/libtracker-common/tracker-file-utils.h
+++ b/src/libtracker-common/tracker-file-utils.h
@@ -32,9 +32,7 @@ G_BEGIN_DECLS
 #endif
 
 /* File utils */
-FILE*    tracker_file_open          (const gchar  *path,
-                                     const gchar  *how,
-                                     gboolean      sequential);
+FILE*    tracker_file_open          (const gchar  *path);
 void     tracker_file_close         (FILE         *file,
                                      gboolean      need_again_soon);
 goffset  tracker_file_get_size      (const gchar  *path);
diff --git a/src/tracker-extract/tracker-extract-jpeg.c b/src/tracker-extract/tracker-extract-jpeg.c
index 6a2f1bd..f0be441 100644
--- a/src/tracker-extract/tracker-extract-jpeg.c
+++ b/src/tracker-extract/tracker-extract-jpeg.c
@@ -175,7 +175,7 @@ tracker_extract_get_metadata (TrackerExtractInfo *info)
 		return FALSE;
 	}
 
-	f = tracker_file_open (filename, "rb", FALSE);
+	f = tracker_file_open (filename);
 	g_free (filename);
 
 	if (!f) {
diff --git a/src/tracker-extract/tracker-extract-png.c b/src/tracker-extract/tracker-extract-png.c
index 4091166..e424353 100644
--- a/src/tracker-extract/tracker-extract-png.c
+++ b/src/tracker-extract/tracker-extract-png.c
@@ -800,7 +800,7 @@ tracker_extract_get_metadata (TrackerExtractInfo *info)
 		return FALSE;
 	}
 
-	f = tracker_file_open (filename, "r", FALSE);
+	f = tracker_file_open (filename);
 	g_free (filename);
 
 	if (!f) {
diff --git a/src/tracker-extract/tracker-extract-ps.c b/src/tracker-extract/tracker-extract-ps.c
index f86f96c..f2eb2c5 100644
--- a/src/tracker-extract/tracker-extract-ps.c
+++ b/src/tracker-extract/tracker-extract-ps.c
@@ -205,7 +205,7 @@ extract_ps (const gchar          *uri,
 	gchar *filename;
 
 	filename = g_filename_from_uri (uri, NULL, NULL);
-	f = tracker_file_open (filename, "r", TRUE);
+	f = tracker_file_open (filename);
 	g_free (filename);
 
 	if (!f) {
diff --git a/src/tracker-extract/tracker-extract-vorbis.c b/src/tracker-extract/tracker-extract-vorbis.c
index 2de4bcb..d5b37b4 100644
--- a/src/tracker-extract/tracker-extract-vorbis.c
+++ b/src/tracker-extract/tracker-extract-vorbis.c
@@ -103,7 +103,7 @@ tracker_extract_get_metadata (TrackerExtractInfo *info)
 
 	file = tracker_extract_info_get_file (info);
 	filename = g_file_get_path (file);
-	f = tracker_file_open (filename, "r", FALSE);
+	f = tracker_file_open (filename);
 	g_free (filename);
 
 	preupdate = tracker_extract_info_get_preupdate_builder (info);
diff --git a/src/tracker-extract/tracker-gsf.c b/src/tracker-extract/tracker-gsf.c
index a715d6c..b33aeba 100644
--- a/src/tracker-extract/tracker-gsf.c
+++ b/src/tracker-extract/tracker-gsf.c
@@ -102,7 +102,7 @@ tracker_gsf_parse_xml_in_zip (const gchar          *zip_file_uri,
 		           zip_file_uri, error ? error->message : "no error given");
 	} else { /* Create a new Input GSF object for the given file */
 
-		file = tracker_file_open (filename, "rb", FALSE);
+		file = tracker_file_open (filename);
 		if (!file) {
 			g_warning ("Can't open file from uri '%s': %s",
 			           zip_file_uri, g_strerror (errno));



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]