Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Iteration and append methods for byte sequence used in standards are undefined #571

Open
SuperSonicHub1 opened this issue May 2, 2023 · 1 comment

Comments

@SuperSonicHub1
Copy link

The URL Standard makes use of two byte sequence methods for percent-decoding that go undefined in the Infra Standard:

  • iteration and "for each" (used on L409; defined in Infra for lists, sets, and maps)
  • appending a byte to a byte sequence (used on L412, L416, 427; defined in Infra for lists and sets)

I believe copy-pasting the existing definitions for lists should be adequate. This brings into question if a byte sequence should be defined as a list of bytes so it can inherit the properties and methods of a list, but that is better left for another issue.

@annevk
Copy link
Member

annevk commented May 2, 2023

Thanks for raising this. In a way this is a duplicate of #91, but since it's very clear I'm inclined to leave this open for now.

(#110 makes an argument against strings being lists, but maybe byte sequences is still reasonable.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants