microsoft/Terminal

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

开放

#4,262 创建于 2020年1月16日

 (1 条评论) (0 个反应) (0 位负责人)C++ (9,275 个派生)batch import
Area-CodeHealthHelp WantedIssue-TaskPriority-3Product-Conhost

仓库指标

星标
 (103,173 个星标)
PR 合并指标
 (平均合并 27天 19小时) (30 天内合并 24 个 PR)

描述

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.

贡献者指南