tree d9301de921009468681515860f94c4921654d9ab
parent 3ea648e44bbaade2edfd46cb397c43c20e860255
author QUICHE team <quiche-dev@google.com> 1660755469 -0700
committer Copybara-Service <copybara-worker@google.com> 1660755566 -0700

`balsa_headers`: fix a C++ compatibility issue.

The C++17 standard [requires](https://timsong-cpp.github.io/cppwp/n4659/vector#overview-3) that when you refer to a member of `std::vector<T>`, `T` must be a complete type. `BalsaHeaders` currently has a `std::vector<BalsaHeaders::HeaderLineDescription>`/`HeaderLines` member, but before `BalsaHeaders::HeaderLineDescription` has been defined. Currently this happens to work, but it breaks in certain build modes after a recent llvm commit.

Fix this by defining `BalsaHeaders::HeaderLineDescription` before uses of `std::vector<BalsaHeaders::HeaderLineDescription>`.

Tested with:

```
blaze build --config=cpp20 --config=llvm-unstable //dos/leafservice/client:thick-leaf-client
```

Before this CL: http://sponge2/2c0f6af2-57df-42a6-9b7d-0ecbd0cf7acc
After this CL: http://sponge2/ce3c08c3-6d32-4e85-9cbe-1a5e97932f5f

PiperOrigin-RevId: 468224411