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

WIP (depends on #22) - Rename StreamReader -> IrStreamReader; Extract a StreamReader base class from IrStreamReader. #23

Closed
wants to merge 8 commits into from

Conversation

junhaoliao
Copy link
Collaborator

Description

  1. Rename ClpIrStreamReader to ClpIRStreamReader in bindings.
  2. Add get_version to decoding methods.
  3. Extract a StreamReader base class from IRStreamReader.
  4. Implement factory method StreamReader::create to perform version checking before creating derived classes.

Validation performed

TBA

@junhaoliao junhaoliao changed the title WIP (depends on #22) - Rename StreamReader -> IRStreamReader; Extract a StreamReader base class from IRStreamReader. WIP (depends on #22) - Rename StreamReader -> IrStreamReader; Extract a StreamReader base class from IrStreamReader. Oct 15, 2024
@junhaoliao
Copy link
Collaborator Author

This PR was replaced by #26

@junhaoliao junhaoliao closed this Nov 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant