fix(ecs): getZipper([]) and getIndexedZipper([]) loop up to SIZE_MAX

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#490 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
74/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
cpp, typescript, wasm
Domaine
backend, testing

Piste de recherche

Lis les deux fonctions dans modules/ecs/wasm/Registry.hpp:300-322 et 338-360, en te concentrant sur la façon dont une liste de composants vide définit max et dont la boucle crée des objets JavaScript. Vérifie les tests ECS dans modules/ecs/test, qui, d’après l’issue, ne contiennent actuellement aucun test de zipper. Le travail est terminé lorsque les deux appels zipper avec des listes de composants vides renvoient des tableaux vides, que la faute de frappe std::size_t est corrigée et que les tests couvrent ce comportement.

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

Description

bug modules:ecs
Which package is this bug report for?

ecs

Issue description

With no component in the list, max stays SIZE_MAX, every index passes, and a JS object is pushed for each one: one call hangs the tab or the server and fills memory. std:size_t (lines 309 and 347) is a typo that compiles as a label. The two functions are near copies.

Where:

Evidence:

std::size_t max = SIZE_MAX;
...
std:size_t zipper_idx = 0;
for (std::size_t idx = 0; idx < max; idx++) {
    emscripten::val obj = emscripten::val::object();

Suggested fix: Return an empty array when comps is empty, fix the std::size_t typo, and merge the two zippers behind an indexed flag. Add zipper tests (modules/ecs/test has none).

Found during a code audit of main at be72076.

Code sample

No response

Versions
  • Engine main at be72076
  • pnpm 12.0.0, Node.js 26, Bun 1.3.14
Issue priority

Medium (should be fixed soon)

I have tested this issue on a development release

be72076

Langage dominant
TypeScript
Étoiles
7
Forks
2
Merge moyen
1 j 17 h
PR mergées (30 j)
17

Préparer son environnement

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 NanoForge-dev/Engine

Toutes les issues de NanoForge-dev/Engine

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

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