<div dir="ltr">Hello world !<br><br>I wish to work with Tux4kids during GSoC 2013, maybe on a TuxType project, so I navigated through its source code.<br><br>I tried to do some modifications ; I saw a patch someone did previous years ago on the language files (translation of "fifty" is "FÜNFZIG", not "FÜNZIG"), but it seems it has never been merged, here is the diff :<br>
<br>---<br> data/themes/deutsch/words/lang7-8-mit.txt   |    1 +<br> data/themes/deutsch/words/mittel4-6-mit.txt |    1 -<br> 2 files changed, 1 insertion(+), 1 deletion(-)<br><br>diff --git a/data/themes/deutsch/words/lang7-8-mit.txt b/data/themes/deutsch/words/lang7-8-mit.txt<br>
index 02ac150..9d9ec32 100644<br>--- a/data/themes/deutsch/words/lang7-8-mit.txt<br>+++ b/data/themes/deutsch/words/lang7-8-mit.txt<br>@@ -7,4 +7,5 @@ LÄCHELN<br> MÄDCHEN<br> SCHÜLER<br> ÜBERALL<br>+FÜNFZIG<br> FRÜHLING<br>
diff --git a/data/themes/deutsch/words/mittel4-6-mit.txt b/data/themes/deutsch/words/mittel4-6-mit.txt<br>index 72f66aa..d6b0eae 100644<br>--- a/data/themes/deutsch/words/mittel4-6-mit.txt<br>+++ b/data/themes/deutsch/words/mittel4-6-mit.txt<br>
@@ -32,7 +32,6 @@ SÜDEN<br> ZÜGEL<br> ZWÖLF<br> BRÜCKE<br>-FÜNZIG<br> HEIßEN<br> KÖRPER<br> LÖFFEL<br>-- <br><br><br><br>I also have several questions about some FIXME comments.<br><br>=> about macros in globals.h:<br>
// FIXME if we really need these, make them into functions rather than<br>// "evil" macros<br>#define to_upper(c) (((c) >= 'a' && (c) <= 'z') ? (c) -32 : (c))<br>#define COL2RGB( col ) SDL_MapRGB( screen->format, col->r, col->g, col->b )<br>
#define MIN(x,y) ((x) < (y) ? (x) : (y))<br>#define MAX(x,y) ((x) > (y) ? (x) : (y))<br><br>I checked the source code, and:<br>- to_upper() and MAX() are unused<br>- COL2RGB() is only used 2 times<br>- MIN() is only used 4 times, and is #define'd again in input_methods.c<br>
<br>Could I add a Color2RGB() function in SDL_extras.c/.h, and a min() function somewhere else and then submit the patch to you ?<br><br><br><br>=> about the exit() calls:<br>exit(0);/* FIXME may need to cleanup memory and exit more cleanly */<br>
exit(0); /* FIXME does memory get cleaned up properly if we do this? */<br><br>Modern operating systems free the memory once a program exited, if it didn't. But it's a feature of the OS and it seems freeing memory shouldn't rely on that ; and unless it's certain the program will run on that kind of OS, it's a bad habit to do that for reusability of the code, etc etc .... So I assume a FreeMemory() function could be created ?<br>
<br>Thanks for reading !<br><br>Regards,<br>Jerome<br></div>