Review

Ouverte
#30 12 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
5/5
Temps estimé
Plus d'une semaine
Accessibilité débutants
25/100
Type d'issue
Fonctionnalité
Clarté
À clarifier
Activité
À l'abandon
Stack technique
javascript
Domaine
web-dev

Piste de recherche

Commencez par examiner la checklist de l’issue #30, puis lisez les issues référencées #38 et #33, ainsi que le WHATWG structured-clone pull request. L’issue est terminée lorsque les questions normatives, éditoriales et de conception restantes ont fait l’objet de décisions explicites ou d’un suivi documenté.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Informative

  • I plan to bring this proposal up at the Wasm CG meeting next week to get Wasm's take. I think this is orthogonal and does not step on Wasm's toes. There is an interesting future design question for reflecting WasmGC Arrays of scalars as JS TypedArrays. I imagine those reflections will not have detachable buffers, but still warrants some discussion with the CG.

Normative

  • DetachArrayBuffer needs to throw for immutable buffers.
    #38
  • Please have a reviewed PR against HTML for structured clone. I don't think there's much design space here, so I'm personally happy for this to be a stage 3 blocker instead of a stage 2.7 blocker. But should WHATWG give feedback that requires this proposal to change, we'd need to make normative changes during 2.7.
    https://github.com/whatwg/html/pull/11033
  • I'd love to implement this with mprotect. However, that requires page alignment. We can't reveal the page size to user code, as that's a fingerprinting vector. Fixing a page size seems too unergonomic. Ideas? (Using mprotect is a nice-to-have for V8, not a requirement.)

Editorial

  • Prefer ValidateTypedArray's use argument be renamed accessMode. Ditto for the other Validate AOs.
    #33
  • Rename [AllocateArrayBuffer]'s maxByteLength parameter to something more descriptive since it can be IMMUTABLE. (I suppose you're minimizing the diff for now?)
    #33
Langage dominant
JavaScript
Étoiles
38
Forks
7
Aucune PR mergée en 30 j

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de tc39/proposal-immutable-arraybuffer

Toutes les issues de tc39/proposal-immutable-arraybuffer

Issues similaires

Plus d'issues JavaScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.