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

A2L parsing error due to the preprocessing the multipline comments #48

Open
xfkeivn opened this issue Dec 31, 2023 · 1 comment
Open

Comments

@xfkeivn
Copy link

xfkeivn commented Dec 31, 2023

In the preprocessing of multiple line comments as following lines, all this COMPU_METHOD will be removed. It will cause the error in the lexcer paring. so either

  1. don't preprocess the comments, because the lexcer will first channel hidden to remove the comments
  2. keep the text after / when / and */ at the same line.

/begin COMPU_METHOD
/* Name of CompuMethod / SWC_JctTempEstm_CM_single
/
Long identifier / "Q = V"
/
Conversion Type / RAT_FUNC
/
Format / "%8.6"
/
Units / ""
/
Coefficients */ COEFFS 0 1 0 0 0 1
/end COMPU_METHOD

/begin COMPU_METHOD
/end COMPU_METHOD

@christoph2
Copy link
Owner

I will soon release a completely new parser written in C++, which fixes this and other problems (not to mention dramatically increased processing speed and reduced memory consumption).
Note: the parsing result of your example will be still incorrect, because parameters Name to Unit are mandatory!

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

No branches or pull requests

2 participants