Skip to content

Conversation

Dangthrimble and others added 13 commits July 23, 2015 12:24
Added definitions for 'Expression', 'Expression-Oriented Language' and
'Statement'.
Sorted the definitions alphabetically.
Added the fact that expression statements can form part of larger
expressions.
Tidied up glossary.md and added link from hello-world.md to
'expression-oriented language' in glossary.md
Since most lang items are actually defined in core, these hack reexports don't actually
do anything useful.
The API we're calling requires us to pass an absolute point in time as an
argument (`pthread_cond_timedwait`) so we call `gettimeofday` ahead of time to
then add the specified duration to. Unfortuantely the current "add the duration"
logic forgot to take into account the current time's sub-second precision (e.g.
the `tv_usec` field was ignored), causing sub-second duration waits to return
spuriously.
Print the error message and then what is expected by the repeat count so the output makes more sense when there is an error in the const expression
Added definitions for 'Expression', 'Expression-Oriented Language' and 'Statement' to glossary.
Sorted the definitions alphabetically.

r? @steveklabnik
Since most lang items are actually defined in core, these hack reexports don't actually
do anything useful.
…=brson

The API we're calling requires us to pass an absolute point in time as an
argument (`pthread_cond_timedwait`) so we call `gettimeofday` ahead of time to
then add the specified duration to. Unfortuantely the current "add the duration"
logic forgot to take into account the current time's sub-second precision (e.g.
the `tv_usec` field was ignored), causing sub-second duration waits to return
spuriously.
@Manishearth
Copy link
Member Author

@bors r+ p=5

@bors
Copy link
Collaborator

bors commented Jul 29, 2015

📌 Commit b904b45 has been approved by Manishearth

@bors
Copy link
Collaborator

bors commented Jul 29, 2015

⌛ Testing commit b904b45 with merge 1867078...

bors added a commit that referenced this pull request Jul 29, 2015
@bors bors merged commit b904b45 into rust-lang:master Jul 29, 2015
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants