Skip to content
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

In JS interop, add Invoke/InvokeAsync overloads that use C#11 param span #40098

Open
SteveSandersonMS opened this issue Feb 9, 2022 · 9 comments
Labels
area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one good first issue Good for newcomers. Pillar: Technical Debt Priority:2 Work that is important, but not critical for the release
Milestone

Comments

@SteveSandersonMS
Copy link
Member

SteveSandersonMS commented Feb 9, 2022

Hopefully we can save one allocation per call, at least in the cases that had to use params in the past.

@SteveSandersonMS SteveSandersonMS added the area-blazor Includes: Blazor, Razor Components label Feb 9, 2022
@javiercn javiercn added this to the .NET 7 Planning milestone Feb 10, 2022
@ghost
Copy link

ghost commented Feb 10, 2022

Thanks for contacting us.

We're moving this issue to the .NET 7 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@mkArtakMSFT mkArtakMSFT added triaged good first issue Good for newcomers. Priority:2 Work that is important, but not critical for the release enhancement This issue represents an ask for new feature or an enhancement to an existing one labels Mar 30, 2022
@mu88
Copy link

mu88 commented Jun 21, 2022

Is this still part of the C# 11 / .NET 7 roadmap? Or will it be shipped later? Because I don't find params span here

@SteveSandersonMS
Copy link
Member Author

@mu88 It's still a .NET 7 candidate, however we haven't assigned it to a particular milestone as it's not clear we have capacity to do it. If you're interested in doing a PR for this, that might be what's needed to get it in on time. Otherwise I can't predict yet whether or not it will make it.

@SteveSandersonMS
Copy link
Member Author

@mu88 Oh actually I see your point that the language feature itself isn't there yet. In that case it's fairly clear that it wouldn't be something we could do for Blazor!

@mu88
Copy link

mu88 commented Jul 4, 2022

Yeah that's what I mean @SteveSandersonMS - I'm happy to assist as soon as the language feature is available

@ghost
Copy link

ghost commented Oct 26, 2022

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@ghost
Copy link

ghost commented Dec 14, 2023

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@WeihanLi
Copy link
Contributor

Is there an approved API proposal?

@camkinney
Copy link

It looks like this C# language feature was original proposed on this Issue: dotnet/csharplang#1757
It has now been subsumed by this Proposal issue for Parent Collections support in general dotnet/csharplang#7700

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one good first issue Good for newcomers. Pillar: Technical Debt Priority:2 Work that is important, but not critical for the release
Projects
None yet
Development

No branches or pull requests

6 participants