Skip to content

Commit

Permalink
Merge pull request #125 from nats-io/tyler_remove_handler_Sync_requir…
Browse files Browse the repository at this point in the history
…ement

Remove Sync requirement for the handler function used with a Subscription
  • Loading branch information
spacejam authored Nov 17, 2020
2 parents 5ff1173 + aaade4e commit 6242d68
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 2 deletions.
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,10 @@
# 0.8.5

## Improvements

- #125 Remove Sync requirement for the handler function
passed to Subscription::with_handler.

# 0.8.4

## Bug Fixes
Expand Down
2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[package]
name = "nats"
version = "0.8.4"
version = "0.8.5"
description = "A Rust NATS client"
authors = ["Derek Collison <[email protected]>", "Tyler Neely <[email protected]>", "Stjepan Glavina <[email protected]>"]
edition = "2018"
Expand Down
2 changes: 1 addition & 1 deletion src/subscription.rs
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ impl Subscription {
/// ```
pub fn with_handler<F>(self, handler: F) -> Handler
where
F: Fn(Message) -> io::Result<()> + Sync + Send + 'static,
F: Fn(Message) -> io::Result<()> + Send + 'static,
{
// This will allow us to not have to capture the return. When it is dropped it
// will not unsubscribe from the server.
Expand Down

0 comments on commit 6242d68

Please sign in to comment.