-
Notifications
You must be signed in to change notification settings - Fork 10
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
EvaluatorOptions class for defining behavioral options in vbSparkle #2
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add EvaluatorOptions with following option - IndentSpacing (implemented) - PerfomPartialEvaluation (to implement) - JunkCodeProcessingMode (implemented) - SymbolRenamingMode (to implement) - Implements JunkCodeProcessingMode option in If Then Else blocks - Implements IndentSpacing in overall generated code - Propagate the Option through context overall statement visitors & scoped objects
Create main.yml
Also, the CLI has been updated accordingly to the new Options features:
@citronneur , Let me know if you need more clarification to review/merge the PR. ;) |
Provide syntax coloring Try improve workflow Fix naming in publish workflow Provide real CLI with syntax Coloring Fix readme with information about CLI options Improve way of processing CLI options & standard input
Add of ExecuteObserver Add of CreateObjectObserver
Add Shell() interceptor Add CreateObject() interceptor
Constants vbUseCompareOption vbBinaryCompare vbTextCompare vbDatabaseCompare String Manipulation Methods InStr InStrB InStrRev LCase LCase_S UCase UCase_S Len LenB Mid Mid_S MidB MidB_S Left Left_S LeftB LeftB_S Right Right_S RightB RightB_S RTrim RTrim_S LTrim LTrim_S Join Split Financial Methods DDB FV IPmt IRR MIRR NPer NPV Pmt PPmt PV Rate SLN SYD Array Methods Array LBound UBound
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I am happy to release this new feature that I have been waiting to implement for a long time: a new option class, called EvaluatorOptions, that enables customization of the behavior of vbSparkle.
These options are:
IndentSpacing
: define number of indentation spaces (default is4
spaces)SymbolRenamingMode
: renaming of symbols (defaultNone
)JunkCodeProcessingMode
: junk/dead code processing (default isComment
)Basically, here is how to declare these options:
The currently available options for SymbolRenamingMode are :
It's related to issue #1.
The options for dead/junk code processing are:
Nothing
: the junk code will be kept with all the logicRemove
: the junk code will be removedComment
: the junk code will be commentedThe variable renaming feature currently implemented is simple. It's using the format of
var_{idx}
orconst_{idx}
.It can be applied to variables and constants but it does not have scope-specific renaming capability (it means that
idx
counter is global).Other changes where done:
$(SolutionDir)\\Resources\\samples\\
If / Then / ElseIf / Else
blocks where evaluated