Bug #198
geschlossenDebug-Ausgabe des Parsers beruhigen
Beschreibung
Gwenhywfar : 5.2.0.0
Nur so ein Vorschlag, das
fprintf(stderr
auf
DBG_DEBUG(GWEN_LOGDOMAIN
umzustellen.
Falls der Patch nicht gefällt, bitte Ticket einfach wieder schließen.
Dateien
Von martin vor mehr als 5 Jahren aktualisiert
- Status wurde von New zu Rejected geändert
Hierbei handelt es sich um eine Debug-Funktion, die absichtilich auf den Error-Kanal ausgibt.
Wenn man das hier mit DBG_DEBUG() macht, wird in der Regel gar nichts ausgegeben, weil insbesondere DBG_DEBUG meist deaktiviert ist.
Diese Ausgabe sollte man also an der Quelle auskommentieren, falls sie noch irgendwo vergessen worden ist. Ansonsten sollte sie aber genauso ausgegeben werden, wie das bisher der Fall ist. Insbesondere auch, weil das alle anderen *_Dump()-Funktionen auch genauso machen, und ein hier abweichendes Verhalten doch eher ueberraschen wuerde.
Von Berny vor mehr als 5 Jahren aktualisiert
- Datei jobgettransactions-dont-dump.patch jobgettransactions-dont-dump.patch wurde hinzugefügt
- Tracker wurde von Feature zu Bug geändert
Ok, hab's kapiert, insbesondere nachdem ich die Verwendung von GWEN_DB_Dump() in diversen Fehlerpfaden gesehen habe.
Aktuell sehe ich einen Dump beim Abruf von Kreditkartenumsätzen (ich weiß, das hatte ich bisher verschwiegen). Der neue Patch kommentiert nun "an der Quelle" aus.