search

RomanYankovsky / DelphiAST

Abstract syntax tree builder for Delphi
Mozilla Public License 2.0
271 stars 116 forks source link

Suggested fixes for most of the issues I reported #231

Open JBontes opened 6 years ago

JBontes commented 6 years ago

Note that #224 also fixes a slight issue where the SyntaxNodeNames got out of sync with the SyntaxNodeTypes (My bad). Also note that #224 allows you to put the operators anywhere in the list, as long as all operators are kept together.

Added all remaining fixes in my local version.

This concludes the list of issues for which I have an easy fix.

RomanYankovsky commented 6 years ago

Thank you for creating a pull request. It will save a lot of time. I'll review it as soon as possible.

JBontes commented 6 years ago

You're welcome. I'm just trying to get AST to parse all code correctly so I can use it as a macro/code rewrite tool. Unfortunately my own version has got quite a few alterations that are out of scope for DelphiAST, so I cloned a fresh version and plugged in the changes there.

JBontes commented 5 years ago

I'll review it as soon as possible.

I fixed the conflict, have you had time to look at it yet? No rush :-)

alex-ilin commented 3 years ago

@RomanYankovsky, will you review the PR already? The suspense is killing me!