swiftlang/swift

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

Offen

#43.157 geöffnet am 14.01.2016

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Swift (10.719 Forks)batch import
bugcompilergood first issue

Repository-Metriken

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

Beschreibung
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.

Contributor Guide

Research-Richtung
Identifizieren Sie die aufgelisteten .cpp.gyb Dateien, verstehen Sie, wie gyb C++ Code generiert, und ersetzen Sie die gyb Vorlagen durch C Präprozessor Anweisungen (#define, #if usw.). Stellen Sie sicher, dass die generierten .cpp Dateien identisch bleiben.
Tech Stack
swiftcpp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
SwiftC++C preprocessor
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80