-
-
Couldn't load subscription status.
- Fork 49
Feat/code formatter #443
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
Feat/code formatter #443
Conversation
|
It occurred to me that a lot of calls of getCursor were added, and this method recompute the position every time it's called, from scratch. An alternative will have to be created, so that we can dynamically compute the line and column number when calling next/backtrack, without requiring to recompute from scratch said cursor. Edit: this has been fixed, see BaseParser and the usage of m_it_to_row. |
2e33e10 to
ed5db0f
Compare
…alysis tools to run on PRs
…tering the interface
…s go together in blocks
ed5db0f to
59ccf88
Compare
ee2784e to
e2198ea
Compare
c9d2a01 to
a0ff755
Compare
Static analysis reportLizard reportListing only functions with cyclomatic complexity >= 15 or NLOC >= 100 or parameters >= 10.
CppCheck report |
a0ff755 to
0ac0153
Compare
3d8bafa to
2337008
Compare
2337008 to
80ca896
Compare
2f188a7 to
5907063
Compare
…comments after nodes too
5907063 to
6e81769
Compare
6e81769 to
a773898
Compare
Description
Adds a very simple and non-configurable ArkScript code formatter to the CLI.
It seems that with the parser modifications this PR required, the parsing times went up by a lot. I'll need to confirm and check why.
Closes #421
Checklist