Instead of content-length, we delimit messages with ---.
This also removes the need for (most) dos-formatted test files.
Details
Details
Diff Detail
Diff Detail
- Repository
- rCTE Clang Tools Extra
- Build Status
Buildable 14648 Build 14648: arc lint + arc unit
Event Timeline
Comment Actions
LGTM
Have we kept a lit test that uses content-length? It's unclear from the patch.
| clangd/tool/ClangdMain.cpp | ||
|---|---|---|
| 89 | To be more specific, maybe --lit-test-mode? | |
Comment Actions
Yes, protocol.test tests the real protocol parser. (The other tests that use this parser are are too_large, which i'll merge with protocol, and input_mirror, which i'll cut down a little bit, as followup)
| clangd/tool/ClangdMain.cpp | ||
|---|---|---|
| 89 | changed to --lit-test. ("mode" seems redundant in a flag name?) | |
To be more specific, maybe --lit-test-mode?