-
-
Notifications
You must be signed in to change notification settings - Fork 46
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
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