Skip to content

add support for multiplex signal parsing and syntax highlighting#32

Open
n0rbury wants to merge 2 commits intolharri73:masterfrom
n0rbury:DBC-Language-Syntax-PR-Branch
Open

add support for multiplex signal parsing and syntax highlighting#32
n0rbury wants to merge 2 commits intolharri73:masterfrom
n0rbury:DBC-Language-Syntax-PR-Branch

Conversation

@n0rbury
Copy link
Copy Markdown

@n0rbury n0rbury commented Apr 27, 2026

Implements #4

Before:
Syntax highlighting for multiplex signals is buggy. Signals (and messages after that) don't show up in preview window.
img_v3_02u5_830256ac-9830-464d-a9b5-d7f0daf838bg

After:
img_v3_02u5_98b7968a-0a16-45c7-8157-477e00268cdg

Attached is the sample dbc:

VERSION ""

NS_ :

BS_:

BU_:

BO_ 100 VehicleStatus: 8 Vector__XXX
  SG_ StatusCode M : 0|8@1+ (1,0) [0|255] "" ECU1
  SG_ Speed m0 : 8|16@1+ (0.1,0) [0|6553.5] "km/h" ECU1
  SG_ RPM m0 : 24|16@1+ (1,0) [0|65535] "rpm" ECU1
  SG_ Temperature m1 : 8|8@1+ (1,-40) [-40|215] "°C" ECU1
  SG_ Pressure m1 : 16|8@1+ (0.1,0) [0|25.5] "kPa" ECU1
  SG_ FuelLevel m2 : 8|8@1+ (0.5,0) [0|127.5] "L" ECU1

BO_ 200 SensorData: 8 Vector__XXX
  SG_ SensorID : 0|16@1+ (1,0) [0|65535] "" ECU2
  SG_ SensorValue : 16|32@1+ (0.001,0) [0|4294967.295] "V" ECU2

@lharri73
Copy link
Copy Markdown
Owner

lharri73 commented Apr 28, 2026

This looks good!

You can ignore the codeql check, looks like that job is broken :/
You'll need to increment the version string in the package.json (should probably be 2.1.0) and add a changelog entry (TODO: reverse the order of the changelog).

Update: I may need to do some version updating before CI will pass for any of these.

@n0rbury n0rbury force-pushed the DBC-Language-Syntax-PR-Branch branch from f0d603e to b8d58af Compare April 28, 2026 02:22
@n0rbury
Copy link
Copy Markdown
Author

n0rbury commented Apr 28, 2026

This looks good!

You can ignore the codeql check, looks like that job is broken :/ You'll need to increment the version string in the package.json (should probably be 2.1.0) and add a changelog entry (TODO: reverse the order of the changelog).

Update: I may need to do some version updating before CI will pass for any of these.

done :)

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