Skip to content

Conversation

syg
Copy link
Contributor

@syg syg commented May 19, 2021

This commit adds the [AllowResizable] extended attribute, which allow
BufferSources to take resizable buffers. It composes with [AllowShared]
in the obvious way. The default behavior is unchanged, with only
fixed-length buffers allowed.

Proposal link: https://github.com/tc39/proposal-resizablearraybuffer


💥 Error: 500 Internal Server Error 💥

PR Preview failed to build. (Last tried on Aug 12, 2021, 7:12 PM UTC).

More

PR Preview relies on a number of web services to run. There seems to be an issue with the following one:

🚨 CSS Spec Preprocessor - CSS Spec Preprocessor is the web service used to build Bikeshed specs.

🔗 Related URL

<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<html><head>
<title>500 Internal Server Error</title>
</head><body>
<h1>Internal Server Error</h1>
<p>The server encountered an internal error or
misconfiguration and was unable to complete
your request.</p>
<p>Please contact the server administrator at 
 [no address given] to inform them of the time this error occurred,
 and the actions you performed just before this error.</p>
<p>More information about this error may be available
in the server error log.</p>
<hr>
<address>Apache/2.4.10 (Debian) Server at api.csswg.org Port 443</address>
</body></html>

If you don't have enough information above to solve the error by yourself (or to understand to which web service the error is related to, if any), please file an issue.

@syg
Copy link
Contributor Author

syg commented May 19, 2021

PTAL @annevk and @domenic

Copy link
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with minor nits

@annevk
Copy link
Member

annevk commented May 21, 2021

Looks good to me as well. (I still think we need to do #961 (comment), but that doesn't really influence this I think.)

@syg
Copy link
Contributor Author

syg commented May 21, 2021

Looks good to me as well. (I still think we need to do #961 (comment), but that doesn't really influence this I think.)

Agree, if SABs get their own IDL type, [AllowResizable] will compose with that type as well.

@TimothyGu TimothyGu marked this pull request as draft July 1, 2021 23:09
@domenic domenic self-assigned this Aug 2, 2021
@domenic
Copy link
Member

domenic commented Aug 11, 2021

@syg I noticed resizable buffers is at stage 3. And, this is probably worth landing since it helps prohibit resizable buffers from making their way into Web APIs that don't expect them. Should we merge?

@syg
Copy link
Contributor Author

syg commented Aug 11, 2021

Good call, this had fallen off my radar. I'll remove the draft label.

@syg syg marked this pull request as ready for review August 11, 2021 23:55
This commit adds the [AllowResizable] extended attribute, which allow
BufferSources to take resizable buffers. It composes with [AllowShared]
in the obvious way. The default behavior is unchanged, with only
fixed-length buffers allowed.

Proposal link: https://github.com/tc39/proposal-resizablearraybuffer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants