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

"permission query algorithm" is passed a redundant argument #394

Open
annevk opened this issue Nov 24, 2022 · 1 comment
Open

"permission query algorithm" is passed a redundant argument #394

annevk opened this issue Nov 24, 2022 · 1 comment

Comments

@annevk
Copy link
Member

annevk commented Nov 24, 2022

Since status holds a descriptor in [[query]], it's not clear to me why we're passing in a descriptor as well. It's rather confusing untangling this all.

@jyasskin
Copy link
Member

The descriptor is input to the algorithm, and the status is the place to write its output. It's true that the algorithm could have pulled the descriptor out of the status, but I was trying to make things as easy as possible for other specs that were implementing this. I think I overestimated how many specs would actually want to override any of this.

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

No branches or pull requests

2 participants