[gcompris-devel] Segmentation fault CVS

[Bruno Coudoin]

It makes sense. There is probably other places like this where the same
should apply.
It will be part of the windows port task list.

Bruno.

Le mar 27/01/2004 à 14:40, Olivier Samyn a écrit :
> Hello,
> 
> Just a five cents note:
> why not using the glib G_DIR_SEPARATOR in path ? this way an eventual 
> windows port will be simpler... :)
> 
> so the line soundfile=... should be:
> soundfile = g_strdup_printf(
>     "%s%c%s%c%s",
>     PACKAGE_DATA_DIR,
>     G_DIR_SEPARATOR,
>     "sounds",
>     G_DIR_SEPARATOR,
>     board->mandatory_sound_file
> );
> 
> 
> Olivier ;)
> 
> >Index: menu.c
> >===================================================================
> >RCS file: /cvs/gnome/gcompris/src/boards/menu.c,v
> >retrieving revision 1.38
> >diff -u -r1.38 menu.c
> >--- menu.c	10 Dec 2003 22:14:57 -0000	1.38
> >+++ menu.c	27 Jan 2004 10:20:50 -0000
> >@@ -351,7 +351,7 @@
> > 					      board->mandatory_sound_file);
> >       } else {
> > 	/* We search a fixed path sound file */
> >-	soundfile = g_strdup_printf("%s/%s/%s", PACKAGE_DATA_DIR "/sounds", 
> >+	soundfile = g_strdup_printf("%s/%s/%s", PACKAGE_DATA_DIR, "/sounds", 
> > 				    board->mandatory_sound_file);
> > 	g_warning("Checking mandatory_sound_file %s\n", soundfile);
> >       }
> >  
> >
> 
> 
> -------------------------------------------------------
> The SF.Net email is sponsored by EclipseCon 2004
> Premiere Conference on Open Tools Development and Integration
> See the breadth of Eclipse activity. February 3-5 in Anaheim, CA.
> http://www.eclipsecon.org/osdn
> _______________________________________________
> gcompris-devel mailing list
> gcompris-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/gcompris-devel
>