Bug #198
closedDebug-Ausgabe des Parsers beruhigen
Description
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.
Files
Updated by martin about 6 years ago
- Status changed from New to Rejected
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.
Updated by Berny about 6 years ago
- File jobgettransactions-dont-dump.patch jobgettransactions-dont-dump.patch added
- Tracker changed from Feature to Bug
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.