Added by Heffer over 15 years ago. Updated over 15 years ago.
0%
Description
Der mg_tools.h fehlt ein #include <cstdarg>. Sonst baut es nicht.
Files
| vdr-muggle-0.2.3-gcc44.patch (2.03 KB) vdr-muggle-0.2.3-gcc44.patch | Patch | Heffer, 03/31/2009 03:55 PM | |
| build.log (14 KB) build.log | Buildlog | Heffer, 03/31/2009 03:55 PM |
Hmm... das war nicht wirklich alles. Hier ein kleiner Patch. Beachtet jedoch, dass ich so ziemlich 0 kenntnisse in C habe.
Anbei auch ein Buildlog.
gcc4.4 habe ich noch nicht - debian unstable hat erst gcc4.3. Ich möchte muggle eigentlich erst ändern, wenn ich auch gcc4.4 habe, um das dann auch testen zu können. Das sind ja auch alles nur Warnungen, also erstmal nicht so schlimm.
Zum cstdarg in mg_tools.h finde ich in Deinem build.log keine Fehlermeldung, wie lautet sie?
Das mit dem strdup im Patch auf Zeile 1143 von mg_db.c ist aber wohl falsch - wenn das wirklich nur mit strdup zu lösen ist, dann eher so:
char *tcon=strdup(genre1.c_str())
...
free(tcon)
und mg_tools.c, Zeile 173: Auch hier: was mit strdup alloziiert wird, muss später mit free() freigegeben werden. Man sollte wohl eher aus char *extension ein const char *extension machen. Das hat natürlich eine ganze Reihe weiterer Änderungen zur Folge, ist aber sauberer.
die anderen Patche sehe ich mir an, sobald ich auch gcc4.4 habe.
nun habe ich gcc 4.4 installiert und das warnungsfrei kompilierbar gemacht. Nur - warum hast mg_db.c, mg_image_provider.c und mg_tools.c geändert? Da bekomme ich gar keine Warnungen.