From ae692c05b78959ce43899526a3d14d6fdf031354 Mon Sep 17 00:00:00 2001
From: Pablo Hoffman <pablo@pablohoffman.com>
Date: Thu, 12 Aug 2010 03:04:19 -0300
Subject: [PATCH] made MiddlewareManager an abstract class, and minor change to
 log message

---
 scrapy/middleware.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scrapy/middleware.py b/scrapy/middleware.py
index 148b9fd50..1a5d72c21 100644
--- a/scrapy/middleware.py
+++ b/scrapy/middleware.py
@@ -18,7 +18,7 @@ class MiddlewareManager(object):
 
     @classmethod
     def _get_mwlist_from_settings(cls, settings):
-        return []
+        raise NotImplementedError
 
     @classmethod
     def from_settings(cls, settings):
@@ -36,7 +36,7 @@ class MiddlewareManager(object):
                 if e.args:
                     log.msg(e)
         enabled = [type(x).__name__ for x in middlewares]
-        log.msg("Enabled %ss: %s" % (cls.component_name, ",".join(enabled)), \
+        log.msg("Enabled %ss: %s" % (cls.component_name, ", ".join(enabled)), \
             level=log.DEBUG)
         return cls(*middlewares)