facebookarchive/prepack

Prepack emits code containing invalid global.TypedArray access

Open

#2 292 ouverte le 18 juil. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (14 268 stars) (520 forks)batch import
bughelp wantedinterpreter

Description

TypedArray is a global not exposed to the user JavaScript runtime. Try global.TypedArray in your JavaScript environment of choice. We must emit Object.getPrototypeOf(Int8Array) or similar instead of TypedArray.

Input

var TypedArray = Object.getPrototypeOf(Int8Array);
console.log(TypedArray.from.name === "from");

Output

var TypedArray;
(function() {
  var _$0 = this;

  var _$1 = _$0.TypedArray; // <-------------------- `TypedArray` is not exposed on the global object!
  _$0.TypedArray = _$1;
  console.log(true);
}.call(this));

Guide contributeur

Stack technique
javascriptnodejs
Domaine
build system
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Understanding of JavaScript prototypesFamiliarity with Prepack's code generation
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
20
Direction de recherche
The issue is that Prepack incorrectly emits `global.TypedArray` instead of a valid expression like `Object.getPrototypeOf(Int8Array)`. To fix, locate where Prepack serializes global object property accesses and replace the usage of `TypedArray` with a reference that works in all JavaScript environments. The relevant code is likely in the serializer or environment module of Prepack. Check the source files in the `lib/` directory for handling of built in objects. The fix should ensure that the emitted code does not assume `TypedArray` is a global.