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:[EMAIL PROTECTED]";>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:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
