Hi all,

The attached patch removes an unnecessary call for fop1.foxdest mode template 
which is a copy-paste from bookmark generation code. Should speed up XSLT a bit 
:)

As a related question, does it make sense to limit the template to "*[@id | 
@xml:id] instead of current match pattern? Given that IDs on subsequent runs 
are 
going to be different for elements without an explicit ID attribute, it won't 
be 
possible to create a permanent URL to point to such ID-less sections. On other 
hand, current match pattern does not generate destinations for 
tables/figures/equations, even if they have IDs - making impossible to <olink> 
to 
them.

Regards,
Alexey.
Index: fo/fop1.xsl
===================================================================
--- fo/fop1.xsl	(revision 663)
+++ fo/fop1.xsl	(working copy)
@@ -107,16 +107,6 @@
   <xsl:variable name="id">
     <xsl:call-template name="object.id"/>
   </xsl:variable>
-  <xsl:variable name="bookmark-label">
-    <xsl:apply-templates select="." mode="object.title.markup"/>
-  </xsl:variable>
-  <!--xsl:if test="$id != ''">
-    <fox:destination internal-destination="{$id}"/>
-  </xsl:if-->
-
-  <!-- Put the root element bookmark at the same level as its children -->
-  <!-- If the object is a set or book, generate a bookmark for the toc -->
-
   <xsl:choose>
     <xsl:when test="self::index and $generate.index = 0"/>	
     <xsl:when test="parent::*">

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to