OpenXRay/xray-16

Get rid of Loki

Aberta

#240 aberto em 9 de set. de 2018

Ver no GitHubGuia do colaborador
 (3 comentários) (2 reações) (0 responsável)C++ (519 forks)auto 404
Code QualityExternal (3rd party)Help wantedPortability

Métricas do repositório

Stars
 (3.524 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Pesquise todos os usos de tipos Loki como Loki::Select e Loki::IsSameType no codebase e substitua os por equivalentes C++11 padrão, como std::conditional e std::is same. Para quaisquer recursos Loki que não sejam diretamente substituíveis, considere implementar uma substituição mínima ou remover o código não utilizado.
Pilha de tecnologia
ccpp
Domain
game dev
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++C++ templates
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40