Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

implement AsPtr for CanAnyFrame#43

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

jreppnow
Copy link
Contributor

Hi! This fixes the somewhat comical issue that I cannot directly put aCanAnyFrame back into aCanFdSocket from which I have received it. Bit ugly with thec_void and making thesize function non-static, but the smallest possible increment within the current implementation..

Urcra reacted with heart emoji
@jreppnowjreppnowforce-pushed thefix_trait_requirements branch from1efb834 to6261346CompareJune 4, 2023 16:00
@jreppnow
Copy link
ContributorAuthor

CI failure is not related to PR, see#44 (and merge that one first, will rebase this one onto the other PR)

@jreppnowjreppnowforce-pushed thefix_trait_requirements branch from6261346 to7cdc0a6CompareJune 4, 2023 16:08
@fpagliughi
Copy link
Collaborator

Thanks for the PR! I just quickly pushed out a pair of releases and totally forgot to look at the PR's. I'll get these in an do another release ASAP.

Copy link
Collaborator

@fpagliughifpagliughi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM

@fpagliughifpagliughi merged commit6a9fdf7 intosocketcan-rs:masterSep 21, 2023
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@fpagliughifpagliughifpagliughi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@jreppnow@fpagliughi
[8]ページ先頭
©2009-2025 Movatter.jp