OpenXRay/xray-16

Get rid of Loki

Offen

#240 geöffnet am 09.09.2018

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)C++ (519 Forks)auto 404
Code QualityExternal (3rd party)Help wantedPortability

Repository-Metriken

Stars
 (3.524 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Suche nach allen Verwendungen von Loki Typen wie Loki::Select und Loki::IsSameType im Codebase und ersetze sie durch Standard C++11 Äquivalente wie std::conditional und std::is same. Für nicht direkt ersetzbare Loki Funktionen erwäge die Implementierung eines minimalen Ersatzes oder das Entfernen von ungenutztem Code.
Tech Stack
ccpp
Domain
game dev
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++C++ templates
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40