Removed temporary array allocations for properties

[epeshk]

Similar to #1779, but uses on-stack allocation instead of pooling.

[epeshk]

I'm not sure it's worth (and possible) to optimize the default ILogger interface implementation before params ReadOnlySpan<T> support is released in C#.

https://github.com/dotnet/csharplang/blob/main/proposals/params-span.md

[nblumhardt]

Thanks - this looks like a nice option 👍 - may take some more time to review.

[nblumhardt]

I think this is the way to go - looks like CI produced some failures, though.

[nblumhardt]

May be worth including a few quick speculative tests to check that removing the == null test here is safe even when callers without strict null checking enabled call various logger methods, e.g.:

[Fact]
public void NullMessageTemplateParametersDoNotBreakBinding()
{
    var log = new LoggerConfiguration().WriteTo.Sink(new CollectingSink()).CreateLogger();
    log.Information("test", (object?[]?) null);
    log.BindMessageTemplate("test", (object?[]?)null, out _, out _);
}

?