Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

escape semantically meaningful braces in [exec.getcomplsigs] #7537

Closed
wants to merge 1 commit into from

Conversation

ericniebler
Copy link
Contributor

P2300R10 [exec.getcomplsigs] para 1 contains the following in code font:

transform_sender(decltype(get-domain-late(sndr, env)){}, sndr, env)

In the current working draft, this is rendered as:

transform_sender(decltype(get-domain-late(sndr, env)), sndr, env)

Note the missing braces. This is an editing mistake where the braces are not escaped for LaTeX, and hence are ignored.

[P2300R10 [exec.getcomplsigs]](https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2024/p2300r10.html#spec-execution.getcomplsigs) para 1 contains the following in code font:

```
transform_sender(decltype(get-domain-late(sndr, env)){}, sndr, env)
```

In the current working draft, this is rendered as:

```
transform_sender(decltype(get-domain-late(sndr, env)), sndr, env)
```

Note the missing braces. This is an editing mistake where the braces are not escaped for LaTeX, and hence are ignored.
@ericniebler
Copy link
Contributor Author

@jensmaurer i don't understand the LaTeX error this change caused. the change is simple enough, i'm surprised it causes problems.

@jensmaurer
Copy link
Member

jensmaurer commented Jan 3, 2025

@ericniebler

Now that we have the braces in the output, the resulting line gets too long. Because this is all code, LaTeX has troubles finding good linebreaking opportunities. (Actually, the next line has a problem, because LaTeX doesn't hyphenate an already-hyphenated word.)

I've fixed this here: #7541

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.

2 participants