microsoft/Terminal

Investigate include order issue with `InputBuffer`, `readData`

Ouverte

#4 262 ouverte le 16 janv. 2020

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)C++ (9 275 forks)batch import
Area-CodeHealthHelp WantedIssue-TaskPriority-3Product-Conhost

Métriques du dépôt

Stars
 (103 173 étoiles)
Métriques de merge PR
 (Merge moyen 27j 19h) (24 PRs mergées en 30 j)

Description

Found in #4213.

I had to fwddecl InputBuffer, because I think there's a include order problem with in InputBuffer.hpp and readData.hpp.

readData.hpp uses InputBuffer in a bunch of places, but doesn't include it first

InputBuffer.hpp includes readData.hpp before it defines InputBuffer

So if you try to

#include ".../host/inputBuffer.hpp"

You'll get a compilation error, because it'll include readData, which will assume InputBuffer is defined, which it isn't yet.

Guide contributeur

Direction de recherche
Tracez l'ordre d'inclusion entre InputBuffer.hpp et readData.hpp pour résoudre le problème de dépendance circulaire ou de déclaration anticipée.
Stack technique
cpp
Domaine
cli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65