In Bezug auf die defaults der ersetzten Settings wurde ja drauf hingewiesen, dass es schöner wäre, die Flags aus der settings.h zu nehmen. Da bin ich völlig d’accord!
Eine Frage des Geschmacks (ich bin eher Web-Entwickler und hab mit C++ wenig zu tun) ergibt sich für mich da aber grade: Sollte ich die
#ifndef SOME_FLAG
#define SOME_FLAG false
#endif
Routinen lieber im jeweiligen Codeschnippsel einfügen, wo das jeweilige SOME_FLAG dann verwendet wird einfügen, oder eine einheitliche Definitionsstelle suchen. Damit das einheitlich und ein einem Aufwaschen passiert.
Konkret steh ich schon beim PLAY_MONO vor der Entscheidung, es sonst an vier Stellen einfügen zu müssen, siehe Commit - was ich nicht so schön fände.
Ich kann es schwer einschätzen, wo so ein allgemeiner #ifndef-Block gut hinpassen würde - sowohl vom Design, als auch von der Code-Execution her… settings.h fällt ja schon mal flach, das würd beim Checkout alle individuell getroffenen Einstellungen zerschießen.
Meine Vermutung wäre mal main.cpp oder main.h gewesen? Zweiteres wäre auch noch recht leer, da würd es nicht so viel verschandeln.
Thoughts on that?
Ich mach derweil mal weiter bei den anderen Flags 