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

\opt vs. \opt{} #761

Closed
burblebee opened this issue Jun 22, 2016 · 8 comments
Closed

\opt vs. \opt{} #761

burblebee opened this issue Jun 22, 2016 · 8 comments
Assignees

Comments

@burblebee
Copy link
Contributor

I see both \opt and \opt{} used in the grammars in Latex. Can these all be changed to be just \opt?

@tkoeppe
Copy link
Contributor

tkoeppe commented Jun 22, 2016

We had a PR to fix those just recently - the {} is needed to produce space. \opt doesn't use anything fancy like \xspace.

@zygoloid
Copy link
Member

We should use \opt{} everywhere to avoid the possibility of problems / missing whitespace.

@burblebee
Copy link
Contributor Author

Should we instead add \xspace to \opt so that it acts like \opt{}?

@tkoeppe
Copy link
Contributor

tkoeppe commented Jun 22, 2016

I'm really worried about xspace because I don't really understand how it interacts with other punctuators. I'd be perfectly happy to use \opt{} when necessary.

@zygoloid
Copy link
Member

On 23 Jun 2016 12:03 a.m., "burblebee" notifications@github.com wrote:

Should we instead add \xspace to \opt so that it acts like \opt{}?

I'd like to move away from xspace. It's too unpredictable.


You are receiving this because you commented.

Reply to this email directly, view it on GitHub, or mute the thread.

@burblebee
Copy link
Contributor Author

OK. Shall I turn all \opts into \opt{}s then? Or leave as is and close this issue as "NAD"?

@tkoeppe
Copy link
Contributor

tkoeppe commented Jun 22, 2016

Well, is the spacing actually wrong? I'd say we should have the {} if and only if it's needed.

@burblebee
Copy link
Contributor Author

Great - let's close this as a non-issue then.

@burblebee burblebee self-assigned this Jun 23, 2016
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

No branches or pull requests

3 participants