Project

General

Profile

Bug #112

GCC 4.4 fix

Added by Heffer over 10 years ago. Updated over 10 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
03/31/2009
Due date:
% Done:

0%


Description

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

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

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

History

#1 Updated by Heffer over 10 years ago

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.

#2 Updated by wrohdewald over 10 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.

#3 Updated by wrohdewald over 10 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.

Also available in: Atom PDF