Skip to content

Fix a clang-format-induced parse break in three erased_call_thunk uses (19/20) - #119

Draft
jbcoe wants to merge 1 commit into
reflection/17-protocol-viewfrom
reflection/18-fix-clang-format-splice-break
Draft

Fix a clang-format-induced parse break in three erased_call_thunk uses (19/20)#119
jbcoe wants to merge 1 commit into
reflection/17-protocol-viewfrom
reflection/18-fix-clang-format-splice-break

Conversation

@jbcoe

@jbcoe jbcoe commented Jul 27, 2026

Copy link
Copy Markdown
Owner

clang-format sometimes wraps a typename[: :] splice across a line
break between ':' and ']', which GCC 16 trunk's splice parser
misparses, producing a wrong-argument-count error at an unrelated
call site. Guards the three affected instantiations with
// clang-format off/on.

@jbcoe jbcoe changed the title Fix a clang-format-induced parse break in three erased_call_thunk uses Fix a clang-format-induced parse break in three erased_call_thunk uses (19/20) Jul 27, 2026
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from 9d0f043 to ccf7f63 Compare July 28, 2026 11:33
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from e8e79e8 to 9cb2d3e Compare July 28, 2026 11:33
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from ccf7f63 to 357f971 Compare July 28, 2026 14:22
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from 9cb2d3e to d4ef8ba Compare July 28, 2026 14:23
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from 357f971 to 1e62cfd Compare July 28, 2026 14:27
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from d4ef8ba to 9d38cca Compare July 28, 2026 14:27
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from 1e62cfd to 442c07b Compare July 28, 2026 14:30
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from 9d38cca to 7de2afc Compare July 28, 2026 14:30
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from 442c07b to 45cd500 Compare July 28, 2026 15:14
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from 7de2afc to b6104b2 Compare July 28, 2026 15:14
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from 45cd500 to 53df339 Compare July 28, 2026 15:29
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from b6104b2 to 5d85448 Compare July 28, 2026 15:29
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from 53df339 to 1ca29c9 Compare July 28, 2026 18:27
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from 5d85448 to 4b171ca Compare July 28, 2026 18:27
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from 1ca29c9 to db2a0dd Compare July 28, 2026 19:37
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from 4b171ca to f938b89 Compare July 28, 2026 19:38
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from db2a0dd to cd1661a Compare July 28, 2026 20:23
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from f938b89 to bdc5116 Compare July 28, 2026 20:23
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from cd1661a to 7207351 Compare July 29, 2026 09:48
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from bdc5116 to 796befc Compare July 29, 2026 09:48
clang-format sometimes wraps a typename[: :] splice across a line
break between ':' and ']', which GCC 16 trunk's splice parser
misparses, producing a wrong-argument-count error at an unrelated
call site. Guards the three affected instantiations with
// clang-format off/on.
@jbcoe
jbcoe force-pushed the reflection/17-protocol-view branch from 7207351 to 0e0d4bb Compare July 29, 2026 10:14
@jbcoe
jbcoe force-pushed the reflection/18-fix-clang-format-splice-break branch from 796befc to e92ea3e Compare July 29, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant