Clean generate_filename_for_uri using g_access

(Old svn revision: 4999)

ChangeLog

+2008-06-27 jeromeg
+
+	* src/screenshooter-utils.c: clean generate_filename_for_uri using g_access.
+
 2008-06-20 09:19 jeromeg
 
 	* NEWS: updated.

po/POTFILES.in

 src/screenshooter-plugin.c
 src/main.c
 src/screenshooter.desktop.in.in
-

src/screenshooter-utils.c

@@ -27,8 +27,6 @@
 
 #include <screenshooter-utils.h>
 
-#define MODE 0644
-
 /* Prototypes */
 
 static Window get_window_property (Window xwindow, Atom atom);
@@ -151,22 +149,29 @@ GdkPixbuf *take_screenshot (gint fullscreen, gint delay)
 
 gchar *generate_filename_for_uri(char *uri)
 {
-    int test;
-    gchar *file_name;
-    unsigned int i = 0;
-    if(uri == NULL)
-        return NULL;
-    file_name = g_strdup ("Screenshot.png");
-    if((test=open(g_build_filename(uri, file_name, NULL),O_RDWR,MODE))==-1) 
-    {
-        return file_name;
-    }
-    do
-    {
-        i++;
-        g_free (file_name);
-        file_name = g_strdup_printf ("Screenshot-%d.png",i);
-    }
-    while((test=open(g_build_filename(uri, file_name, NULL),O_RDWR,MODE))!=-1);
+  int test;
+  gchar *file_name;
+  unsigned int i = 0;
+    
+  if ( uri == NULL )
+  {
+  	return NULL;
+  }      
+  
+  file_name = g_strdup ("Screenshot.png");
+    
+  if( ( test = g_access ( g_build_filename (uri, file_name, NULL), F_OK ) ) != 0 ) 
+  {
     return file_name;
+  }
+    
+  do
+  {
+    i++;
+    g_free (file_name);
+    file_name = g_strdup_printf ("Screenshot-%d.png", i);
+  }
+  while( ( test = g_access ( g_build_filename (uri, file_name, NULL), F_OK ) ) == -1 );
+    
+  return file_name;
 }