You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by se...@apache.org on 2009/02/15 16:11:32 UTC
svn commit: r744676 -
/httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/BasicHttpProcessor.java
Author: sebb
Date: Sun Feb 15 15:11:32 2009
New Revision: 744676
URL: http://svn.apache.org/viewvc?rev=744676&view=rev
Log:
Always clear then copy both lists in copyInterceptors()
Modified:
httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/BasicHttpProcessor.java
Modified: httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/BasicHttpProcessor.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/BasicHttpProcessor.java?rev=744676&r1=744675&r2=744676&view=diff
==============================================================================
--- httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/BasicHttpProcessor.java (original)
+++ httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/BasicHttpProcessor.java Sun Feb 15 15:11:32 2009
@@ -264,15 +264,17 @@
}
}
+ /**
+ * Sets up the target to have the same list of interceptors
+ * as the current instance.
+ *
+ * @param target object to be initialised
+ */
protected void copyInterceptors(final BasicHttpProcessor target) {
- if (this.requestInterceptors.size() > 0) {
- target.requestInterceptors.clear();
- target.requestInterceptors.addAll(this.requestInterceptors);
- }
- if (this.responseInterceptors.size() > 0) {
- target.responseInterceptors.clear();
- target.responseInterceptors.addAll(this.responseInterceptors);
- }
+ target.requestInterceptors.clear();
+ target.requestInterceptors.addAll(this.requestInterceptors);
+ target.responseInterceptors.clear();
+ target.responseInterceptors.addAll(this.responseInterceptors);
}
/**