Re: svn commit: r149408 - in cocoon/branches/BRANCH_2_1_X/src: deprecated/java/org/apache/cocoon/components/source/ java/org/apache/cocoon/components/ java/org/apache/cocoon/components/source/impl/ java/org/apache/cocoon/components/treeprocessor/ java/org/apache/cocoon/environment/ java/org/apache/cocoon/environment/wrapper/ webapp/samples/test/pass-through/

[Carsten Ziegeler]

[EMAIL PROTECTED] wrote:
Author: sylvain
Date: Tue Feb  1 07:43:40 2005
New Revision: 149408
     /**
-     * Remove attribute from the current instance, as well as from the
-     * wrapped environment.
+     * Remove attribute from the current instance.
      *
      * @param name a <code>String</code> value
      */
     public void removeAttribute(String name) {
         super.removeAttribute(name);
-        this.environment.removeAttribute(name);
     }
 
Hmm, isn't this an incompatible change? Ok, I always thought that the 
implementation was not the way one would expect, but we shouldn't imho 
introduce such changes during maintenance releases. I'm fine with 
changing this for 2.2.

Carsten