Please see my remarks below ...

On Fri, Mar 7, 2008 at 5:03 PM, David Carlisle <davidc@xxxxxxxxx> wrote:
> <xsl:template match="/">
>  <xsl:apply-templates select="*"/>
> </xsl:template>

I think we can avoid this template. The built in rule for root node
can take care of this.

> <xsl:template match="VisualObject[@xsi:type='CBarCode']/FontFace">
>  <xsl:copy>
>    <xsl:call-template name="while">
>      <xsl:with-param name="foo" select="."/>
>    </xsl:call-template>
>  </xsl:copy>
> </xsl:template>

I think this is better design. By this way, you can reduce the
(cyclomatic) complexity of the code, and make the code easily
testable.


-- 
Regards,
Mukul Gandhi