Replies: 3 comments
-
Prinzipiell bin ich dafür. Generell würde ich das gern weniger macrobasiert haben, sondern eher sowas wie: Ich denke vordefinierte Bereiche werden schnell wieder unflexibel und genauer als pro Klasse muss man vermutlich nicht loggen. In der Logger-Klasse kann man dann mit macro die ganzen nicht benötigten Methodenaufrufe zu NOOPs machen. Die sollte der Compiler dann eigentlich automatisch rausoptimieren, so dass die Strings auch nicht im Binary drin sind. Evtl. könnte man das sogar nutzen um zur Laufzeit das Logging zu konfigurieren und an eine bestimmte GA zu loggen ;) |
Beta Was this translation helpful? Give feedback.
-
Um ehrlich zu sein habe ich da schon einiges an Aufwand spendiert und eine Lösung erarbeitet uind auch getestet die aus meiner Sicht der aktuellen Lösungen in vielerlei Hinsicht überlegen ist. Es gibt immer viele Wege die nach Rom führen, das ist meiner. Ich werde das nicht nochmal komplett überarbeiten, dazu fehlt mir die Zeit und die Motivation. Über Anpassungen können wir gerne reden, und wenn du konkrete Probleme bei meiner Lösung siehst auch. Aber nicht über linksrum oder rechtsrum. Insofern - ist das im Grundsatz so aktzeptabel, oder nicht? |
Beta Was this translation helpful? Give feedback.
-
Wenn du über konkreten Code reden willst, mach einfach einen PR, wenn noch nicht fertig ist einen Draft PR dann kann ich mir den Code anschauen und kommentieren. |
Beta Was this translation helpful? Give feedback.
-
Hi Thomas,
ich hatte ja eine Überarbeitung des Logging-Konzepts im Stack angekündigt.
Bevor ich jetzt da aber viel Zeit in das Fine-Tuning und dann die Umstellung stecke, würde ich gern kurz deine Meinung dazu haben..
Meine Ziele:
Damit kann der Benutzer durch eine Einstellung im Programm auswählen welche Meldungen er sehen will, zB.
...
Hier ist ein (noch nicht fertiger) Entwurf:
https://github.com/OpenKNX/knx/tree/draft/advanced_logging
Beta Was this translation helpful? Give feedback.
All reactions