Logging mit __FILE__

[rhabarber1848]

Hi,

macht folgender Patch, und dessen sinngemäße globale Anwendung, Sinn?

Code: Alles auswählen

--- apps/tuxbox/neutrino/src/neutrino.cpp	2009-10-03 19:53:40.000000000 +0200
+++ apps/tuxbox/neutrino/src/neutrino.cpp	2009-10-03 21:28:20.000000000 +0200
@@ -171,10 +172,10 @@
 		if (false == blocking)
 			command += " &";
 		if (verbose)
-			printf("[neutrino] executing %s\n", command.c_str());
+			printf("[%s] executing %s\n", __FILE__, command.c_str());
 		int result = system(command.c_str());
 		if (result !=0 ) {
-			printf("[neutrino] %s failed with return code = %d...\n", filename, WEXITSTATUS(result));
+			printf("[%s] %s failed with return code = %d...\n", __FILE__, filename, WEXITSTATUS(result));
 			return false;
 		}
 	}

[dbt]

Wenns mit dem Makro funktioniert, schon.

[trulli]

Wenns mit dem Makro funktioniert, schon.

Hi,

wieder so eine sinnlose Antwort. Wenn es funktioniert ..... dann testet es.

Und ich sage Euch .... lasst es ......es funktioniert nicht

MfG

[seife]

Weiss ich nicht. Den Benutzer interessiert ja eher, welches Programm die Meldung verursacht, nicht welches Sourcefile.

Bei Debugmeldungen ist das was anderes, aber dort hängt es vom Debuggenden ab, was er wissen will (ich mach dann oft "fprintf(stderr, "%s:%d ...\n", __FILE__, __LINE__);", stderr deswegen, weil das ungepuffert ist).

[rhabarber1848]

welches Programm die Meldung verursacht, nicht welches Sourcefile.

Im Fall von neutrino.cpp dürfte es dasselbe sein, oder?

grep "\[neutrino\]" apps/tuxbox/neutrino/src/neutrino.cpp | wc -l
13

Diese Codezeilen könnten __FILE__ vertragen, oder nicht?

Anderswo könnte es so aussehen:

Code: Alles auswählen

--- apps/tuxbox/neutrino/src/gui/audioplayer.cpp 2009-10-03 12:51:59.000000000 +0200
+++ apps/tuxbox/neutrino/src/gui/audioplayer.cpp      2009-10-03 23:30:21.000000000 +0200
@@ -323,7 +323,7 @@
        irs.Send();
 #endif

-       puts("[audioplayer.cpp] executing " AUDIOPLAYER_START_SCRIPT ".");
+       printf("[%s] executing %s\n", __FILE__, AUDIOPLAYER_START_SCRIPT);
        if (system(AUDIOPLAYER_START_SCRIPT) != 0)
                perror("Datei " AUDIOPLAYER_START_SCRIPT " fehlt.Bitte erstellen, wenn gebraucht.\nFile " AUDIOPLAYER_START_SCRIPT " not found. Please create if needed.\n");