You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As for now, the Alt.EmitClients takes IPlayer[] as its first argument.
It seems better to prefer IEnumerable<IPlayer> here, so in piece of code like below example can avoid creating a new array with ToArray.
As for now, the
Alt.EmitClients
takesIPlayer[]
as its first argument.It seems better to prefer
IEnumerable<IPlayer>
here, so in piece of code like below example can avoid creating a new array withToArray
.After having a look at the core implementation, I think this change can be made into the core, as a
foreach
loop can replace thefor
one https://github.com/FabianTerhorst/coreclr-module/blob/dev/api/AltV.Net/Core.cs#L343.Plus, this change also doesn't break anything because
IEnumerable
is array base type.Example where an extra array must be created to satisfy
EmitClients
first argument:If it's reasonable, I can try working on this.
The text was updated successfully, but these errors were encountered: