Add MemoryExtensions.TryWriteUtf8 #83761
Conversation
ghost
assigned 
|
Note regarding the This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change. |
|
Tagging subscribers to this area: @dotnet/area-system-memory Issue DetailsInitial implementation of MemoryExtensions.TryWriteUtf8. The performance of this won't be great at present, but will improve as our cores types add implementation of IUtf8SpanFormattable. For tests, I copy/pasted the tests we had for TryWrite and searched/replaced to make them work for TryWriteUtf8.
|
Initial implementation of MemoryExtensions.TryWriteUtf8. The performance of this won't be great at present, but will improve as our cores types add implementation of IUtf8SpanFormattable. For tests, I copy/pasted the tests we had for TryWrite and searched/replaced to make them work for TryWriteUtf8.
stephentoub
force-pushed
the
memextutf8trywrite
branch
from
4863a7d
to
c36a782
Compare
|
We decided to move the support to a different type. I'll close this and open a new PR with the revision soon. |
Initial implementation of MemoryExtensions.TryWriteUtf8. The performance of this won't be great at present, but will improve as our cores types add implementation of IUtf8SpanFormattable. For tests, I copy/pasted the tests we had for TryWrite and searched/replaced to make them work for TryWriteUtf8.
Fixes #79376
Contributes to #81500