swiftlang/swift

[SR-540] eliminate the .cpp.gyb files from the source tree

Open

#43.157 aperta il 14 gen 2016

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Swift (10.719 fork)batch import
bugcompilergood first issue

Metriche repository

Star
 (69.989 star)
Metriche merge PR
 (Merge medio 8g 17h) (510 PR mergiate in 30 g)

Descrizione
Previous ID SR-540
Radar None
Original Reporter @lattner
Type Bug
Votes 0
Component/s Compiler
Labels Bug, StarterBug
Assignee None
Priority Medium

md5: ff02b1c09cc812ba86f097ddd5c3cf2d

Issue Description:

Gyb is a necessary artifact used in the standard library and test suite due to Swift not having a preprocessor. C++ on the other hand DOES have a preprocessor, so we should definitely not use gyb for .cpp files, we should use ".def" files consistently. Unfortunately we are using it in a couple of cases:

lib/Basic/UnicodeExtendedGraphemeClusters.cpp.gyb
stdlib/public/stubs/UnicodeExtendedGraphemeClusters.cpp.gyb
unittests/Basic/UnicodeGraphemeBreakTest.cpp.gyb

These should switch to using the C preprocessor.

Guida contributor

Direzione di ricerca
Identificare i file .cpp.gyb elencati, capire come gyb genera codice C++ e sostituire il templating gyb con direttive del preprocessore C (#define, #if, ecc.). Assicurarsi che i file .cpp generati rimangano identici.
Tech stack
swiftcpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
SwiftC++C preprocessor
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80