-
Notifications
You must be signed in to change notification settings - Fork 32
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
[FR]: Add ReadOnlySpan-based overload for FirebaseAnalytics.LogEvent #1013
Comments
I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight. |
Hi @abogarsukov-braingames, thanks for suggesting a feature request! This does seem like an efficient way of handling allocations. That said, I'll go ahead and mark this as a feature request, and bring this up to our engineering sync to see if we can get some feedback. While we are unable to promise any timeline for this, we'll definitely keep this under our radar. |
Description
Currently there are overloads of the
FirebaseAnalaytics.LogEvent
for a single parameter and array of parameters. In most cases this is suboptimal as each event requires allocating an array of parameters. Adding an overload acceptingReadOnlySpan<Parameter>
would help developers avoid unnecessary allocations.API Proposal
Add a new
LogEvent
overload like this:Firebase Product(s)
Analytics
Targeted Platform(s)
Apple Platforms, Android, Desktop
The text was updated successfully, but these errors were encountered: