Skip to content

Commit 6c9a7b4

Browse files
committed
Merge pull request #4 from yyuu/master
raise AssertionError only if the response is None
2 parents c8e720d + a8721a8 commit 6c9a7b4

3 files changed

Lines changed: 7 additions & 5 deletions

File tree

.gitignore

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1-
build/*
2-
dist/*
1+
/build/*
2+
/dist/*
3+
/*.egg-info
34
*.pyc
5+
*.pyo

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ MessagePack RPC implementation based on Tornado.
1010
import msgpackrpc
1111

1212
class SumServer:
13-
def sum(x, y):
13+
def sum(self, x, y):
1414
return x + y
1515

1616
server = msgpackrpc.Server(SumServer())

msgpackrpc/future.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,15 +44,15 @@ def result(self):
4444
return self._result
4545

4646
def set_result(self, result):
47-
assert result
47+
assert result is not None
4848
self.set(result=result)
4949

5050
@property
5151
def error(self):
5252
return self._error
5353

5454
def set_error(self, error):
55-
assert error
55+
assert error is not None
5656
self.set(error=error)
5757

5858
def attach_callback(self, callback):

0 commit comments

Comments
 (0)