-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
check for invalid use of Trice0/Trice #464
Comments
Usually you do not need to put the parameters count into the trice name. |
The use of Trice0 is related to #452 issue (compile on C++)
On these cases, trice insert is not giving an ID or report any message. it should report this mistake. |
Such trice tool check is a good idea, but it would need an additional parsing which would slow down the whole process. Also it is no solution for the limits of the current parsing because some other weird sequence could occur as well. I assume, that the compiler anyway will complain about missing IDs and the developers will be able to fix the code. |
Actually what was happening to me was something like:
trice insert
trice clean
I'm not sure if this was the sequence but somehow it ended with "orphan" ID's. |
The inserted IDs are expected to be |
When you clean the til.json file you need execute |
I just tested this, started from a empty li.json/til.json
Then I run
|
Yes, that is consistent wrong. |
In https://github.com/rokath/trice/blob/master/docs/TriceUserGuide.md#922-linited-trice-parser-capabilities there is now this documented as limitation. I close this issue now as we still have #465 and a separate check makes not much sense as explained. |
Trice0( variable ); // invalid use
Trice0( "tex %u", variable ); // invalid use
Should the trice tool flag this invalid usage of the Trice0 ?
The text was updated successfully, but these errors were encountered: