chore!: comment out unused enums and structs in lib.rs

[nyurik]

For some reason these structs and enums were never used. Should I delete them instead, or is there a plan to use them for something?

• SignalAttributeValue
• AttributeValueType - also renamed values
• AttrDefault

[Copilot]

Pull Request Overview

This PR comments out three unused data structures in the main library file to clean up the codebase. The maintainer is questioning whether these should be deleted entirely or kept for future use.

• Comments out unused enum and struct definitions with FIXME notes
• Renames enum variant names for consistency during the commenting process
• Adds explanatory comments about the unused status

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[tegimeki]

It might be better to get #22 up to date once the other PRs land; it adds some of the unused features commented-out here and could be built upon for remaining items such as BA_DEF_DEF_REL_ etc.

[nyurik]

It might be better to get #22 up to date once the other PRs land; it adds some of the unused features commented-out here and could be built upon for remaining items such as BA_DEF_DEF_REL_ etc.

Agree, it might be a better overall approach, esp when we switch to the new pest lexer from https://github.com/oxibus/can-dbc-pest