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

[rand.util.seedseq] Replace nested ternary expressions with definition by cases. #1345

Closed
wants to merge 1 commit into from

Conversation

Eelis
Copy link
Contributor

@Eelis Eelis commented Jan 9, 2017

As suggested by @jensmaurer in #1329. Diff:

diff

@tkoeppe
Copy link
Contributor

tkoeppe commented Jan 9, 2017

This should be styled roughly as follows:

t = { a <tab> for ..., \n b <tab> for ..., and \n c <tab> for ... . }

(if you see what I mean)

@Eelis
Copy link
Contributor Author

Eelis commented Jan 9, 2017

You mean for all such definitions by cases in the document (like the other one in the screenshot)?

After all, I wouldn't want to introduce inconsistency with the definition by cases in the very next item.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jan 9, 2017

No, just in this case. I don't want the second column to start with a dangling comma.

In pseudo-latex:

t = \begin{cases} 1 & \text{for $a = 1$,}\\ 2 & \text{for $a > 3$, and}\\ 3 & \text{otherwise.} \end{cases}

just to demonstrate how to integrate sentence punctuation into cases.

@Eelis Eelis closed this Jan 9, 2017
@Eelis Eelis deleted the uglyternary branch January 9, 2017 15:26
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.

None yet

2 participants