Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Clean up binary templates #122

Open
exectails opened this issue Sep 9, 2017 · 1 comment
Open

Clean up binary templates #122

exectails opened this issue Sep 9, 2017 · 1 comment
Labels
enhancement This issue is about improving the quality of code or a feature. refactoring This issue relates to the refactoring of the code.

Comments

@exectails
Copy link
Member

The binary templates are a little chaotic atm, and it's only gonna get worse. We'll have to clean up in there some time.

  • Move every group of related structures from common.bt into a dedicated file in a sub folder "inc".
  • Introduce a general naming scheme for variables and structs.

This will make maintaining them and keeping track of the history much easier.
@ovflowd ovflowd added enhancement This issue is about improving the quality of code or a feature. refactoring This issue relates to the refactoring of the code. and removed backlog labels Aug 31, 2021
@ovflowd ovflowd added this to the Code Maintenance milestone Aug 31, 2021
@exectails
Copy link
Member Author

The structures have been moved to dedicated files for cleaner management, but the naming scheme is still all over the place.

@exectails exectails self-assigned this Oct 1, 2021
@exectails exectails removed their assignment May 17, 2023
@exectails exectails removed this from the Code Maintenance milestone May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement This issue is about improving the quality of code or a feature. refactoring This issue relates to the refactoring of the code.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@exectails @ovflowd