Bug #112

GCC 4.4 fix

Added by Heffer 487 days ago. Updated 369 days ago.

Status:New Start:03/31/2009
Priority:Normal Due date:
Assigned to:- % Done:

0%

Category:- Spent time: -
Target version:-

Description

Der mg_tools.h fehlt ein #include <cstdarg>. Sonst baut es nicht.

vdr-muggle-0.2.3-gcc44.patch - Patch (2 KB) Heffer, 03/31/2009 03:55 PM

build.log - Buildlog (14 KB) Heffer, 03/31/2009 03:55 PM

History

03/31/2009 03:55 PM - Heffer

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.

03/31/2009 04:23 PM - wrohdewald

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.

07/27/2009 11:36 PM - wrohdewald

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.

Also available in: Atom PDF