OpenXRay/xray-16

Get rid of Loki

Ouverte

#240 ouverte le 9 sept. 2018

Voir sur GitHubGuide contributeur
 (3 commentaires) (2 réactions) (0 personne assignée)C++ (519 forks)auto 404
Code QualityExternal (3rd party)Help wantedPortability

Métriques du dépôt

Stars
 (3 524 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Loki is a C++ library found in the sdk folder. There are several reasons to remove it:

  • Most of the stuff of this library, written for C++98 by Andrei Alexandrescu, can be found in C++11, e.g. Loki::Select is just std::conditional, Loki::IsSameType can be replaced with std::is_same.
  • Some of the code does not compile in GCC. (With some changes, it compiled in GCC)
  • The things that haven't been included in modern C++, can be rewritten with fewer lines of code.
  • It's barely used.

Guide contributeur

Direction de recherche
Recherchez toutes les utilisations des types Loki comme Loki::Select et Loki::IsSameType dans la base de code, puis remplacez les par des équivalents C++11 standard tels que std::conditional et std::is same. Pour toute fonctionnalité Loki non directement remplaçable, envisagez de mettre en œuvre un remplacement minimal ou de supprimer le code inutilisé.
Stack technique
ccpp
Domaine
game dev
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++C++ templates
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40