Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Centralize the #ifdef DEBUG_ ... #endif constructs #10

Open
eyalroz opened this issue Aug 7, 2020 · 0 comments
Open

Centralize the #ifdef DEBUG_ ... #endif constructs #10

eyalroz opened this issue Aug 7, 2020 · 0 comments
Assignees
Labels

Comments

@eyalroz
Copy link
Owner

eyalroz commented Aug 7, 2020

Instead of surrounding every debug-logging with preprocessor directives, we could centralize them in one place - a wrapper for console.log() within the RemoveDupes object - and have that wrapper be aware of which DEBUG_XYZ values were set on compilation. Specifically, those could be strings in a set / entries in a dictionary, which would be looked up before actually calling console log; and the wrapper would be invoked with the relevant string.

@eyalroz eyalroz self-assigned this Aug 7, 2020
@eyalroz eyalroz added the task label Aug 7, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant