Skip to content

Debug trait with meaningful output #485

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

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Debug trait with meaningful output #485

wants to merge 1 commit into from

Conversation

Marcel583
Copy link
Member

@Marcel583 Marcel583 commented Mar 19, 2025

Description

This PR implements the Debug trait for all types by using our to_string method, so users do not get pointers as a debug message.

Testing

  • I added basic working examples (possibly in doc-comment)
  • I added tests for large (pointer representation) values
  • I triggered all possible errors in my test in every possible way
  • I included tests for all reasonable edge cases

Checklist:

  • I have performed a self-review of my own code
    • The code provides good readability and maintainability s.t. it fulfills best practices like talking code, modularity, ...
      • The chosen implementation is not more complex than it has to be
    • My code should work as intended and no side effects occur (e.g. memory leaks)
    • The doc comments fit our style guide
    • I have credited related sources if needed

@Marcel583 Marcel583 added the enhancement📈 New feature label Mar 19, 2025
@Marcel583 Marcel583 self-assigned this Mar 19, 2025
@Marcel583 Marcel583 marked this pull request as ready for review March 19, 2025 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement📈 New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant