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

somersault ECU: reorder the definition of the data #150

Merged
merged 1 commit into from
Jun 30, 2023

Conversation

andlaus
Copy link
Collaborator

@andlaus andlaus commented Jun 30, 2023

this moves the definition of the tables, multiplexer, environment data and environment data descriptions behind the definition of the global negative responses. This is required because in a later commit, we would like to let the structure_ref of the table objects to actual structures, which is greatly simplified if the structures are already defined before the tables.

This commit does not change anything functional in these objects, but since git does not recognize that the code simply moved en-bloc, it makes it very hard to review this if there were any functional changes mixed in.

Andreas Lauser <[email protected]>, on behalf of MBition GmbH.
Provider Information

this moves the definition of the tables, multiplexer, environment data
and environment data descriptions behind the definition of the global
negative responses. This is required because in a later commit, we
would like to let the `structure_ref` of the table objects to actual
structures, which is greatly simplified if the structures are already
defined before the tables.

This commit does not change anything functional in these objects, but
since git does not recognize that the code simply moved en-bloc, it
makes it very hard to review this if there were any functional changes
mixed in.

Signed-off-by: Andreas Lauser <[email protected]>
Signed-off-by: Gerrit Ecke <[email protected]>
@andlaus andlaus requested a review from kayoub5 June 30, 2023 09:43
@kayoub5 kayoub5 merged commit 1343603 into mercedes-benz:main Jun 30, 2023
6 checks passed
@andlaus andlaus deleted the reorder_somersaultecu branch December 7, 2023 13:30
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