-
-
Notifications
You must be signed in to change notification settings - Fork 555
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
Add --with-comments option to dump #283
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good. If the potentially problematic module imports in database.py
can be avoided, I'm all for merging this!
if with_comments: | ||
com = [] | ||
if siginst.comment: | ||
com.append(siginst.comment) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just noticed this: the ARXML file format supports multi-lingual comments, so the canonical way to access the comments is the .comments
attribute of messages/signals. (which is a language string -> comment string dictionary. .comment
is just the first entry of this dictionary.) If you don't consider it worthwhile to support such an exotic feature, I think this PR should go in as-is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds like an additional PR to support that. I will keep it in the back of my head though :) Thanks for your review effort!
Add an option to include signal comments and units in the database dump output.