Remove one layer of abstraction and multiple intermediate types - #2039

Closed

adamsitnik wants to merge 16 commits into

dotnet:mainfrom

adamsitnik:removeLayer

Commits on Jan 24, 2023

Commits on Jan 25, 2023

address code review feedback:
adamsitnik
committed
f8370ae

Commits on Jan 26, 2023

refactor the code: merge validation and default values creation into one (20% perf gain on simple scenarios)
adamsitnik
committed
66c3e4f
refactor:
adamsitnik
committed
94f9705
fix a bug I have introduced: when no Tokens are available the result should be None
adamsitnik
committed
49ab482
re-implement OnlyTake, remove some virtual properties and get all the tests passing again
adamsitnik
committed
ecf99fd
reduce the number of ctors and methods
adamsitnik
committed
aa852d1
solve the TODO: use Enum.TryParse when available
adamsitnik
committed
bae82fe

Commits on Jan 27, 2023

Commits on Jan 30, 2023

Apply suggestions from code review

adamsitnik
and
jonsequitur
authored
10fdeeb
Address code review feedback
adamsitnik
committed
1307432

Commits on Jan 31, 2023

merge ParseOperation and ParseResultVisitor into one to reduce the number of JITted types and allocated memory
adamsitnik
committed
1aa6937