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

Be more lenient about trailing spaces #4

Merged
merged 1 commit into from
Oct 28, 2020

Conversation

killercup
Copy link
Contributor

Hi, I've recently used canmatrix to generate a dbc file (from kcd) and can-dbc had some issues parsing it. Figuring out what went wrong took me about an hour, but the fix was rather trivial -- it was just a bunch of trailing spaces.

@marcelbuesing
Copy link
Owner

Awesome thanks for fixing this. I actually have experienced this as well. Sorry for not fixing it earlier. I haven't really found a good way yet to return proper errors, e.g. where exactly the parsing failed, I assume that would have helped as well.

@marcelbuesing marcelbuesing merged commit ab18b7a into marcelbuesing:dev Oct 28, 2020
@killercup
Copy link
Contributor Author

Thanks for the super quick merge!

I'm writing a small codegen tool for embedded usage right now (right now it's way simpler than your dbcc) and I'll try to make more PRs when I run into other things :)

@marcelbuesing
Copy link
Owner

marcelbuesing commented Oct 28, 2020

I just released 3.0.2 containing the fix to crates.io. Btw by coincidence I saw you worked on bitsh I assume this could make some generated code in dbcc easier to read. Maybe this dbc viewer using this crate is also interesting for. Easiest way to start it is via docker.

@killercup
Copy link
Contributor Author

Wow, that was super quick, thanks!

Yup, we have a small protoype with handwritting code right now. I'm in the process of cleaning that up, I'll ping you when I have a repo to show for it. bitsh will most likely be part of it, but maybe not with that name :)

@marcelbuesing
Copy link
Owner

Looking forward to see that! I had actually planned on making dbcc useable with no_std once the CAN PR is merged.

@killercup
Copy link
Contributor Author

Pushed a first draft here, will try to keep it updated with the code we actually put on devices :)

https://github.com/technocreatives/dbc-codegen

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.

None yet

2 participants