Replace "int x, int y" arguments with "Point point"

(2 commentaires) (1 réaction) (0 personne assignée)C++ (743 forks)batch import

good first issuehelp wanted

Métriques du dépôt

A lot of functions take two ints as the coordinates for their operations, this is both verbose and error-prone. Instead we should use the Point structure, see an example of this type of cleanup done for the GetDirection() https://github.com/diasurgical/devilutionX/commit/f77c52941afd73d4e734c0696d082218cd04c3a7

Try to split the changes up in one function per commit to making it easy to review and less prone to typos.

Direction de recherche: Étudiez le commit d'exemple (f77c529) pour comprendre le modèle de remplacement de `int x, int y` par `Point`. Recherchez dans la base de code les fonctions avec deux paramètres de coordonnées int, comme celles des modules de géométrie ou de rendu. Remplacez chaque paire de paramètres par une référence `Point`, mettez à jour tous les sites d'appel et assurez la cohérence avec l'utilisation existante de `Point`. Validez les modifications par fonction comme demandé pour faciliter la revue.
Stack technique: cpp
Domaine: backend
Type d'issue: Refactorisation
Difficulté: 3
Temps estimé: 1-2 jours
Statut d'activité: Active
Clarté: Claire
Prérequis: GitC++
Accessibilité débutant: 60