Improved user-facing messages

async_push.py

Index: async_push.py
diff --git a/async_push.py b/async_push.py
index 7571d0cb4c40e16c011730fac12611d77bf1f10a..b2c9adf50b18c69a7da05e1f572ec35505da5080 100644
--- a/async_push.py
+++ b/async_push.py
@@ -59,9 +59,9 @@ class AsyncPushStore(AsyncPushNoop):
 
 
 class AsyncPush(AsyncPushNoop):
-  """Sends HTTP Post in a background worker thread.
+  """Sends HTTP Post asynchronously to the tree status application.
 
-  Thread-safe.
+  This object uses a background worker thread, and is thread-safe.
   """
   _TERMINATE = object()
 
@@ -111,7 +111,7 @@ class AsyncPush(AsyncPushNoop):
           done = True
           logging.debug('Worker thread exiting')
           items.remove(self._TERMINATE)
-        url = self.url + '/receiver'
+        url = self.url + '/receiver'   # application-specific

[Paweł Hajdan Jr., 2014/01/06 11:23:26]

nit: It's not obvious what this comment adds.

I'd rather make this /receiver path part also part of the ctor, so that the
class itself is not application specific.

         logging.debug('Sending %d items to %s' % (len(items), url))
         try:
           data = [('p', json.dumps(item)) for item in items]