Skip to content
This repository has been archived by the owner on Jan 9, 2024. It is now read-only.

Remove common header for constants #205

Open
canepat opened this issue Dec 28, 2021 · 0 comments
Open

Remove common header for constants #205

canepat opened this issue Dec 28, 2021 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@canepat
Copy link
Member

canepat commented Dec 28, 2021

Remove common header for constants and move each of them to an appropriate class or module

Having one common header for constants increases code coupling and compilation time when one constant is added/changed/removed

A separate header just for command-line default options should be considered, anyway

@canepat canepat added enhancement New feature or request good first issue Good for newcomers labels Dec 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant