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.

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

Clean up the codebase #12

Open
4 of 12 tasks
Tracked by #15
Alfredsson418 opened this issue Nov 12, 2024 · 0 comments
Open
4 of 12 tasks
Tracked by #15

Clean up the codebase #12

Alfredsson418 opened this issue Nov 12, 2024 · 0 comments
Assignees
Labels
ongoing development Tag for ongoing development
Milestone

Comments

@Alfredsson418
Copy link
Owner

Alfredsson418 commented Nov 12, 2024

  • Code cleanup
    • Remove unnecessary print statement (possible change the macro to allow no "..." arguments)
    • Structure the code better
    • No unnecessary "magic numbers" (Constants for important values)
    • No unnecessary memory allocation (pass string as an argument instead of return statement)
    • Restructure datatypes (unsigned int for only positive values or short for smaller numbers e.g short for ports)
  • Check for memory leaks
  • More redundancy
    • Correct error handling
    • Different error codes for different errors
    • Check input before proceeding
  • Check spelling
@Alfredsson418 Alfredsson418 self-assigned this Nov 12, 2024
@Alfredsson418 Alfredsson418 mentioned this issue Nov 12, 2024
6 tasks
@Alfredsson418 Alfredsson418 added the ongoing development Tag for ongoing development label Nov 18, 2024
@Alfredsson418 Alfredsson418 added this to the First Release milestone Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ongoing development Tag for ongoing development
Projects
None yet
Development

No branches or pull requests

1 participant