Bug #112
openGCC 4.4 fix
0%
Description
Der mg_tools.h fehlt ein #include <cstdarg>. Sonst baut es nicht.
Files
Updated by Heffer over 15 years ago
- File vdr-muggle-0.2.3-gcc44.patch vdr-muggle-0.2.3-gcc44.patch added
- File build.log build.log added
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.
Updated by wrohdewald over 15 years ago
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.
Updated by wrohdewald over 15 years ago
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.