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

Refactor MessageState to Use Msg Struct and Simplify Initialization #1071

Closed
wants to merge 2 commits into from

Conversation

deanlee
Copy link
Contributor

@deanlee deanlee commented Aug 1, 2024

Refactors the MessageState class to improve its structure and initialization process:

  1. Replaced multiple member variables with a single struct Msg to encapsulate message-related data. These member variables were essentially duplicates of the struct Msg's members.
  2. Added a constructor MessageState::MessageState to simplify the initialization process.
  3. Updated the code to use the new struct and constructor for better readability and maintainability.

This refactor enhances code clarity, reduces redundancy, and streamlines the initialization process.

@deanlee deanlee force-pushed the refactor_parser_ctor branch 2 times, most recently from a5ce472 to 58c43ea Compare August 1, 2024 16:34
@deanlee deanlee marked this pull request as draft August 1, 2024 16:36
@deanlee deanlee force-pushed the refactor_parser_ctor branch 2 times, most recently from 4e23ce1 to 74a4a4f Compare August 1, 2024 16:40
@deanlee deanlee marked this pull request as ready for review August 1, 2024 16:42
@deanlee deanlee force-pushed the refactor_parser_ctor branch from 74a4a4f to 7eea729 Compare August 8, 2024 19:21
@deanlee deanlee force-pushed the refactor_parser_ctor branch from 7eea729 to 808bc49 Compare August 20, 2024 13:20
Copy link
Contributor

github-actions bot commented Aug 20, 2024

Thanks for contributing to opendbc! In order for us to review your PR as quickly as possible, check the following:

  • Convert your PR to a draft unless it's ready to review
  • Read the contributing docs
  • Before marking as "ready for review", ensure:
    • the goal is clearly stated in the description
    • all the tests are passing
    • include a route or your device' dongle ID if relevant

@github-actions github-actions bot added the can related to CAN tools, aka opendbc/can/ label Aug 20, 2024
@deanlee deanlee force-pushed the refactor_parser_ctor branch from 808bc49 to c1c586b Compare August 20, 2024 13:26
@deanlee deanlee force-pushed the refactor_parser_ctor branch from c1c586b to 347b297 Compare September 30, 2024 00:53
@deanlee deanlee force-pushed the refactor_parser_ctor branch from 347b297 to d073dec Compare October 15, 2024 15:09
@deanlee deanlee force-pushed the refactor_parser_ctor branch from d073dec to e87ed84 Compare October 16, 2024 14:44
@deanlee deanlee closed this Nov 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
can related to CAN tools, aka opendbc/can/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant