[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cactus-dev
Subject:    cvs commit: jakarta-cactus/src/framework/share/org/apache/cactus/server AbstractHttpServletRequestWr
From:       vmassol () apache ! org
Date:       2001-11-16 19:19:27
[Download RAW message or body]

vmassol     01/11/16 11:19:27

  Modified:    src/framework/share/org/apache/cactus/server
                        AbstractHttpServletRequestWrapper.java
  Log:
  removed explicit logging of entries and exits of methods (using aspectj now)
  
  Revision  Changes    Path
  1.3       +1 -30     \
jakarta-cactus/src/framework/share/org/apache/cactus/server/AbstractHttpServletRequestWrapper.java
  
  Index: AbstractHttpServletRequestWrapper.java
  ===================================================================
  RCS file: /home/cvs/jakarta-cactus/src/framework/share/org/apache/cactus/server/AbstractHttpServletRequestWrapper.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- AbstractHttpServletRequestWrapper.java	2001/09/22 14:18:32	1.2
  +++ AbstractHttpServletRequestWrapper.java	2001/11/16 19:19:27	1.3
  @@ -68,7 +68,7 @@
    *
    * @author <a href="mailto:vmassol@apache.org">Vincent Massol</a>
    *
  - * @version $Id: AbstractHttpServletRequestWrapper.java,v 1.2 2001/09/22 14:18:32 \
vmassol Exp $  + * @version $Id: AbstractHttpServletRequestWrapper.java,v 1.3 \
                2001/11/16 19:19:27 vmassol Exp $
    */
   public abstract class AbstractHttpServletRequestWrapper implements \
HttpServletRequest  {
  @@ -121,8 +121,6 @@
        */
       public String getContextPath()
       {
  -        logger.entry("getContextPath()");
  -
           String result = this.request.getContextPath();
   
           if (this.url != null) {
  @@ -132,7 +130,6 @@
               }
           }
   
  -        logger.exit("getContextPath");
           return result;
       }
   
  @@ -142,16 +139,12 @@
        */
       public String getPathInfo()
       {
  -        logger.entry("getPathInfo()");
  -
           String result = this.request.getPathInfo();
   
           if (this.url != null) {
               result = this.url.getPathInfo();
               logger.debug("Using simulated PathInfo : [" + result + "]");
           }
  -
  -        logger.exit("getPathInfo");
           return result;
       }
   
  @@ -161,8 +154,6 @@
        */
       public String getServerName()
       {
  -        logger.entry("getServerName()");
  -
           String result = this.request.getServerName();
   
           if (this.url != null) {
  @@ -172,8 +163,6 @@
                       "]");
               }
           }
  -
  -        logger.exit("getServerName");
           return result;
       }
   
  @@ -184,8 +173,6 @@
        */
       public int getServerPort()
       {
  -        logger.entry("getServerPort()");
  -
           int result = this.request.getServerPort();
   
           if (this.url != null) {
  @@ -193,7 +180,6 @@
               logger.debug("Using simulated server port : [" + result + "]");
           }
   
  -        logger.exit("getServerPort");
           return result;
       }
   
  @@ -203,8 +189,6 @@
        */
       public String getRequestURI()
       {
  -        logger.entry("getRequestURI()");
  -
           String result = this.request.getRequestURI();
   
           if (this.url != null) {
  @@ -216,7 +200,6 @@
               logger.debug("Using simulated request URI : [" + result + "]");
           }
   
  -        logger.exit("getRequestURI");
           return result;
       }
   
  @@ -226,8 +209,6 @@
        */
       public String getServletPath()
       {
  -        logger.entry("getServletPath()");
  -
           String result = this.request.getServletPath();
   
           if (this.url != null) {
  @@ -236,7 +217,6 @@
                   "]");
           }
   
  -        logger.exit("getServletPath");
           return result;
       }
   
  @@ -274,8 +254,6 @@
        */
       public String getQueryString()
       {
  -        logger.entry("getQueryString()");
  -
           String result = this.request.getQueryString();
   
           if (this.url != null) {
  @@ -283,8 +261,6 @@
               logger.debug("Using simulated query string : [" + result +
                   "]");
           }
  -
  -        logger.exit("getQueryString");
           return result;
       }
   
  @@ -297,8 +273,6 @@
        */
       public RequestDispatcher getRequestDispatcher(String thePath)
       {
  -        logger.entry("getRequestDispatcher([" + thePath + "])");
  -
           // I hate it, but we have to write some logic here ! Ideally we
           // shouldn't have to do this as it is supposed to be done by the servlet
           // engine. However as we are simulating the request URL, we have to
  @@ -306,7 +280,6 @@
           // (it has to mock some parts of the servlet engine) !
   
           if (thePath == null) {
  -            logger.exit("getRequestDispatcher");
               return null;
           }
   
  @@ -330,7 +303,6 @@
               }
   
   	        if (fullPath == null) {
  -                logger.exit("getRequestDispatcher");
                   return null;
               }
           }
  @@ -340,7 +312,6 @@
           dispatcher = new RequestDispatcherWrapper(
               this.request.getRequestDispatcher(fullPath));
   
  -        logger.exit("getRequestDispatcher");
           return dispatcher;
       }
   
  
  
  

--
To unsubscribe, e-mail:   <mailto:cactus-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:cactus-dev-help@jakarta.apache.org>


[prev in list] [next in list] [prev in thread] [next in thread]