cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Carsten Ziegeler <cziege...@apache.org>
Subject 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/
Date Tue, 01 Feb 2005 16:58:37 GMT
sylvain@apache.org 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

Mime
View raw message