Re: Another change to CADBBeanTemplateSource.xsl...

[Dimuthu Gamage <di...@gmail.com>]

Hi Floyd,
I just committed your suggestion. Yea it was from my recent commit.

Thanks for pointing that out.

Thanks
Dimuthu

On Fri, Mar 28, 2008 at 10:54 PM, Floyd Brown <fb...@terahop.com> wrote:
> I noticed your name in the log for the file, if you could review my
>  change and possibly submit it...
>
>  (around line #4207 - I moved the "</xsl:if>" to be positioned before the
>  two AXIS2_FREE() calls)
>
>  This was causing start_input_string/end_input_string to be leaked for
>  every serialize conversion.
>
>  Thanks
>
>  Floyd Brown
>
>
>
>  Index: CADBBeanTemplateSource.xsl
>  ===================================================================
>  --- CADBBeanTemplateSource.xsl  (revision 642228)
>  +++ CADBBeanTemplateSource.xsl  (working copy)
>  @@ -4207,10 +4207,11 @@
>                     <xsl:if test="@isarray">
>                           }
>                       }
>  +                  </xsl:if>
>                       <!-- This two should be freed after the loop for
>  array-->
>                       AXIS2_FREE(env->allocator,start_input_str);
>                       AXIS2_FREE(env->allocator,end_input_str);
>  -                   </xsl:if>
>  +
>                   } <!-- else for non nillable -->
>
>                   <xsl:if test="$choice">
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: axis-c-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-c-dev-help@ws.apache.org