Never Mix Tabs and Spaces - revisited
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
- Type d'issue
- Documentation
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- cpp
- Domaine
- documentation
Piste de recherche
Aucun fichier, test ou point d’entrée n’est indiqué. Commencez par localiser les règles existantes du dépôt concernant le style de code et déterminez si cette discussion a abouti à une décision approuvée ; la finalisation nécessiterait une politique arrêtée concernant les tabulations et les espaces, documentée dans les règles appropriées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
I was always in the - use only spaces - group for coding.
Until recently, when I started using clang-format. It has many options:
UseTab (UseTabStyle)
The way to use tab characters in the resulting file.
Possible values:
UT_Never (in configuration: Never) Never use tab.
UT_ForIndentation (in configuration: ForIndentation) Use tabs only for indentation.
UT_ForContinuationAndIndentation (in configuration: ForContinuationAndIndentation) Use tabs only for line continuation and indentation.
UT_Always (in configuration: Always) Use tabs whenever we need to fill whitespace that spans at least from one tab stop to the next one.
Now I use the setting: ForIndentation
Which uses Tabs for indenting the the block, and if more indentation is required for aligning, then spaces are used.
So it doesn't matter wether someone has 2, 4, or 8 spaces per tab, the code is always aligned correctly.
Currently my stance is: use Tab for indenting, use space for aligning.
- Langage dominant
- Aucune donnée de langage
- Étoiles
- 8.9k
- Forks
- 903
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de cpp-best-practices/cppbestpractices
-
Broken Link Chapter 2: Use_the_Tools_AvailablePeut-être à nouveau libre @vishesharma20 l’a pris il y a 172 jours, et aucune pull request n’est ouverte. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 64/100
cpp-best-practices/cppbestpractices#169 · 1 commentaire ·
-
Broken Link - 11 Further ReadingPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 55/100
cpp-best-practices/cppbestpractices#168 · 1 commentaire ·
-
C++ PracticeOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 10/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 35/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
cpp-best-practices/cppbestpractices#160 · 1 commentaire ·
Toutes les issues de cpp-best-practices/cppbestpractices
Issues similaires
-
area-multi-tenancy p1-recommended technical-debt
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
Docs: 2 broken linksOuverteBug pulumi/pulumi-terraform-bridge R2 S2
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
pulumi/pulumi-terraform-bridge#3713 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
-
[Sync EN] Document the timezone of DateTime(Immutable)::createFromTimestamp() (#6074)Peut-être pris @lacatoire l’a pris aujourd’hui. Ouvertesync-en
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
[Sync EN] Document the timezone of DateTime(Immutable)::createFromTimestamp() (#6074)Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertesync-en
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Les mainteneurs répondent en général sous 1 jour