Changeset 1111

Show
Ignore:
Timestamp:
06/29/08 16:44:40 (2 months ago)
Author:
kris
Message:

more code cleanup

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/mango/net/servlet/ServletProvider.d

    r1102 r1111  
    7070class ServletProvider : ServiceProvider 
    7171{ 
    72         private Logger                  log;            // log output 
    7372        private QueuedCache             cache;          // url cache 
     73        private Logger                  logger;         // log output 
    7474        private HashMap                 contexts;       // context set 
    7575        private ServletMapping[]        mappings;       // regex->servlets 
    …… 
    103103 
    104104                // remember logger 
    105                 this.log = log; 
     105                this.logger = log; 
    106106        } 
    107107 
    …… 
    145145        **********************************************************************/ 
    146146 
    147         Logger getLog () 
    148         { 
    149                 return log; 
     147        Logger log () 
     148        { 
     149                return logger; 
    150150        } 
    151151 
    …… 
    170170 
    171171        ServletContext getContext (char[] name) 
    172         in { 
    173            assert (name !is null); 
    174            } 
    175         body 
    176         { 
     172        { 
     173                assert (name !is null); 
    177174                return cast(ServletContext) contexts.get (name); 
    178175        } 
    …… 
    186183 
    187184        ServletContext addContext (ServletContext context) 
    188         in { 
    189            assert (context); 
    190            } 
    191         body 
    192         { 
     185        { 
     186                assert (context); 
    193187                contexts.put (context.getName, context); 
    194188                return context; 
    …… 
    216210 
    217211        void addMapping (char[] pattern, IRegisteredServlet servlet) 
    218         in { 
    219            assert (servlet); 
    220            } 
    221         body 
    222         { 
     212        { 
     213                assert (servlet); 
     214 
    223215                // context is always used, even when it's "" for the default context 
    224216                char[] context = "^" ~ servlet.getContext.getName; 
    …… 
    249241                // add to list of mappings 
    250242                auto mapping = new ServletMapping (servlet, new Regex(pattern, null)); 
    251                 log.info ("{} maps to path [{}] ", servlet.getName, pattern); 
     243                logger.info ("{} maps to path [{}] ", servlet.getName, pattern); 
    252244                mappings ~= mapping; 
    253245        } 
    …… 
    272264 
    273265        IRegisteredServlet addServlet (Servlet servlet, ServletContext context) 
    274         in { 
    275            assert (context !is null); 
    276            } 
    277         body 
    278         { 
     266        { 
     267                assert (context !is null); 
    279268                return addServlet (servlet, new ServletConfig (context)); 
    280269        } 
    …… 
    287276 
    288277        IRegisteredServlet addServlet (Servlet servlet, ServletConfig config) 
    289         in { 
    290            assert (config !is null); 
    291            assert (servlet !is null); 
    292            assert (config.getServletContext() !is null); 
    293            } 
    294         body 
    295         { 
     278        { 
     279                assert (config !is null); 
     280                assert (servlet !is null); 
     281                assert (config.getServletContext() !is null); 
     282 
    296283                auto proxy = new ServletProxy (servlet, servlet.toString, config.getServletContext); 
    297284 
    …… 
    363350                      { 
    364351                      // nope; go home ... 
    365                       log.warn ("No service configured for [{}]", request.uri); 
     352                      logger.warn ("No service configured for [{}]", request.uri); 
    366353                      return response.sendError (HttpResponses.NotFound); 
    367354                      } 
    …… 
    395382                            { 
    396383                            response.sendError (HttpResponses.InternalServerError, o.toString); 
    397                             getDefaultContext.log ("Internal exception: ", o); 
     384                            logger.error ("ServletProvider :: unexpected exception: {}", o); 
    398385                            } 
    399386        }