Document (or fix) stack safety issues

Ouverte
#194 5 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
24/100
Type d'issue
Documentation
Clarté
À clarifier
Activité
À l'abandon
Domaine
backend, testing

Piste de recherche

The issue names sort, nubEq, and nub, and points to #192 and #179 for related work. Start by reading those issues and checking the functions’ stack-safety behavior, then assess whether tests can cover each function and whether the suggested V8 stack-size setting helps them run faster. Done means documenting or fixing the identified cases and establishing stack-safety tests; the issue does not specify which outcome to prioritize.

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

Description

type: enhancement

Some example of existing stack-unsafe functions:

  • sort - reported in #192
  • nubEq, can be fixed for strict lists by applying same strategy used for struct nub in #179
  • nub for lazy lists. Don't know of a good way to fix this. The strict nub strategy of #179 cannot be applied here because you can't reverse an infinite lazy list.

We should ideally setup stack-safety tests for every function. To help make those tests run faster, we could shrink the v8 stack size with something like --stack_size=10 (default is 984 kB).

Langage dominant
PureScript
Étoiles
61
Forks
50
Merge moyen
2 j 6 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

  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 purescript/purescript-lists

Toutes les issues de purescript/purescript-lists

Issues similaires

Plus d'issues Backend & API Design

Recevez les nouvelles issues par e-mail

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