Skip to content

[release/11.0] Add SequenceReaderExtensions.TryPeekEndian - #132574

Closed
github-actions[bot] wants to merge 1 commit into
release/11.0from
backport/pr-130178-to-release/11.0
Closed

[release/11.0] Add SequenceReaderExtensions.TryPeekEndian#132574
github-actions[bot] wants to merge 1 commit into
release/11.0from
backport/pr-130178-to-release/11.0

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Customer Impact

  • Customer reported
  • Found internally

Adds the SequenceReader<byte>TryPeekLittleEndian and TryPeekBigEndian extension methods, allowing callers to inspect numeric values without advancing the reader. This addresses customer demand captured in API proposal #116414 and backports the API addition from #130178. It is not strictly associated with other .NET 11 features, and does not change existing behavior.

Regression

  • Yes
  • No

Testing

The API and behavior are covered by the tests included in #130178. The tests verify little-endian and big-endian peeking, including insufficient data, and that peeking does not advance the reader.

Risk

Low. This adds new API surface area on top of what is included in RC1, but it is a focused, additive API with isolated implementation and tests. It does not alter existing APIs or behavior. Other endian-aware SequenceReader APIs remain outside the scope of this backport.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/area-system-memory
See info in area-owners.md if you want to be subscribed.

@jeffhandley

Copy link
Copy Markdown
Member

@jeffhandleyjeffhandley added the Servicing-consider Issue for next servicing release review label Aug 20, 2026
@jeffhandley

Copy link
Copy Markdown
Member

@artl93 This does not strictly meet the bar, so I was on the fence for bringing it for consideration. The standalone aspect and tangential relation to other feature areas made me want to run it by you for consideration.

@artl93

Copy link
Copy Markdown
Member

@jeffhandley - I'm struggling with this one too - the timing is unfortunate, since we're locking down. This doesn't strike me as a quality or scenario completeness improvement based on the description or the API request. Is there more context?

@jeffhandley

Copy link
Copy Markdown
Member

@artl93 -- Thanks for considering it. I thought it was worth consideration since it's in the same spirit of some other work in .NET 11 even though it's not directly related, but we can reject this one and let it get introduced in .NET 12.

/cc @tannergooding

@jeffhandleyjeffhandley added Servicing-rejected and removed Servicing-consider Issue for next servicing release review labels Aug 21, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@jeffhandley@artl93@lilinus