Skip to content

added multiplex support #22

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

Closed
wants to merge 5 commits into from
Closed

added multiplex support #22

wants to merge 5 commits into from

Conversation

mork2020
Copy link

@mork2020 mork2020 commented Apr 21, 2023

dbc mux feature added to parsing and printing
tests for mux added

what to consider:
With my additions to the code the multiplex index variable must be defined before a multiplexed variable in the DBC file.
If a multiplexed variable is defined/read before the multiplex index variable is read, the property MultiplexName in the SignalDescriptor wont be set at all.

mux_test.dbc.txt

@astand
Copy link
Owner

astand commented May 18, 2023

Hi @mork2020, thanks for participation, and sorry for delay in response

This change is really huge, mostly from generated code prospective. For users who work with multiplex signals in manual way (as I prefer to do as well) this feature will bring a lot of undesirable changes.

For deploying this feature, it is necessary to add two configuration steps:

  1. To add command line argument which will enable this advanced multiplex generation
  2. Inside generated source code multiplex using also must be protected by special macro like it is done for floating, monitoring etc

@astand astand deleted the branch astand:master October 9, 2023 15:02
@astand astand closed this Oct 9, 2023
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.

2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy