Skip to content

Apply .NET8 MethodInvoker/ConstructorInvoker optimization #163

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

Merged
merged 4 commits into from
Nov 28, 2023

Conversation

SergeiPavlov
Copy link
Collaborator

@SergeiPavlov SergeiPavlov requested a review from botinko November 21, 2023 23:14
@SergeiPavlov SergeiPavlov changed the title Apply .NET8 MethodInvoker optimization Apply .NET8 MethodInvoker optimization Nov 21, 2023
@SergeiPavlov SergeiPavlov changed the title Apply .NET8 MethodInvoker optimization Apply .NET8 MethodInvoker/ConstructorInvoker optimization Nov 22, 2023
Copy link

@botinko botinko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch.

@SergeiPavlov SergeiPavlov merged commit 401e803 into master-servicetitan Nov 28, 2023
@SergeiPavlov SergeiPavlov deleted the methodInvoker branch November 28, 2023 04:29
SergeiPavlov added a commit that referenced this pull request Nov 29, 2023
* Apply .NET8 MethodInvoker optimization

* Remove unused line

* Use ConstructorInvoker

* Simplify conditionallly-compiled code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants