Does Array.index need to be a foreign import?
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Domaine
- backend
Piste de recherche
The issue names Array.index, length, and unsafeIndexImpl, and mentions the backend optimizer. Start by reading the implementation of Array.index and how its foreign import is used, then check whether the optimizer change described in the issue has landed. Done means determining whether the bounds check can be implemented in PureScript while preserving the expected behavior and enabling the stated optimization.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
The logic of Array.index can be implemented in terms of length and unsafeIndexImpl. Is it necessary for it to be foreign? In backend-optimizer, I'm preparing a change to inline the bounds check so that the Just allocation can be eliminated in some cases. If index bounds checking was implemented in PureScript rather than FFI, this would just work.
- Langage dominant
- PureScript
- Étoiles
- 58
- Forks
- 70
- Merge moyen
- 2 j 22 h
- PR mergées (30 j)
- 1
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Aucun guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de purescript/purescript-arrays
-
Description text of `difference`Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 55/100
purescript/purescript-arrays#246 ·
-
generate functionOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
purescript/purescript-arrays#244 · 2 commentaires ·
-
deleteWith function & friendsOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
purescript/purescript-arrays#242 · 2 commentaires ·
-
`groupBy` has unexpected `l` argument for `a -> a -> Boolean` predicatePeut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 25/100
purescript/purescript-arrays#229 · 3 commentaires ·
-
type: enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 30/100
purescript/purescript-arrays#215 · 3 commentaires ·
Toutes les issues de purescript/purescript-arrays
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
-
ManageSieve proxy: a backend's NO/BYE greeting is skipped as a capability line and reported as EOFOuvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Mend: dependency security vulnerability
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
Mend: dependency security vulnerability
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
ManageIQ/manageiq-pods#1476 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
prime-radiant-inc/evener#6366 ·
Les mainteneurs répondent en général sous 1 jour