Fix gcc warnings and infinite loop.

(Old svn revision: 5001)

ChangeLog

+2008-06-27 jeromeg
+
+	* src/Makefile.am: enable -Wall
+	* src/{main.c, screenshooter-utils.{c,h}}: fix gcc warnings except one...
+	* BUGFIX: fix infinite loop in generate_filename_for_uri !
+
 2008-06-27 jeromeg
 
 	* src/screenshooter-utils.c: clean generate_filename_for_uri using g_access.

src/Makefile.am

@@ -4,7 +4,7 @@ bin_PROGRAMS = xfce4-screenshooter
 
 xfce4_screenshooter_plugin_CFLAGS =						\
 	-DPACKAGE_LOCALE_DIR=\"$(localedir)\"				\
-	@LIBXFCE4PANEL_CFLAGS@
+	@LIBXFCE4PANEL_CFLAGS@ -Wall
 
 xfce4_screenshooter_plugin_LDFLAGS =						\
 	@LIBXFCE4PANEL_LIBS@
@@ -17,7 +17,7 @@ xfce4_screenshooter_CFLAGS = \
 	@GTK_CFLAGS@ \
 	@GLIB_CFLAGS@ \
 	@LIBXFCE4UTIL_CFLAGS@ \
-	-DPACKAGE_LOCALE_DIR=\"$(localedir)\"
+	-DPACKAGE_LOCALE_DIR=\"$(localedir)\" -Wall
 	
 xfce4_screenshooter_LDFLAGS =                       \
 	@GTK_LIBS@ \

src/main.c

@@ -103,7 +103,7 @@ int main(int argc, char **argv)
   gtk_dialog_set_default_response (GTK_DIALOG (chooser), GTK_RESPONSE_ACCEPT);
   gtk_file_chooser_set_current_folder( GTK_FILE_CHOOSER ( chooser ), xfce_get_homedir () );  
   
-  filename = generate_filename_for_uri ( xfce_get_homedir () );
+  filename = generate_filename_for_uri ( g_strdup ( xfce_get_homedir () ) );
   preview = gtk_image_new ();
   
   gtk_file_chooser_set_current_name (GTK_FILE_CHOOSER (chooser), filename);
@@ -123,5 +123,6 @@ int main(int argc, char **argv)
   
   gtk_widget_destroy(chooser);
   g_free(filename);
+  
+  return 0;
 }
-

src/screenshooter-utils.c

@@ -149,7 +149,6 @@ GdkPixbuf *take_screenshot (gint fullscreen, gint delay)
 
 gchar *generate_filename_for_uri(char *uri)
 {
-  int test;
   gchar *file_name;
   unsigned int i = 0;
     
@@ -160,7 +159,7 @@ gchar *generate_filename_for_uri(char *uri)
   
   file_name = g_strdup ("Screenshot.png");
     
-  if( ( test = g_access ( g_build_filename (uri, file_name, NULL), F_OK ) ) != 0 ) 
+  if( g_access ( g_build_filename (uri, file_name, NULL), F_OK ) != 0 ) 
   {
     return file_name;
   }
@@ -171,7 +170,7 @@ gchar *generate_filename_for_uri(char *uri)
     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 );
+  while( g_access ( g_build_filename (uri, file_name, NULL), F_OK ) == 0 );
     
   return file_name;
 }

src/screenshooter-utils.h

@@ -31,6 +31,7 @@
 
 #include <gtk/gtk.h>
 #include <gdk/gdkx.h>
+#include <glib/gstdio.h>
 
 #include <fcntl.h>
 #include <X11/Xatom.h>