Merge pull request #23 from maralla/develop

some minor fix

Author: lxyu

thriftpy/parser.py

@@ -4,6 +4,7 @@
 import functools
 import hashlib
 import itertools
+import os.path
 import pickle
 import types
 
@@ -115,7 +116,8 @@ def load(thrift_file, cache=True):
     if thrift_file in _thriftloader:
         return _thriftloader[thrift_file]
 
-    module_name = thrift_file[:thrift_file.find('.')]
+    basename = os.path.basename(thrift_file)
+    module_name, _ = os.path.splitext(basename)
 
     with open(thrift_file, "r") as fp:
         schema = fp.read()

thriftpy/server.py

@@ -56,8 +56,8 @@ class TThreadedServer(TServer):
     """Threaded server that spawns a new thread per each connection."""
 
     def __init__(self, *args, **kwargs):
-        TServer.__init__(self, *args)
-        self.daemon = kwargs.get("daemon", False)
+        self.daemon = kwargs.pop("daemon", False)
+        TServer.__init__(self, *args, **kwargs)
 
     def serve(self):
         self.trans.listen()

thriftpy/thrift.py

@@ -108,6 +108,9 @@ def __getattr__(self, api):
         if api in self._service.thrift_services:
             return functools.partial(self._req, api)
 
+        raise AttributeError("{} instance has no attribute '{}'".format(
+            self.__class__.__name__, api))
+
     def __dir__(self):
         return self._service.thrift_services