OpenXRay/xray-16

Get rid of Loki

Aperta

#240 aperta il 9 set 2018

Vedi su GitHubGuida contributor
 (3 commenti) (2 reazioni) (0 assegnatari)C++ (519 fork)auto 404
Code QualityExternal (3rd party)Help wantedPortability

Metriche repository

Star
 (3524 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Cerca tutti gli usi dei tipi Loki come Loki::Select e Loki::IsSameType nel codebase, quindi sostituiscili con equivalenti C++11 standard come std::conditional e std::is same. Per qualsiasi funzionalità Loki non direttamente sostituibile, considera l'implementazione di una sostituzione minima o la rimozione del codice non utilizzato.
Tech stack
ccpp
Dominio
game dev
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++C++ templates
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40