Skip to content

Commit

Permalink
don't say compromise
Browse files Browse the repository at this point in the history
@dherman doesn't like it 😄
  • Loading branch information
steveklabnik committed Sep 5, 2014
1 parent 67b97ab commit d929d83
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/doc/guide.md
Expand Up @@ -765,7 +765,7 @@ This is a deliberate design decision. While full-program inference is possible,
languages which have it, like Haskell, often suggest that documenting your
types explicitly is a best-practice. We agree that forcing functions to declare
types while allowing for inference inside of function bodies is a wonderful
compromise between full inference and no inference.
sweet spot between full inference and no inference.

What about returning a value? Here's a function that adds one to an integer:

Expand Down

13 comments on commit d929d83

@bors
Copy link
Contributor

@bors bors commented on d929d83 Sep 5, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from thestinger
at steveklabnik@d929d83

@bors
Copy link
Contributor

@bors bors commented on d929d83 Sep 5, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging steveklabnik/rust/remove_compromise = d929d83 into auto

@bors
Copy link
Contributor

@bors bors commented on d929d83 Sep 5, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

steveklabnik/rust/remove_compromise = d929d83 merged ok, testing candidate = 0f142e8f

@bors
Copy link
Contributor

@bors bors commented on d929d83 Sep 5, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from thestinger
at steveklabnik@d929d83

@bors
Copy link
Contributor

@bors bors commented on d929d83 Sep 5, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging steveklabnik/rust/remove_compromise = d929d83 into auto

@bors
Copy link
Contributor

@bors bors commented on d929d83 Sep 5, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

steveklabnik/rust/remove_compromise = d929d83 merged ok, testing candidate = b5c34bed

@bors
Copy link
Contributor

@bors bors commented on d929d83 Sep 7, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from thestinger
at steveklabnik@d929d83

@bors
Copy link
Contributor

@bors bors commented on d929d83 Sep 7, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging steveklabnik/rust/remove_compromise = d929d83 into auto

@bors
Copy link
Contributor

@bors bors commented on d929d83 Sep 7, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

steveklabnik/rust/remove_compromise = d929d83 merged ok, testing candidate = 57781c3

@bors
Copy link
Contributor

@bors bors commented on d929d83 Sep 7, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 57781c3

Please sign in to comment.