- From: Anne van Kesteren <notifications@github.com>
- Date: Mon, 19 Oct 2026 15:55:55 -0700
- To: whatwg/encoding <encoding@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <whatwg/encoding/pull/368@github.com>
Infra list indices are zero-based, so peek skipped the first item and could go out-of-bounds. This aligns peek with implementations and tests.
Also remove "run these steps" from "To X" sentences.
Fixes #367.
<!--
This comment and the below content is programmatically generated.
You may add a comma-separated list of anchors you'd like a
direct link to below (e.g. #idl-serializers, #idl-sequence):
Don't remove this comment or modify anything below this line.
If you don't want a preview generated for this pull request,
just replace the whole of this comment's content by "no preview"
and remove what's below.
-->
***
<a href="https://whatpr.org/encoding/368.html" title="Last updated on Oct 19, 2026, 10:55 PM UTC (3c4f081)">Preview</a> | <a href="https://whatpr.org/encoding/368/dd79434...3c4f081.html" title="Last updated on Oct 19, 2026, 10:55 PM UTC (3c4f081)">Diff</a>
You can view, comment on, or merge this pull request online at:
https://github.com/whatwg/encoding/pull/368
-- Commit Summary --
* Fix off-by-one error in peek
-- File Changes --
M encoding.bs (36)
-- Patch Links --
https://github.com/whatwg/encoding/pull/368.patch
https://github.com/whatwg/encoding/pull/368.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/encoding/pull/368
You are receiving this because you are subscribed to this thread.
Message ID: <whatwg/encoding/pull/368@github.com>
Received on Monday, 19 October 2026 22:55:59 UTC