infinitespace-studios/VectorRumble

Clean up usage of Linq and `.ToArray()`.

Open

#59 ouverte le 27 août 2025

Voir sur GitHub
 (0 commentaires) (1 réaction) (1 assigné)C# (5 forks)auto 404
bughelp wanted

Métriques du dépôt

Stars
 (15 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

Both Linq and ToArray produce allot of Garbage, and should not be used at all unless absolutely necessary. Using it for loading and saving is ok, but in a game loop no. This is because things like ToArray make copies of the data. If these are only used for a short period of time then the garbage collector will need to clean them up. It would be better to use the lists directly. Also .Any() should be removed in favour of .Length > 0 for exactly the same reason.

We should review all the code and remove usage.

Guide contributeur

Direction de recherche
Stack technique
Domaine
Type d'issue
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Prérequis
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.