OpenXRay/xray-16

Switch to UTF-8

Aperta

#419 aperta il 20 giu 2019

Vedi su GitHubGuida contributor
 (0 commenti) (2 reazioni) (0 assegnatari)C++ (519 fork)auto 404
CoreDeveloper ExperienceHelp wantedPortability

Metriche repository

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

Descrizione

Rationale

Currently, engine uses system native locales and just doesn't support UTF-8. This results in several problems, like player save files having wrong characters when written on disk, inconvenient interoperability with third party libraries that use UTF-8 by default.

Proposed solution

To my knowledge, C++17/C++20 doesn't provide required UTF-8 functionality. There are two big libraries that can be used:

Their functionality should be equal, ICU is preferred, and we likely want to support both (i.e. iconv too), because e.g. iconv is native to some systems and doesn't require additional packages to be installed.

Useful links

FreeBSD documentation:

Methodology

https://utf8everywhere.org/

Guida contributor

Direzione di ricerca
Ricerca su come integrare le librerie ICU e iconv nel sistema di build CMake. Identifica tutti i punti nel codice di xray 16 che utilizzano funzioni di localizzazione di sistema (es. setlocale, std::locale, ecc.) e sostituiscili con chiamate compatibili con UTF 8. Guarda progetti open source esistenti che hanno effettuato migrazioni simili per riferimento.
Tech stack
cpp
Dominio
game dev
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++CMakeGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45