Clean up copy-pasted code in fop1.xsl

[Alexey Neyman <stilor@att.net>]

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::*">